<div dir="auto"><div class="gmail_extra" dir="auto"><div class="gmail_quote">Am 03.06.2017 7:18 nachm. schrieb "Eliot Miranda" <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>>:<blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text">
> On Jun 3, 2017, at 2:32 AM, Jakob Reschke <<a href="mailto:jakob.reschke@student.hpi.de">jakob.reschke@student.hpi.de</a>> wrote:<br>
><br>
> 2017-06-01 22:22 GMT+02:00 <a href="mailto:commits@source.squeak.org">commits@source.squeak.org</a><br>
> <<a href="mailto:commits@source.squeak.org">commits@source.squeak.org</a>>:<br>
>> Eliot Miranda uploaded a new version of Morphic to project The Trunk:<br>
>> <a href="http://source.squeak.org/trunk/Morphic-eem.1340.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/<wbr>trunk/Morphic-eem.1340.mcz</a><br>
>><br>
>> =============== Diff against Morphic-eem.1339 ===============<br>
>><br>
>> +                       (anEntry isBehavior and: [anEntry name == aSymbol]) ifFalse: "When is this ever false?"<br>
>> +                               [anEntry := anEntry class].<br>
>> -                       (anEntry isKindOf: Class)<br>
>> -                               ifFalse:        [anEntry := anEntry class].<br></div><div class="quoted-text">
><br>
> The condition is false for globals such as Smalltalk and SystemOrganization.<br>
<br>
</div>Right.  I meant when is anEntry name == aSymbol false. </blockquote></div></div><div dir="auto"><br></div><div dir="auto">The only thing that comes to my mind is a renaming import from another environment, an Alias value. But it does not make sense to browse the class of it just because of the name, or does it? Could the name check be left out?</div><div dir="auto"><br></div><div class="gmail_extra" dir="auto"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Smalltalk and SystemOrganization are not classes so no issue.  So the issue is traits.  Do you  want to update the comment here?<br>
<br>
</blockquote></div><br></div><div class="gmail_extra" dir="auto">I would have to go via the Inbox, so it would be faster if a core committer did it. And thanks to your change of using isBehavior, traits are no longer an issue for this piece of code.</div></div>