Jorge,

I looked at the code to integrate in 1.0. And I really have troubles 
understanding it (even after drawing a boolean table for the different 
combinations of or: ifTrue: and: ifFalse:). Is it correct that the warning is 
shown when node isTemp is false and requestor interactive is true? Is the 
comment still accurate?

This is the code:

"When non-interactive raise the error only if its a duplicate"
(node isTemp or: [requestor interactive])
        ifTrue:[ ((node isTemp) and: [node scope <= 0] ) 
                                        ifFalse: [^self notify:'Name is already 
defined' ]]

Cheers,
Adrian

On Mar 3, 2010, at 20:47 , Stéphane Ducasse wrote:

> 11243
> -----
> 
> - Issue 2102: Fix the fact that local temp can shadow silently instance var 
> --- fixed
>       Problem fixed:
> We cannot have twice the same block arg in a method
> 
>       [:each | each ...]
>       [:each | each ...]
> 
> THANKS jorge :)
> Do you like good bio beer?
> 
> Stef
> 
> 
> _______________________________________________
> 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

Reply via email to