<div dir="ltr"><div>Found it! The bug was happening when there where two undeclared temporaries being declared in the same block. Here's a ChangeSet that fixes that and other related issue with the corresponding tests.</div><div><br></div><div>Cheers,</div><div>Eric<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El vie., 20 dic. 2019 a las 14:44, Eric Brandwein (<<a href="mailto:brandweineric@gmail.com">brandweineric@gmail.com</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div>Sure, I'll take a look at it later today.<br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il ven 20 dic 2019, 14:29 Hernan Wilkinson <<a href="mailto:hernan.wilkinson@10pines.com" target="_blank">hernan.wilkinson@10pines.com</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Eric,<div> the other day I got an error when defining an undef. temp var when saving a method. the method was:</div><div>-------</div><div> SafelyRemoveClass class>>#assertNoReferencesTo: aClassToSafelyRemove<br><br>| references withAllSubclasses withAllSubclassesNames referenced |<br><br>withAllSubclasses := aClassToSafelyRemove withAllSubclasses.<br>withAllSubclassesNames := withAllSubclasses collect: [:aClass | aClass name ].<br>references :=OrderedCollection new.<br>referenced := OrderedCollection new.<br>referencesToName := OrderedCollection new.<br>referencedByName := OrderedCollection new.<br><br>withAllSubclasses do: [ :aClass | | allReferences referencesOutsideHierarchy |<br>allReferences := aClass allCallsOn.<br>referencesOutsideHierarchy := allReferences reject: [ :aReference | withAllSubclassesNames includes: aReference classSymbol ].<br>referencesToVariableBinding := referencesOutsideHierarchy select: [ :aReference | aReference hasVariableBindingTo: aClass ].<br>referencesToClassName := referencesOutsideHierarchy difference: referencesToVariableBinding.<br><br>referencesToVariableBinding notEmpty ifTrue: [<br>referenced add: aClass.<br>references addAll: referencesToVariableBinding ].<br>referencesToClassName notEmpty ifTrue: [<br>referencedByName add: aClass.<br>referencesToName addAll: referencesToClassName ].<br><br>].<br><br>references notEmpty ifTrue: [ self signalCanNotRemove: aClassToSafelyRemove dueToReferences: references toAll: referenced ].</div><div>--------</div><div>If you save it and answer to declare as method temp referencesToName and referencedByName, and to define as block-temp referencesToVariableBinding and referencesToClassName, the debugger popups because #+ is sent to a BlockNode</div><div><div>Can you take a look at it? I did not have time to see it, but it is really weird because tempsMark is returning a BlockNode instead of an integer...<br></div><div>I verified that this is a new error added by the changes you made.</div><div><br></div><div>Thanks!</div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal"><span style="font-weight:bold">Hernán Wilkinson</span><br>Agile Software Development, Teaching & Coaching</span></font></span></span></span></strong></span></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">Phone: +54-011</span></font></span></span></span></strong></span><font size="2" face="tahoma, sans-serif">-4893-2057</font></div><div><strong style="font-family:tahoma,sans-serif;font-size:xx-small"><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Twitter: @HernanWilkinson</span></font></span></span></strong></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">site: <a href="http://www.10pines.com/" style="color:rgb(17,65,112)" rel="noreferrer" target="_blank">http://www.10Pines.com</a></span></font></span></span></span></strong></span></div><div><font face="tahoma, sans-serif"><span style="border-collapse:collapse">Address: Alem 896</span></font>, Floor 6, Buenos Aires, Argentina</div></div></div></div></div></div></div></div></div></div></div></div></div>
</blockquote></div></div></div>
</blockquote></div>