[squeak-dev] The Trunk: Tools-topa.563.mcz
Tobias Pape
Das.Linux at gmx.de
Fri Mar 27 06:27:47 UTC 2015
Hi
On 27.03.2015, at 02:33, Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com> wrote:
>
>
> 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 :)
Thanks for watching, nevertheless :)
Best
-Tobias
>
>
> pa = pb and: [
> ma selector <= mb selector and: [
> mb methodClass inheritsFrom: ma methodClass]]]]
> !
>
>
>
>
>
More information about the Squeak-dev
mailing list
|