Shadowing the ivar by temps in Pharo

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

Shadowing the ivar by temps in Pharo

Igor Stasenko
This is not first time, i discovering that Pharo silently accepts the
methods which having argument/temp names
same as intance variable names.

Squeak 3.10, however warns about it, upon installation and prevents
edited method to be accepted until this is fixed.

-- Transcript --
(doubleBuffered is shadowed in "GLDisplayScreen")
(depth is shadowed in "GLDisplayScreen>>#glEnableMainWindowContext:")

Is this a bug or feature?

--
Best regards,
Igor Stasenko AKA sig.

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

Re: Shadowing the ivar by temps in Pharo

Schwab,Wilhelm K
IMHO, it is a bug.  





-----Original Message-----
From: [hidden email] [mailto:[hidden email]] On Behalf Of Igor Stasenko
Sent: Wednesday, July 29, 2009 8:52 PM
To: Pharo Development
Subject: [Pharo-project] Shadowing the ivar by temps in Pharo

This is not first time, i discovering that Pharo silently accepts the methods which having argument/temp names same as intance variable names.

Squeak 3.10, however warns about it, upon installation and prevents edited method to be accepted until this is fixed.

-- Transcript --
(doubleBuffered is shadowed in "GLDisplayScreen") (depth is shadowed in "GLDisplayScreen>>#glEnableMainWindowContext:")

Is this a bug or feature?

--
Best regards,
Igor Stasenko AKA sig.

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

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

Re: Shadowing the ivar by temps in Pharo

johnmci
In reply to this post by Igor Stasenko
Well if someone is fixing that there might be a few other gotha's they  
might want to consider.

For example...

Garble class>>foobar
name := 'messylongthing that is going to cause grief'.


(a) bonus point one is who and what is 'name'
(b) bonus point two is what evil thing did that do?
(c) double points for fixing your saved image, especially if you do  
name := 'VolumePath/more/things/'
(d) Even more points/grief if  you check that into MC and contaminate  
all your team members images as a side effect.


On 29-Jul-09, at 6:52 PM, Igor Stasenko wrote:

> This is not first time, i discovering that Pharo silently accepts the
> methods which having argument/temp names
> same as intance variable names.
>
> Squeak 3.10, however warns about it, upon installation and prevents
> edited method to be accepted until this is fixed.
>
> -- Transcript --
> (doubleBuffered is shadowed in "GLDisplayScreen")
> (depth is shadowed in "GLDisplayScreen>>#glEnableMainWindowContext:")
>
> Is this a bug or feature?
>
> --
> Best regards,
> Igor Stasenko AKA sig.
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

--
=
=
=
========================================================================
John M. McIntosh <[hidden email]>   Twitter:  
squeaker68882
Corporate Smalltalk Consulting Ltd.  http://www.smalltalkconsulting.com
=
=
=
========================================================================





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