[ENH] DeclareInstVar

Lukas Renggli renggli at hotmail.com
Thu Jul 17 09:30:31 UTC 2003


Be creafull, there are still several bugs ...

:-(

In <200307161856.h6GIuIxN014589 at obelix.spectraweb.ch> Lukas Renggli 
wrote:
> 
> --==CelesteAttachment62531==
> 
> from preamble:
> 
> "Change Set:		DeclareInstVar
> Date:			16 July 2003
> Author:			Lukas Renggli
> 
> In <E19cqBN-0006Rl-00 at logrus.dnsalias.net> Lex Spoon wrote:
>> Lukas Renggli <renggli at hotmail.com> wrote:
>>>> However, it seems to compile the variable incorrectly.  It declares 
>>>> an instance variable, but it compiles it as a temporary variable.  
>>>> Here's the relevant code:
>>>>  [...snip...]
>> 
>> [...snip...]
>>
>> It doesn't work the same in all cases.  Here's a specific test case:
>> 
>> 	1. Create a class named Frob.
>> 	2. Create a method 'a ^a', and choose to declare an instance 
>> variable. 	3. Inspect ""Frob new"". 	4. Set the ""a"" variable to 42 
>> within the inspector. 	5. Type ""self a"", and observe that it 
>> returns nil instead of 42. 	6. Recompile the 'a' method (add a space 
>> and then accept).	 	7.  Now ""self a"" in the inspector will return 
>> 42.  This in Squeak 3.6, but it probably doesn't make a difference.  
>> The  code is quite clearly make it a temporary variable instead of an 
>> instance variable on the first compile.
> 
> Thanks for the test-case, I observed exactly the same problem in 
> Squeak 3.5. The attached changeset should solve it.
>
> Lukas"!
> --==CelesteAttachment62531==
> Content-type: application/octet-stream
> Content-transfer-encoding: base64
> Content-disposition: attachment;filename="DeclareInstVar.cs.gz"
> 
> H4sIAAAAAAAAAJ1XXW/
> UOBR9zkP+w+3sQ4rUhk6hBWYBUQplZ8UWBAUeUKk8iafjbcYOttMy
> K378nmsnM5N+LNJKldokvtfnnnvusZsdWTOnj98bKS4e5HtkppRlwyEd1FZVtLuz8yDL6G
> sl
> vHSemrrEHyP6bW/4eOeUjKbhPv3ZVIuwkISnx6O9vdHeY6rn2UY6OJwJfS7po/
> SjJHkli0pY
> OdbOfxY2fcWZkiRZz5AeNH5mLL9+21wIRx+kPj+vVJqONT19PXxSfH95vL2zs7P/
> ocKvF5U5 t43LS+1EpYTLtfTP6a38QR9rA3BX1mCPFK/Wk9FTG/
> 94MTN+LlSVF2b+fLn4+XP6w1zJS2m3
> SHlyUs4deUNYVKtKkp9JuhRWiQkelC6MtbLw1SInGnsqY5GOQiKhscJ5oYtVzBZNGs+Z24
> SO
> /wY6QV7Oa2OFXSzXImcEJK3MXNjaykpeCs3hZQuXvuZ57rSq8esU1eKn/
> wYvGJqRTmeeroy9
> CKmcmHMFJKqKCuGkw3btToJcLQs1VQWFxvPnUUydDHM6tBLdwyoU6xxpJCrpyJpJzgt21x
> bM
> JRpaUibom8i2QEhJxcwYJ5nSlqxbaQqZHuQEuQCJp8GA85OWV4NB+PYwZ12FQgYDMRismo
> LM
> D3fpSvkZiuPvKuYwNgTu5XSyqDnKyWpKCI24zMRJe8kNFqE/
> VvrGalSHOWB4UpQ8HQ93Q5b9 HHpa10Qmsq7aTVGWgUJRyJAa30FzUcja38sTDn8Ero/
> N1RoIug4WFaAxEUW7K35OZsrx0jix
> 9CDfXwqqBkEgYLHs9FxccA9KNZ2irWAXm55gC6Rh9RAyfW8U+lRUUlgEhgCW4y1aXOcADU
> OO
> Gz1jQ+ASpsq6pb7zND2BDVw4mqIm/
> syK2mZFbdG4Y70k+UPwGK2UyeVUct4rlnuH78J7Ucwk
> a4kNxkEHbmaaqiRnqksuAbuGoR9spGm68V5Y7NK2xx3BYSiT1gJPO77K6IxQzbzGl8rSo/
> vD
> vfvB04ZPRntDeFm78HNb6ojh1dBx+XnFj0clohqh8canyeAwhoSKbqyGSAVdaHOl15wBgL
> qZ
> 6EwBgk+TVQ+m5JpiBi0xZG6gleAGjSnR2y9grqnRt86HlD6nCmZmme1VGsepucOOd+Q2bu
> OT
> i2OABHE99Za/qSCuKgSkySGP/
> ecQUspa6pI3YsOdgWEZu1wY8PHDM0aeoM1gFc8+6VJOlZbl
> u8nf4OZenibw6rAqij9yuD69wYCch6PMWXvr/
> psP0jT5GUoZ4x1VYiKBMbLjqMJGeKqQUwuv
> LvEwURFsAz18hJVGHmFJikc1Pv1E0sF4SlcoQvAcCRcNk4OCileq2VrOTKPb1pWDNAlDHY
> rh
> nFBkmiRqegQCoZyv9A3DiAG0a0G3COo0ZyBHcZhmsrhg0eA8YhXEoyC0HnhYRyhUL5i0cx
> IT
> 0/geUYC0JOns5k5xYJV722kFbdlcBkAVDHqz4+5sbef23bvpTfz3SBt/jNaf3gvVn9imX/
> x5
> kNRo2RM+Rm6nIem3ax2A696OW+lg9fWvODnmcSrbSr1pH6FJf0tAIP5lo6pg3fAN3QRF9r
> QE
> Tlu9ndE7i4JkeWiqDiPmNl8K8c4FUaF3fU6TnnbPqGMOBDkFbjqOoqVdZy5fa3qvA3jooO
> Ok
> guN1psPLszx876CHBV8pCLpddoJVB7p8D+7u6tcd+bsTI8t/
> sUV7SbwtO532Z+nOzaK8frnV m1aFd9TRUR7s6zU/
> sFSfPaNrTkbXIN2OKd6XuqmM2P4LXGe0txMRILYmx7Htlk79I69rpzSc
> eiRxbNJPBrgOj98ylGtAGFwA45qJ88o3Xn4xtl3P18hy3Dvy4M5THMUj2ukxF3+vDs4Q/
> Qvs PfIKlgw3EkP5Srm6EvGWUJt6u6nDdGIWWw8/o833pv5U/
> 8UzG9IcWCsWyw2EC89x8kbdERFf sk9BoNZ/
> qr9g2g9FzXwAwSfdP6nxautGR7YooxoXKSe78yEc6BH9iHstOvtaEg2yWtxqeoCb
> kPbBIQPrU8HeGap+/UMWYD/ejYJD4e4TkwzW3auUnX/dsLSxvlTsGYtohn1DXbli/
> 31rj99a 4Oh0Izdo43/eqPbjjWp3Z/RwFzeq66PO9s+nOU4D/
> Af3M71j9KCJNuQ4nBccBowxamWR/SC1
> inBxAz6x2tBvbxUf1FXXyGO+G4d7V6KXW8SoUYtum4bk8U/EiIa/
> Y9nCNJZ7Frj5F10Uj1dS DwAA
>
> --==CelesteAttachment62531==
> Content-Type: text/plain; charset="us-ascii"
> MIME-Version: 1.0
> Content-Transfer-Encoding: 7bit
> 
> 
> 
> --==CelesteAttachment62531==--
> 
> 



More information about the Squeak-dev mailing list