Issue 5047 in pharo: Stream should not print its contents in printOn:

Status: FixToInclude
Owner: [hidden email]
Labels: Type-Bug Milestone-1.4

New issue 5047 by [hidden email]: Stream should not print its  
contents in printOn:
http://code.google.com/p/pharo/issues/detail?id=5047

Reason:

=> Debugging leads to change the state of Stream when looking at it
=> Network streams load all content
=> unitialized Streams has undefined behavior for printing

Solution: remove #printOn:

Attachments:
        RemoveStreamPrintOn.1.cs  129 bytes


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Invalid

Comment #1 on issue 5047 by [hidden email]: Stream should not print  
its contents in printOn:
http://code.google.com/p/pharo/issues/detail?id=5047

Issue 5052 has a better fix for the problem


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #2 on issue 5047 by [hidden email]: Stream should not print  
its contents in printOn:
http://code.google.com/p/pharo/issues/detail?id=5047

Still, Stream>>#printOn: is wrong, I have 'removed' it myself for  
subclasses many times for exactly the same reasons.

Both fixes are important and not exclusive IMHO.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixToInclude

Comment #3 on issue 5047 by [hidden email]: Stream should not print  
its contents in printOn:
http://code.google.com/p/pharo/issues/detail?id=5047

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #4 on issue 5047 by [hidden email]: Stream should not print  
its contents in printOn:
http://code.google.com/p/pharo/issues/detail?id=5047

in 14244


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker