<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p>Thanks for the review, Vanessa!</p>
<p><br>
</p>
<p>Yeah, this example is very likely not a proper application of extension categories. I think this might be the cause of some package reorganizations of Etoys/Squeakland in the past.</p>
<p><br>
</p>
<p>On the other hand, I have some projects where I plan to split up two different system categories (something like Core + UI) in different categories later, but until then, I already use extension methods as a preparation for the later package split up. So
 it's a temporary solution only.</p>
<p><br>
</p>
<p>Best,</p>
<p>Christoph</p>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von Vanessa Freudenberg <vanessa@codefrau.net><br>
<b>Gesendet:</b> Freitag, 25. März 2022 18:41:28<br>
<b>An:</b> The general-purpose Squeak developers list<br>
<b>Betreff:</b> Re: [squeak-dev] Review Request: packageSelfExtensions.1.cs</font>
<div> </div>
</div>
<div>
<div dir="ltr">
<div dir="ltr">On Fri, Mar 25, 2022 at 8:06 AM <<a href="mailto:christoph.thiede@student.hpi.uni-potsdam.de">christoph.thiede@student.hpi.uni-potsdam.de</a>> wrote:<br>
</div>
<div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<b>=============== Summary ===============</b><br>
<br>
Change Set:        packageSelfExtensions<br>
Date:            25 March 2022<br>
Author:            Christoph Thiede<br>
<br>
Revises the notion of extension method so that also extensions to the same package are found. For instance, browsing all extensions of the package 'Etoys' local to the category 'Etoys-Squeakland-MorphicExtras-Charts' will now also bring up the extensions on
 Player.<br>
<br>
Fixes #extensionCategoriesForClass:localTo: to also find different casings of the package name in the extension category. Also revises #methodCategoryPrefix to no longer destroy the original casing of the package name.<br>
<br>
Note that with this change, #extensionMethods and #coreMethods are no longer necessarily disjunct.<br>
</blockquote>
<div><br>
</div>
<div>If a method is in an extension category of its own package, that's a categorization bug. Rather than muddling the definition of extension methods vs core methods I'd rather we fix the categorization of miscategorized methods. </div>
<div><br>
</div>
<div>In the particular case of your example the method category should probably be simply "charts", or possibly "morphicextras-charts".</div>
<div><br>
</div>
<div>If your goal is to browse all methods in a certain category (or category prefix) across the whole image then that should probably be a method in SystemNavigation, not PackageInfo. PackageInfo
<i>defines</i> a package (its name is somewhat misleading, could also be PackageSpec).</div>
<div><br>
</div>
<div>Your fixes look fine though.</div>
<div><br>
</div>
<div>Vanessa</div>
</div>
</div>
</div>
</body>
</html>