Issue 4972 in pharo: NullStream breaks the debugger

Status: New
Owner: ----
Labels: Milestone-1.3 Type-Bug

New issue 4972 by [hidden email]: NullStream breaks the debugger
http://code.google.com/p/pharo/issues/detail?id=4972

Instances of NullStream break the debugger because Stream>>printOn: relies  
on #contents to be implemented, while NullStream>>contents: is self  
shouldNotImplement. Emergency evaluator pops up.

To reproduce:
[| a |
a := NullStream new.
self break] value

Applies to latest 1.4 too.

Attached a fix.
Perhaps the debugger should be fault tolerant regarding such errors?
Inspector, for example, handles it gracefully.


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

Comment #1 on issue 4972 by [hidden email]: NullStream breaks the  
debugger
http://code.google.com/p/pharo/issues/detail?id=4972

somehow the attachment didn't make it... here.

Attachments:
        NullStream.MilanMimica.1.cs  242 bytes


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

Comment #2 on issue 4972 by [hidden email]: NullStream breaks the  
debugger
http://code.google.com/p/pharo/issues/detail?id=4972

Yes, the debugger should be fixed to tolerate this, I think.


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

Updates:
        Status: FixReviewNeeded
        Labels: Milestone-1.4

Comment #3 on issue 4972 by [hidden email]: NullStream breaks the  
debugger
http://code.google.com/p/pharo/issues/detail?id=4972

(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: FixToInclude

Comment #4 on issue 4972 by [hidden email]: NullStream breaks the  
debugger
http://code.google.com/p/pharo/issues/detail?id=4972

(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 #5 on issue 4972 by [hidden email]: NullStream breaks the  
debugger
http://code.google.com/p/pharo/issues/detail?id=4972

in 14227


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