<div dir="ltr"><div>Another fix: the pipes weren't being removed when removing the last temporary within a block. Thanks to Hernán for pointing that out to me. Plus, there was a bug when removing a temporary from inside a block while there was only one temporary declared in the method, and this ChangeSet fixes that too.</div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El mar., 28 may. 2019 a las 16:44, Hernan Wilkinson (<<a href="mailto:hernan.wilkinson@10pines.com">hernan.wilkinson@10pines.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">Cool! It works great!<div>On github now.</div><div><br></div><div>Thank you.</div><div>Hernan.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 28, 2019 at 12:21 AM Eric Brandwein <<a href="mailto:brandweineric@gmail.com" target="_blank">brandweineric@gmail.com</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>I noticed that the refactor doesn't work when moving a variable declared inside a block while another block declares it, and that the resulting error isn't very descriptive. I've added a better error message for this case, and the corresponding test.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El dom., 26 may. 2019 a las 16:00, 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>Here's a ChangeSet that fixes the things Hernán mentioned about showing references to the variable in other methods on the error message, and about removing the empty pipes when there are no temporary variables left on the method, along with the corresponding tests.<br></div><div><br></div><div>Cheers,</div><div>Eric<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El lun., 20 may. 2019 a las 17:13, Hernan Wilkinson (<<a href="mailto:hernan.wilkinson@10pines.com" target="_blank">hernan.wilkinson@10pines.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">👍👍<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 20, 2019 at 5:02 PM Eric Brandwein <<a href="mailto:brandweineric@gmail.com" target="_blank">brandweineric@gmail.com</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 dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El lun., 20 may. 2019 a las 8:10, Hernan Wilkinson (<<a href="mailto:hernan.wilkinson@10pines.com" target="_blank">hernan.wilkinson@10pines.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 dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, May 19, 2019 at 9:26 PM Eric Brandwein <<a href="mailto:brandweineric@gmail.com" target="_blank">brandweineric@gmail.com</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="auto"><div>Thanks Hernán! I'll check your formatting changes so I can apply them in future contributions. Is there a 'Smalltalk formatting guidelines' or something that I could read to learn them? Maybe in the Terse Guide, I haven't checked actually. </div></div></blockquote><div><br></div><div>There is... not only one convention but there are some basic guidelines. Formatting conventions are a matter of taste, so it is not easy for all to share the same conventions and it is always a matter of furious discussions :-) </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 dir="auto"><br></div><div dir="auto">About the comments:</div><div dir="auto"><br></div><div dir="auto">1) The pipes are currently left empty when the last temporary variable is removed because it is not used, so I followed that convention. I do think that they should be removed though, but then we should implement the same behaviour when removing unused variables.</div></div></blockquote><div><br></div><div>hmm generally if you don't declare temp var you don't put the || in the method... I think it is better to remove them if there is no more temp var</div></div></div></blockquote><div><br></div><div>No no, what I meant is that when saving, Cuis asks you if you want to remove unused variables, and when the last temporary variable is removed this way, the pipes are not removed. What I'm saying is that we should change this behaviour too.<br></div><div> </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 class="gmail_quote"><div> </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 dir="auto"><br></div><div dir="auto">2) & 3) I'll look into them!</div><div dir="auto"><br></div></div></blockquote><div>Cool!</div><div><br></div><div>Cheers!</div><div>Hernan. </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 dir="auto"></div><div dir="auto">Cheers,</div><div dir="auto">Eric<br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">El dom., 19 may. 2019 17:52, Hernan Wilkinson <<a href="mailto:hernan.wilkinson@10pines.com" target="_blank">hernan.wilkinson@10pines.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">Hi Eric,<div> as I said before, great job! Nice implementation and very well tested.</div><div> It is on github now.</div><div><br></div><div> I made a few changes:</div><div>1) Changed from cmd+P to cmd+O (as you said cmd+T is used) (Juan, we will have to talk about shortcuts I guess, we are running out of them :-) )</div><div>2) Made some formatting changes to make it more "Smalltalk oriented" :-)</div><div><br></div><div> I have a few improvements to suggest that it would be great if you could do them:</div><div>1) When the temporal to refactor is the only one, the source code keeps the temp. definition pipes, the | |. It would be great if the were removed from the source code</div><div>2) When there is another method that defines the same temporary, it would be great that the error message would say the method and class to easily find it. </div><div>What I do in this cases is to show the methods as elements in the menu so selecting them I can browse the method if necessary. Or you could show an option like "browse methods" or something like that to browse all those methods and therefore help to change those temporaries if necessary. Look for some examples in other refactorings and appliers... </div><div>3) The same thing when there are subclasses that define the inst. var.</div><div><br></div><div> Thank you again! this is a great contribution to Cuis. Keep doing more, we want more! :-)</div><div><br></div><div>Cheers!</div><div>Hernan.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 17, 2019 at 7:27 PM Eric Brandwein <<a href="mailto:brandweineric@gmail.com" rel="noreferrer" target="_blank">brandweineric@gmail.com</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="auto">I remember trying with Ctrl+shift+t and it being assigned to something else. I don't have the environment at hand right now, but I'll confirm later.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El vie., 17 may. 2019 14:45, Hernan Wilkinson <<a href="mailto:hernan.wilkinson@10pines.com" rel="noreferrer" target="_blank">hernan.wilkinson@10pines.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>Great! I'll take a look at it. It is really useful now that var shadowing is not allowed.</div><div>One thing, ctrl+ship+p is used in LiveTyping to show the types of the node under the cursor. What about using ctrl+shift+t ?<br></div><div><br></div><div>Hernan</div><div><br></div><div>PS:  Thank you for contributing to minimize the missing refactorings :-) </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 17, 2019 at 1:49 PM Eric Brandwein via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st" rel="noreferrer noreferrer" target="_blank">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>Hi all,</div><div><br></div><div>In my view, Cuis is missing a lot of refactorings. To start with something, here's a ChangeSet that adds the TemporaryToInstanceVariable refactor, along with a ChangeSet that adds tests. I've associated the <ctrl>+<shift>+P shortcut to it.</div><div><br></div><div>To try it, just make a method with a temporary variable, place the cursor above the temporary, and press the shortcut.<br></div><div><br></div><div>Cheers,</div><div>Eric<br></div></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" rel="noreferrer noreferrer" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer noreferrer 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-m_3502718119288726073gmail-m_442848877395430316gmail-m_34312724154568728gmail-m_4463200835000543184gmail-m_-392330816167277355gmail-m_-4249590416597753907gmail-m_9072290509769717454gmail-m_3918960312807572574m_-2120466513998770900gmail-m_5413784980187239220m_3714332335946149315gmail_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"><b><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></b></span></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><b><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></b></span><font size="2" face="tahoma, sans-serif">-4893-2057</font></div><div><b 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></b></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><b><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 noreferrer" target="_blank">http://www.10Pines.com</a></span></font></span></span></span></b></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>
</blockquote></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail-m_3502718119288726073gmail-m_442848877395430316gmail-m_34312724154568728gmail-m_4463200835000543184gmail-m_-392330816167277355gmail-m_-4249590416597753907gmail-m_9072290509769717454gmail-m_3918960312807572574m_-2120466513998770900gmail_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"><b><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></b></span></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><b><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></b></span><font size="2" face="tahoma, sans-serif">-4893-2057</font></div><div><b 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></b></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><b><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></b></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>
</blockquote></div></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail-m_3502718119288726073gmail-m_442848877395430316gmail-m_34312724154568728gmail-m_4463200835000543184gmail-m_-392330816167277355gmail-m_-4249590416597753907gmail-m_9072290509769717454gmail_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"><b><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></b></span></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><b><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></b></span><font size="2" face="tahoma, sans-serif">-4893-2057</font></div><div><b 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></b></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><b><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></b></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>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail-m_3502718119288726073gmail-m_442848877395430316gmail-m_34312724154568728gmail-m_4463200835000543184gmail-m_-392330816167277355gmail_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 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)" 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>
</blockquote></div>
</blockquote></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail-m_3502718119288726073gmail_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 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)" 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>
</blockquote></div>