<br><br><div class="gmail_quote">On Tue, May 21, 2013 at 2:55 PM, Frank Shearar <span dir="ltr"><<a href="mailto:frank.shearar@gmail.com" target="_blank">frank.shearar@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 21 May 2013 22:54, Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>> wrote:<br>
><br>
><br>
> On Tue, May 21, 2013 at 2:53 PM, Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>><br>
> wrote:<br>
>><br>
>><br>
>><br>
>> On Tue, May 21, 2013 at 2:40 PM, Frank Shearar <<a href="mailto:frank.shearar@gmail.com">frank.shearar@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> allMethodsSelect: aBlock localTo: aClass<br>
>>> "Answer a SortedCollection of each methodr in, above, or below the<br>
>>> given<br>
>>> class that, when used as the argument to aBlock, gives a true<br>
>>> result."<br>
>>><br>
>>> | aSet |<br>
>>> aSet := Set new.<br>
>>> Cursor wait showWhile:<br>
>>> [aClass theNonMetaClass withAllSuperAndSubclassesDoGently:<br>
>>> [:class |<br>
>>> class selectorsAndMethodsDo:<br>
>>> [:aSelector :aMethod|<br>
>>> (aBlock value: aMethod) ifTrue:<br>
>>> [aSet add: (MethodReference new<br>
>>> setStandardClass: class methodSymbol: aSelector)]]].<br>
>>> aClass theNonMetaClass class withAllSuperAndSubclassesDoGently:<br>
>>> [:class |<br>
>>> class selectorsAndMethodsDo:<br>
>>> [:aSelector :aMethod|<br>
>>> (aBlock value: aMethod) ifTrue:<br>
>>> [aSet add: (MethodReference new<br>
>>> setStandardClass: class methodSymbol: aSelector)]]]].<br>
>>> ^aSet<br>
>>><br>
>>> Um. That is just an accidental duplication of the aClass<br>
>>> theNonMetaClass class withAllSuperAndSubclassesDoGently: block, right?<br>
>><br>
>><br>
>> No, it's not :) It's preserving existing behavior, which is to search<br>
>> class and instance side. Note that<br>
>><br>
>> aClass theNonMetaClass withAllSuperAndSubclassesDoGently:<br>
>> [:class|<br>
>> {class. class class} do: aBlock<br>
>><br>
>> is not equivalent to<br>
>><br>
>> aClass theNonMetaClass withAllSuperAndSubclassesDoGently: aBlock<br>
>> aClass theNonMetaClass class withAllSuperAndSubclassesDoGently: aBlock<br>
>><br>
>> because the latter searches Behavior, ClassDescription, Class etc, while<br>
>> the former doesn't.<br>
><br>
><br>
> and just to be clear, I'm not claiming one way is better or not.<br></div></div></blockquote><div><br></div><div>and writing it putting the block in a variable (as you did with another method last week, the future: one IIRC) makes the intention far clearer.</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
<br>
</div></div>Ah! I had missed that single token!<br>
<span class="HOEnZb"><font color="#888888"><br>
frank<br>
<br>
> --<br>
> best,<br>
> Eliot<br>
><br>
><br>
><br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>best,<div>Eliot</div>