It would be good to add that to the method and class comment. Henrik? Stef
On Mar 9, 2010, at 3:11 PM, Henrik Johansen wrote: > Den 09.03.2010 14:38, skrev Adrian Lienhard: >> I was just puzzled why "CrLfFileStream defaultToLF" does not work until I >> figured that CrLfFileStream new returns an instance of a different class: >> >> CrLfFileStream class>>new >> ^ (MultiByteFileStream new) wantsLineEndConversion: true; yourself. >> >> >> Does somebody know what the point is? I really sometimes wonder why good >> coders write such code, and not even care to add a comment... >> >> Cheers, >> Adrian >> ___________________ >> http://www.adrian-lienhard.ch/ >> > CrLfFileStream does not handle any encodings other than latin1. > > My guess is: > - MultiByteFileStream probably didn't exist when it was created. > - Using a MultiByteFileStream with no encoding, but a lineEndConvention > is now the preferred method, but CrLfFileStream was kept for > backwards-compatability. > > Cheers, > Henry > > _______________________________________________ > Pharo-project mailing list > [email protected] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project _______________________________________________ Pharo-project mailing list [email protected] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
