<div dir="ltr"><div>Hi all,</div><div><br></div><div>Continuing to the fixes to refactors, here's one for two bugs in TemporaryToInstanceVariable.</div><div>First, the SmalltalkEditor wouldn't check if you were actually extracting a temporary and not an argument, because both of them are TempVariableNodes. This means you could try to make the Temporary to Instance Variable refactor on an argument, and the debugger would pop up.</div><div><br></div><div>Second, the TemporaryToInstanceVariable class didn't check if there was already other block declaring an argument with the same name as the temporary you were trying to extract, and so when trying to compile the method without the temporary declaration the Parser would throw an error. This one can be reproduced by writing this method:</div><div><br></div><div>m1<br></div><div>    [ :a | ^a ].</div><div>    [ | a | ^a ].</div><div><br></div><div>and calling the refactor on the declaration of a on the second block.</div><div><br></div><div>Cheers,</div><div>Eric<br></div></div>