Mantis Bugtracker
  

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0006704 [Squeak] Compiler tweak sometimes 10-03-07 08:30 11-25-08 23:17
Reporter rr View Status public  
Assigned To
Priority normal Resolution fixed  
Status closed   Product Version 3.9
Summary 0006704: Parser proposes to remove temps it can not remove
Description 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.
Additional Information
Attached Files  Parser-removeUnusedTemps.st [^] (1,369 bytes) 10-03-07 08:30

- Relationships
related to 0007232closed nicolas cellier Out of Scope error always displayed when I have a block where a local variable is declared and assigned to but not used. 

- Notes
(0011622 - 138 - 150 - 150 - 546 - 546 - 546)
nicolas cellier
01-03-08 21:57

3.9 did not behave very well, but this change seems to make it worse see 0006831

OutOfScope in OldCompiler is a mess, see 0003448 and 0006720
 
(0012525 - 14 - 14 - 14 - 14 - 14 - 14)
MarcusDenker
08-25-08 12:30

in pharo 10064
 
(0012805 - 55 - 55 - 55 - 55 - 55 - 55)
KenCausey
11-25-08 23:17

Harvested as update 7145 and released with Squeak 3.10.
 

- Issue History
Date Modified Username Field Change
10-03-07 08:30 rr New Issue
10-03-07 08:30 rr File Added: Parser-removeUnusedTemps.st
01-03-08 21:57 nicolas cellier Note Added: 0011622
07-04-08 01:53 matthewf Relationship added related to 0007107
07-04-08 01:53 matthewf Relationship deleted related to 0007107
08-25-08 12:30 MarcusDenker Note Added: 0012525
11-25-08 23:17 KenCausey Status new => closed
11-25-08 23:17 KenCausey Note Added: 0012805
11-25-08 23:17 KenCausey Resolution open => fixed
11-25-08 23:17 KenCausey Fixed in Version  => 3.10
11-26-08 00:00 KenCausey Relationship added related to 0007232


Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
60 total queries executed.
42 unique queries executed.
Powered by Mantis Bugtracker