<div dir="ltr">Hi all,<div><br></div><div>I'm attaching a small fix to <font face="monospace">ReturnNode>>isImplicitSelfReturnIn:</font> that I found while making some fixes to the <font face="monospace">IntroduceNullObject</font> refactoring I've been working on. The problem with the previous code:<br><br></div><div><font face="monospace">ReturnNode>>isImplicitSelfReturnIn: aMethodNode<br></font></div><font face="monospace"><br>  ^self isReturnSelf and: [ (aMethodNode encoder rawSourceRanges includesKey: expr) not ]</font><div><font face="monospace"><br></font></div><div><font face="arial, sans-serif">is that </font><font face="monospace">expr</font><font face="arial, sans-serif"> is </font><font face="monospace">self</font><font face="arial, sans-serif">, so it was returning </font><font face="monospace">false</font><font face="arial, sans-serif"> if </font><font face="monospace">self</font><font face="arial, sans-serif"> was referenced in other parts of the method body.</font></div><div><br></div><div>Best,</div><div>Fernando.</div></div>