Hi guys. Today, there is a part of the system coupled with Compiler. Issue is:�<a href="http://code.google.com/p/pharo/issues/detail?id=6110" target="_blank">http://code.google.com/p/pharo/issues/detail?id=6110</a><div><br>
</div><div>Now, if you see the method:</div>
<div><br></div><div><div>getSourceReplacingSelectorWith: newSelector</div><div><span style="white-space:pre-wrap">	</span>| oldKeywords newKeywords args newSelectorWithArgs source oldSelector s |</div><div>
<span style="white-space:pre-wrap">	</span>source := self sourceCode.</div><div><span style="background-color:rgb(255,255,102)"><span style="white-space:pre-wrap">	</span>oldSelector := self parserClass new parseSelector: source.</span></div>

<div><span style="background-color:rgb(255,255,102)"><span style="white-space:pre-wrap">	</span>oldSelector = newSelector ifTrue: [ ^ source ].</span></div><div><span style="white-space:pre-wrap">	</span>oldKeywords := oldSelector keywords.</div>

<div><span style="white-space:pre-wrap">	</span>newKeywords := (newSelector ifNil: [self defaultSelector]) keywords.</div><div><span style="white-space:pre-wrap">	</span>[oldKeywords size = newKeywords size] assert.</div>

<div><span style="white-space:pre-wrap">	</span>args := (self methodClass parserClass new</div><div><span style="white-space:pre-wrap">		</span>parseArgsAndTemps: source string notifying: nil) copyFrom: 1 to: self numArgs.</div>

<div><span style="white-space:pre-wrap">	</span>newSelectorWithArgs := String streamContents: [:stream |</div><div><span style="white-space:pre-wrap">		</span>newKeywords withIndexDo: [:keyword :index |</div>
<div><span style="white-space:pre-wrap">			</span>stream nextPutAll: keyword.</div><div><span style="white-space:pre-wrap">			</span>stream space.</div><div><span style="white-space:pre-wrap">			</span>args size &gt;= index ifTrue: [</div>

<div><span style="white-space:pre-wrap">				</span>stream nextPutAll: (args at: index); space]]].</div><div><span style="white-space:pre-wrap">	</span>s := source string readStream.</div>
<div><span style="white-space:pre-wrap">	</span>oldKeywords do: [ :each | s match: each ].</div><div><span style="white-space:pre-wrap">	</span>args isEmpty ifFalse: [ s match: args last ].</div>
<div><span style="white-space:pre-wrap">	</span>^newSelectorWithArgs trimBoth, s upToEnd</div></div><div><br></div><div>The 2 magic lines are:</div><div><br></div><div><div><span style="background-color:rgb(255,255,102)">oldSelector := self parserClass new parseSelector: source.</span></div>

<div><span style="background-color:rgb(255,255,102)"><span style="white-space:pre-wrap">	</span>oldSelector = newSelector ifTrue: [ ^ source ].</span></div></div><div><div><br></div><div>So, why we cannot just�replace &quot;self parserClass new parseSelector: source.&quot; with &quot;self selector&quot;. Well, I think (talking with Guille) this ONLY because of Trait transformation/aliases that associates a selector with a method of a different selector.</div>

<div><br></div><div>If this is the case, it means that 99% of the cases,&quot;oldSelector = newSelector ifTrue: [ ^ source ]. �&quot; �is true. Because those aliases are only used in one trait in all the image ().�</div>

<div><br></div><div>So...if that is the case, can&#39;t we get the oldSelector from elsewhere �rather than needing to use the Compiler to get the selector from the source code ?</div><div><br></div><div>thanks</div><div>
<br></div><div><br></div>-- <br>Mariano<br><a href="http://marianopeck.wordpress.com" target="_blank">http://marianopeck.wordpress.com</a><br><br>
</div>