Mantis Bugtracker
  

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0001554 [Squeak] Kernel minor always 07-26-05 19:39 07-28-13 14:57
Reporter KenCausey View Status public  
Assigned To
Priority normal Resolution fixed  
Status resolved   Product Version trunk
Summary 0001554: [FIX] ClassVarsFix-petervr-v2 for compiler class var & global var shadowing issue
Description from preamble:

"Change Set: ClassVarsFix-petervr-v2
Date: 9 October 2003
Author: Peter van Rooijen

fixes the lookup of class variables.
previously, inherited class variables
did not shadow other shared variables.

v2 - minor improvements"!
Additional Information
Attached Files  ClassVarsFix-petervr-v2.cs.gz [^] (1,145 bytes) 07-26-05 19:39
 Tests-ClassVarsFix.st.gz [^] (1,022 bytes) 07-26-05 19:43

- Relationships
related to 0005905resolved nicolas cellier please let class vars shadow globals 

- Notes
(0001971 - 422 - 526 - 569 - 569 - 569 - 569)
KenCausey
07-26-05 19:41

"Peter van Rooijen" <squeak@vanrooijen.com>:

"I added some SUnit tests to demonstrate the bug and give credibility to the
fix.

Only two of the tests fail in an unfixed image. The rest serve as regression
tests.

All the tests pass after the fix is installed.

I added the tests as a file-out. I tried filing it into a change set, but
doing that lost the class comment."

(attaching Tests-ClassVarsFix.st.gz)
 
(0001972 - 128 - 168 - 376 - 376 - 376 - 376)
KenCausey
07-26-05 19:44

marcus@ira.uka.de:

"See the discussion:

http://lists.squeakfoundation.org/pipermail/squeak-dev/2003-October/068300.html [^]
"
 
(0001973 - 159 - 159 - 159 - 159 - 159 - 159)
KenCausey
07-26-05 19:47

I filed in the test into a 3.8-6665-basic image and 2 out of the 10 tests failed. I then filed in the fix changeset and ran the tests again and all 10 passed.
 
(0014425 - 202 - 220 - 530 - 530 - 530 - 530)
nicolas cellier
07-28-13 14:54
edited on: 07-28-13 14:55

The tests have been loaded in Squeak4.5 Trunk http://source.squeak.org/trunk/KernelTests-nice.253.mcz [^]
The lookup workaround has been loaded in Trunk http://source.squeak.org/trunk/Kernel-nice.797.mcz [^]

 

- Issue History
Date Modified Username Field Change
07-26-05 19:39 KenCausey New Issue
07-26-05 19:39 KenCausey File Added: ClassVarsFix-petervr-v2.cs.gz
07-26-05 19:41 KenCausey Note Added: 0001971
07-26-05 19:43 KenCausey File Added: Tests-ClassVarsFix.st.gz
07-26-05 19:44 KenCausey Note Added: 0001972
07-26-05 19:47 KenCausey Note Added: 0001973
07-20-13 23:41 tim Summary [FIX] ClassVarsFix-petervr-v2 => [FIX] ClassVarsFix-petervr-v2 for compiler class var & global var shadowing issue
07-22-13 02:42 tim Relationship added related to 0005905
07-28-13 14:54 nicolas cellier Note Added: 0014425
07-28-13 14:55 nicolas cellier Note Edited: 0014425
07-28-13 14:57 nicolas cellier Status new => resolved
07-28-13 14:57 nicolas cellier Resolution open => fixed
07-28-13 14:57 nicolas cellier version  => trunk


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