[Pharo-project] Bug in Pharo compiler?
Hi All, Currently im running the Redline Smalltalk compiler (http://redline.st) over the Pharo source to make sure that it is compatible with Pharo. What I noticed while compiling the class BitBlt is that an instance variable is also used as a method argument name! If you try and do this with the environment you get a "name already defined" error, yet it is in the source. Not sure what to do about it? Rgs, James.
On 06.10.2010 22:56, James Ladd wrote:
Hi All,
Currently im running the Redline Smalltalk compiler (http://redline.st) over the Pharo source to make sure that it is compatible with Pharo.
What I noticed while compiling the class BitBlt is that an instance variable is also used as a method argument name!
If you try and do this with the environment you get a "name already defined" error, yet it is in the source.
Not sure what to do about it?
Rgs, James. The compiler has been changed to be more strict about shadowing inst vars, but code which has not been since the change was made may still exist.
Submit a bug to the tracker mentioning the affected method, and it will be fixed. Cheers, Henry
Hit the button a bit fast there... On 06.10.2010 23:13, Henrik Sperre Johansen wrote:
On 06.10.2010 22:56, James Ladd wrote:
Hi All,
Currently im running the Redline Smalltalk compiler (http://redline.st) over the Pharo source to make sure that it is compatible with Pharo.
What I noticed while compiling the class BitBlt is that an instance variable is also used as a method argument name!
If you try and do this with the environment you get a "name already defined" error, yet it is in the source.
Not sure what to do about it?
Rgs, James. The compiler has been changed to be more strict about shadowing inst vars, but code which has not been since the change was made may still exist.
... which has not been altered since the change...
Submit a bug to the tracker mentioning the affected method, and it will be fixed. Which would be to rename the method argument name, by the way.
Cheers, Henry
participants (2)
-
Henrik Sperre Johansen -
James Ladd