[squeak-dev] The Trunk: Tools-topa.563.mcz
Nicolas Cellier
nicolas.cellier.aka.nice at gmail.com
Fri Mar 27 01:33:52 UTC 2015
2015-03-27 2:31 GMT+01:00 Nicolas Cellier <
nicolas.cellier.aka.nice at gmail.com>:
>
>
> 2015-03-27 0:27 GMT+01:00 <commits at source.squeak.org>:
>
>> Tobias Pape uploaded a new version of Tools to project The Trunk:
>> http://source.squeak.org/trunk/Tools-topa.563.mcz
>>
>> ==================== Summary ====================
>>
>> Name: Tools-topa.563
>> Author: topa
>> Time: 27 March 2015, 12:27:31.114 am
>> UUID: 7fb2f85e-8ba3-4ec4-be83-3c9275f7f80b
>> Ancestors: Tools-topa.562
>>
>> Fix menu entry sort to actually match (intended) description
>>
>> =============== Diff against Tools-topa.562 ===============
>>
>> Item was changed:
>> ----- Method: StringHolder>>menuBuildersFor:in: (in category
>> '*Tools-pluggable menus') -----
>> menuBuildersFor: someMenus in: aClass
>> "Find all builders and sort them by
>> 1. Priority (default 500)
>> 2. selector name
>> 3. inheritance
>> "
>> ^ (self menuPragmasFor: someMenus in: aClass) sorted: [:a :b |
>> |ma mb pa pb|
>> ma := a method.
>> mb := b method.
>> pa := self methodMenuPriority: ma.
>> pb := self methodMenuPriority: mb.
>> + pa <= pb or: [
>> - pa < pb or: [
>>
>
> I don't understand this.
> pa <= pb = false ==> (pa = pb = false)
> so the whole or: block becomes dead code no?
>
Oh, but it already changed, never mind :)
>
>
>> pa = pb and: [
>> ma selector <= mb selector and: [
>> mb methodClass inheritsFrom: ma
>> methodClass]]]]
>> !
>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20150327/147e7bfe/attachment-0001.htm
More information about the Squeak-dev
mailing list
|