| Anonymous | Login | 02-27-2021 21:37 UTC |
| Main | My View | View Issues | Change Log | Docs |
| 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 |
||||||
|
||||||
|
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. |  |
