Parser-removeUnusedTemps enhancement

Damien Cassou damien.cassou at gmail.com
Wed Oct 3 08:12:54 UTC 2007


Please open a bug report on bugs.squeak.org, write some unit tests and
add a link to your report in http://wiki.squeak.org/squeak/5934

2007/10/3, Romain Robbes <romain.robbes at gmail.com>:
>
>
> When accepting a method, if the parser encounters an unused temporary
> variable, it offers to remove it.
> However half of the time the variable is stored into, and the parser does
> not remove it, instead telling that you should first remove the statement
> where the variable is stored.
>
> With this fix (basically reordering the test and the user confirmation) only
> the variables which are truly unused are removed.
> The code of the method is simpler too.
>
>  Romain
>
> --
> Romain Robbes
> http://romain.robb.es
>
>
>
>
>
>
>


-- 
Damien Cassou



More information about the Squeak-dev mailing list