I've fixed issue 67. SLICE is in PharoInbox. While looking at the classes I found some things. CrLfFileStream is somehow obsolete. Its new class method creates a MultiByteFileStream. The code in both classes is doubled. I didn't check but I would expect it is close to all of the code. This is again a question if we are willing to break back- ward compatibility. I would opt for removing the class completely. If not I would change MultiByteFileStream in a way that it inherits from CrLfFileStream. This way we could at least remove the doubled code/methods.
Or does anybody know reasons why the one or other isn't feasible to do? Furthermore I would remove the handling of line endings with symbols. These could just be the strings they contain. String cr, String lf and String crlf all exist. Using this instead of the symbols we could remove half of the class variables and get rid of the LineEndStrings dictionary. What do you think? And could someone please review my changes. I don't have a Mac. Well, the difference is only noticable if you have a system < 10. Norbert _______________________________________________ Pharo-project mailing list [email protected] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
