grrrr and the same happens for methods that are removed....they are not removed from the RPackage :(<br><br><div class="gmail_quote">On Sat, Aug 4, 2012 at 2:41 PM, Mariano Martinez Peck <span dir="ltr">&lt;<a href="mailto:marianopeck@gmail.com" target="_blank">marianopeck@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br><div class="gmail_quote"><div><div class="h5">On Sat, Aug 4, 2012 at 2:34 PM, Mariano Martinez Peck <span dir="ltr">&lt;<a href="mailto:marianopeck@gmail.com" target="_blank">marianopeck@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Stef. I notice that RPackage does not update the extension methods of removed classes. It is easy to reproduce in Pharo 2.0 by doing:<div><br></div><div>�(RPackageOrganizer �default packageNamed: &#39;FreeType&#39;) allDefinedExtensionMethods � (this fails because FreeType used to have a extension method in TTCFont which was removed)</div>


<div><br></div><div>Or step by step:</div><div><br></div><div>1) Create package PackageX.</div><div>2) Create package PackageY and ClassY.</div><div>3) Create in ClassY a extension method with category *PackageX. If you now browse the instVar &quot;classExtensionSelectors&quot; of PackageX you will see ClassY.</div>


<div>4) Remove ClassY</div><div>5) notice that &#39;classExtensionSelectors&#39; from PackageX still references ClassY. In fact, if you ask its extension methods you will have a KeyNotFound #ClassY in Smalltalk globals ;)</div>


<div><br></div><div>So...I guess this is a bug. �If true, I open an issue.</div><div><br></div></blockquote><div><br></div></div></div><div>and to fix the already existing ones we can do:</div><div><br></div><div><div>| extensions |</div>

<div>RPackage allInstances do: [:aPackage |</div><div><span style="white-space:pre-wrap">	</span>extensions := (aPackage �instVarNamed: &#39;classExtensionSelectors&#39;).</div><div><span style="white-space:pre-wrap">	</span>extensions keysDo: [:aClassName |</div>

<div><span style="white-space:pre-wrap">		</span>(Smalltalk globals includesKey: aClassName) ifFalse: �[extensions removeKey: aClassName].</div><div><span style="white-space:pre-wrap">		</span> ].�</div>
<div><span style="white-space:pre-wrap">	</span>extensions := (aPackage �instVarNamed: &#39;metaclassExtensionSelectors&#39;).</div><div><span style="white-space:pre-wrap">	</span>extensions keysDo: [:aClassName |</div>
<div><span style="white-space:pre-wrap">		</span>(Smalltalk globals includesKey: aClassName) ifFalse: �[extensions removeKey: aClassName].</div><div><span style="white-space:pre-wrap">		</span> ].�</div>
<div><span style="white-space:pre-wrap">	</span>]</div></div><div class="im"><div><br></div><div>�</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div></div><div>

Cheers,<span><font color="#888888"><br clear="all"><div><br></div>-- <br>Mariano<br><a href="http://marianopeck.wordpress.com" target="_blank">http://marianopeck.wordpress.com</a><br>
<br>
</font></span></div>
</blockquote></div></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br>Mariano<br><a href="http://marianopeck.wordpress.com" target="_blank">http://marianopeck.wordpress.com</a><br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>Mariano<br><a href="http://marianopeck.wordpress.com" target="_blank">http://marianopeck.wordpress.com</a><br><br>