Re: SpaceTally dNu: in SpaceTally>>#spaceForInstance:depth:seen:
"Thiede, Christoph via Squeak-dev" <[email protected]> Mon, 3 Aug 2026 23:53:53 +0000
| Newsgroups | gmane.comp.lang.smalltalk.squeak.general |
|---|---|
| Message-ID | <GV2PR09MB83717DC6660DB723F298E9C4FBD52@GV2PR09MB8371.eurprd09.prod.outlook.com> |
--===============7255940306975678547== Content-Language: de-DE Content-Type: multipart/alternative; boundary="_000_GV2PR09MB83717DC6660DB723F298E9C4FBD52GV2PR09MB8371eurp_" --_000_GV2PR09MB83717DC6660DB723F298E9C4FBD52GV2PR09MB8371eurp_ Content-Type: text/plain; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable Hey Tim, thank you too for the hint! See: https://lists.squeakfoundation.org/archive= s/list/[email protected]/thread/COXWTOIO2JGBJDPXDTD5WNQ= CLLJIPLCI/ We just need to merge the fix. :-) Sorry for breaking this... Best, Christoph ________________________________ Von: tim Rowledge <[email protected]> Gesendet: Dienstag, 4. August 2026 01:09 Uhr An: The general-purpose Squeak developers list <[email protected]= dation.org> Betreff: [squeak-dev] SpaceTally dNu: in SpaceTally>>#spaceForInstance:dept= h:seen: Looks like some confusion was introduced in the SpaceTally class a while ba= ck; =91Mirror=92 is initialized in the class init method but set to nil in = the #cleanUp method *and never re-set*. Otherwise it is only referred to in= SpaceTally>>#spaceForInstance:depth:seen: and not in any of the other Spac= eTally>>#spaceForInstance* methods. Moving the #initialize method to the instance side appears to fix this. tim -- tim Rowledge; [email protected]; http://www.rowledge.org/tim To err is human; to forgive, beyond the scope of the Operating System. Squeak-dev mailing list -- [email protected] To unsubscribe send an email to [email protected] --_000_GV2PR09MB83717DC6660DB723F298E9C4FBD52GV2PR09MB8371eurp_ Content-Type: text/html; charset="Windows-1252" Content-Transfer-Encoding: quoted-printable <html> <head> <meta http-equiv=3D"Content-Type" content=3D"text/html; charset=3DWindows-1= 252"> <style type=3D"text/css" style=3D"display:none;"> P {margin-top:0;margin-bo= ttom:0;} </style> </head> <body dir=3D"ltr"> <div class=3D"elementToProof" style=3D"font-family: Aptos, Aptos_EmbeddedFo= nt, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; c= olor: rgb(0, 0, 0);"> Hey Tim,</div> <div class=3D"elementToProof" style=3D"font-family: Aptos, Aptos_EmbeddedFo= nt, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; c= olor: rgb(0, 0, 0);"> <br> </div> <div class=3D"elementToProof" style=3D"font-family: Aptos, Aptos_EmbeddedFo= nt, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; c= olor: rgb(0, 0, 0);"> thank you too for the hint! See: <a href=3D"https://lists.squeakfoundation.= org/archives/list/[email protected]/thread/COXWTOIO2JGB= JDPXDTD5WNQCLLJIPLCI/" id=3D"OWA9aec55a1-a41f-7c87-390d-e5c16514c2fd" class= =3D"OWAAutoLink"> https://lists.squeakfoundation.org/archives/list/[email protected]= ndation.org/thread/COXWTOIO2JGBJDPXDTD5WNQCLLJIPLCI/</a> We just need = to merge the fix. :-) Sorry for breaking this...</div> <div class=3D"elementToProof" style=3D"font-family: Aptos, Aptos_EmbeddedFo= nt, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; c= olor: rgb(0, 0, 0);"> <br> </div> <div class=3D"elementToProof" style=3D"font-family: Aptos, Aptos_EmbeddedFo= nt, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; c= olor: rgb(0, 0, 0);"> Best,</div> <div class=3D"elementToProof" style=3D"font-family: Aptos, Aptos_EmbeddedFo= nt, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; c= olor: rgb(0, 0, 0);"> Christoph</div> <div id=3D"appendonsend"></div> <hr style=3D"display:inline-block;width:98%" tabindex=3D"-1"> <div id=3D"divRplyFwdMsg" dir=3D"ltr"><font face=3D"Calibri, sans-serif" st= yle=3D"font-size:11pt" color=3D"#000000"><b>Von:</b> tim Rowledge <tim@r= owledge.org><br> <b>Gesendet:</b> Dienstag, 4. August 2026 01:09 Uhr<br> <b>An:</b> The general-purpose Squeak developers list <squeak-dev@lists.= squeakfoundation.org><br> <b>Betreff:</b> [squeak-dev] SpaceTally dNu: in SpaceTally>>#spaceFor= Instance:depth:seen:</font> <div> </div> </div> <div> <div class=3D"x_BodyFragment"><font size=3D"2"><span style=3D"font-size:11p= t"> <div class=3D"x_PlainText">Looks like some confusion was introduced in the = SpaceTally class a while back; =91Mirror=92 is initialized in the class ini= t method but set to nil in the #cleanUp method *and never re-set*. Otherwis= e it is only referred to in SpaceTally>>#spaceForInstance:depth:seen: and not in any of the other SpaceTally>>#spaceForInstance* methods. = <br> <br> Moving the #initialize method to the instance side appears to fix this.<br> </div> </span></font></div> <div class=3D"x_BodyFragment"><font size=3D"2"><span style=3D"font-size:11p= t"> <div class=3D"x_PlainText"><br> <br> tim<br> --<br> tim Rowledge; [email protected]; <a href=3D"http://www.rowledge.org/tim">htt= p://www.rowledge.org/tim</a><br> To err is human; to forgive, beyond the scope of the Operating System.<br> <br> <br> </div> </span></font></div> <div class=3D"x_BodyFragment"><font size=3D"2"><span style=3D"font-size:11p= t"> <div class=3D"x_PlainText">Squeak-dev mailing list -- [email protected]= akfoundation.org<br> To unsubscribe send an email to [email protected]= </div> </span></font></div> </div> </body> </html> --_000_GV2PR09MB83717DC6660DB723F298E9C4FBD52GV2PR09MB8371eurp_-- --===============7255940306975678547== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline Squeak-dev mailing list -- [email protected] To unsubscribe send an email to [email protected] --===============7255940306975678547==--