<div dir="ltr">Hi Eric,<div> thank you for the fix! It is in github now.</div><div><br></div><div> Re-parsing is not a problem because methods are usually small in Smalltalk and the parser is fast enough, but it introduces an issue: when you say you don't want to remove the variable and then you remove another one, it will ask you again for the first one.</div><div> I integrated the change anyway because it is a rare condition and it is better to remove unused temps on blocks.</div><div> If you can fix that new issue, the better, but I think it will not be so simple...</div><div><br></div><div> BTW, I also fixed another problem, it there was no temps in the method but unused temps in a block, it did not work.</div><div><br></div><div>Thanks!</div><div>Hernan.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 12, 2019 at 3:03 AM Eric Brandwein via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</a>> wrote:<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"><div>Sorry, I just realised this doesn't work if you're erasing many variables at a time. The obvious solution is to just reparse each time a temporary variable is removed, and that is what this new attached ChangeSet does. <br></div><div><br></div><div>This is maybe slow, and I guess the alternative would be to go through all the declarations of all blocks inside the parsed method from the bottom to the top, so as to not change the positions of other possibly unused declarations when removing one of them. Let me know if this other solution seems better or if the one that reparses every time is fine.</div><div><br></div><div>Cheers,</div><div>Eric  </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El jue., 12 dic. 2019 a las 2:23, Eric Brandwein (<<a href="mailto:brandweineric@gmail.com" target="_blank">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="ltr"><div>Hi all,</div><div><br></div><div>Currently, when you save a method with an unused temporary variable declared, it prompts you to decide if you want to remove it or not. If you click yes, it removes the declaration, but only if it's declared in the method scope, and not if it is declared inside a block.<br></div><div><br></div><div>To reproduce, just create a method with an unused temporary variable declared inside a block, save it, and click 'Yes' on the popup that appears.<br></div><div><br></div><div>This is because the Parser only looks for the variable in the temps declared in the method scope. Here's a ChangeSet that fixes it making the Parser look inside blocks too, taking advantage of the fact that the Encoder has already found the positions for each variable.</div><div><br></div><div>Cheers,</div><div>Eric<br></div></div>
</blockquote></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><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 face="tahoma, sans-serif" size="2">-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)" 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>