Issue 6004 in pharo: BlockClosure>>silentlyValue not working

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 6004 in pharo: BlockClosure>>silentlyValue not working

pharo
Status: New
Owner: [hidden email]
Labels: Milestone-2.0 Type-Bug

New issue 6004 by [hidden email]: BlockClosure>>silentlyValue not  
working
http://code.google.com/p/pharo/issues/detail?id=6004

It is wrong to send #cr in:

silentlyValue
        "evaluates the receiver but avoiding progress bar notifications to show  
up."
       
        ^[ self value ]
                on: ProgressInitiationException
                do: [ :ex |
                        ex sendNotificationsTo: [ :min :max :curr |
                                self traceCr: min printString,'  ',max printString,'  ',curr  
printString; cr
                        ].
                ].



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6004 in pharo: BlockClosure>>silentlyValue not working

pharo

Comment #1 on issue 6004 by [hidden email]:  
BlockClosure>>silentlyValue not working
http://code.google.com/p/pharo/issues/detail?id=6004

Uploaded to PharoInbox:

Name:  
SLICE-Issue-6004-BlockClosuregtgtsilentlyValue-not-working-MartinDias.1
Author: MartinDias
Time: 31 May 2012, 7:59:43.121 pm
UUID: aab72ee2-34d3-5a4b-bddd-10fb842aa094
Ancestors:
Dependencies: Kernel-MartinDias.1103

Incredibly, this is my first SLICE and bug report in Pharo. I hope I  
followed the right procedure.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6004 in pharo: BlockClosure>>silentlyValue not working

pharo

Comment #2 on issue 6004 by [hidden email]:  
BlockClosure>>silentlyValue not working
http://code.google.com/p/pharo/issues/detail?id=6004

Maybe we should add a test in BlockClosureTest.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6004 in pharo: BlockClosure>>silentlyValue not working

pharo

Comment #3 on issue 6004 by marianopeck: BlockClosure>>silentlyValue not  
working
http://code.google.com/p/pharo/issues/detail?id=6004

yes! create a test and then just publish another version of the slice and  
that's all ;)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6004 in pharo: BlockClosure>>silentlyValue not working

pharo

Comment #4 on issue 6004 by [hidden email]: BlockClosure>>silentlyValue  
not working
http://code.google.com/p/pharo/issues/detail?id=6004

personally, i feel this whole approach is abuse..
first, this behavior is clearly not related to block closure, it is just a  
convenience method for progress-bar uses. This method should be an extension
of morphic/progress bar but not block closure.
so ,second,  if you write a test, put it into appropriate category.
It should not be in block closure tests!



_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 6004 in pharo: BlockClosure>>silentlyValue not working

pharo
Updates:
        Status: Closed

Comment #5 on issue 6004 by [hidden email]:  
BlockClosure>>silentlyValue not working
http://code.google.com/p/pharo/issues/detail?id=6004

this has already been fixed


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