<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</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>Ha! Can't assess which implementation is better, but the bugfix time was fantastic! :-)</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 Eliot Miranda <eliot.miranda@gmail.com><br>
<b>Gesendet:</b> Dienstag, 21. September 2021 00:42:25<br>
<b>An:</b> The general-purpose Squeak developers list<br>
<b>Betreff:</b> Re: [squeak-dev] MethodDictionary >> #select: not working</font>
<div> </div>
</div>
<div>
<div dir="ltr">
<div dir="ltr">
<div class="gmail_default" style="font-size:small">Hi Levente, Hi Christoph,<br>
</div>
</div>
<br>
<div class="gmail_quote">
<div dir="ltr" class="gmail_attr">On Mon, Sep 20, 2021 at 3:26 PM Levente Uzonyi <<a href="mailto:leves@caesar.elte.hu">leves@caesar.elte.hu</a>> wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Hi Christoph,<br>
<br>
On Mon, 20 Sep 2021, <a href="mailto:christoph.thiede@student.hpi.uni-potsdam.de" target="_blank">
christoph.thiede@student.hpi.uni-potsdam.de</a> wrote:<br>
<br>
> Hi there.<br>
> <br>
> I think I found a bug in MethodDictionary. Try this:<br>
> <br>
> UIManager methodDict select: [:ea | ea selector beginsWith: 'chooseFrom']<br>
> <br>
> Expected: A dictionary (I don't care which specialization) that I can enumerate to retrieve ca. 10 selectors.<br>
> Actual result: Answer looks empty in the inspector and in the print string; however, the array is not empty. The invariant mentioned in the class comment ((self basicAt: index) isNil = (array at: index) isNil) is violated.<br>
> <br>
> Maybe any of our Collections experts feels like looking into this - for now, I'm going to convert the dictionary before filtering it. :-)<br>
<br>
The implementation of #add: inherited from Dictionary is no sufficient for <br>
MethodDictionary.<br>
See Kernel-ul.1415 in the Inbox with the proposed fix for your case and <br>
some other methods.<br>
</blockquote>
<div><br>
</div>
<div class="gmail_default" style="font-size:small">Also see Kernel-eem.1415. I didn't see your fix Levente before I had a go. I'm happy which ever gets chosen.</div>
<div class="gmail_default" style="font-size:small"></div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
Levente<br>
</blockquote>
</div>
<div><br>
</div>
<div dir="ltr" class="gmail_signature">
<div dir="ltr">
<div><span style="font-size:small;border-collapse:separate">
<div>_,,,^..^,,,_<br>
</div>
<div>best, Eliot</div>
</span></div>
</div>
</div>
</div>
</div>
</body>
</html>