[squeak-dev] The Inbox: Tools-ct.986.mcz
Chris Muller
asqueaker at gmail.com
Wed Sep 2 21:59:46 UTC 2020
Hi Christoph,
For your code contributions in general, please allow methods to have
only a single exit as much as possible, as in the attached.
I like the multilingual change, but the purpose of using
#headingAndAutoselectForLiteral:do: here wasn't obvious to me.
Best,
Chris
On Wed, Sep 2, 2020 at 9:56 AM <commits at source.squeak.org> wrote:
>
> Christoph Thiede uploaded a new version of Tools to project The Inbox:
> http://source.squeak.org/inbox/Tools-ct.986.mcz
>
> ==================== Summary ====================
>
> Name: Tools-ct.986
> Author: ct
> Time: 2 September 2020, 4:55:38.538083 pm
> UUID: b4cdf611-f04b-0b40-8f61-34429f414cca
> Ancestors: Tools-ct.985
>
> Fixes MNU when adding senders of a non-string literal to a message trace (at the end, FindText was set to a number or something similar). Improves multilingual support.
>
> =============== Diff against Tools-ct.985 ===============
>
> Item was changed:
> ----- Method: MessageTrace>>addParentMethodsSending: (in category 'building') -----
> addParentMethodsSending: selectorSymbol
>
> + ^ self systemNavigation
> + headingAndAutoselectForLiteral: selectorSymbol
> + do: [:label :autoSelect |
> + | methodsList |
> + methodsList := self systemNavigation allCallsOn: selectorSymbol.
> + methodsList ifEmpty: [
> + ^ self inform: ('There are no {1}' translated format: {label})].
> + self
> - | methodsList |
> - (methodsList := self systemNavigation allCallsOn: selectorSymbol) isEmpty
> - ifTrue:
> - [ ^(PopUpMenu labels: ' OK ')
> - startUpWithCaption: 'There are no methods that send ', selectorSymbol ]
> - ifFalse:
> - [ self
> addParentMessages: methodsList
> + autoSelectString: autoSelect]
> - autoSelectString: selectorSymbol ]
> !
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: MessageTrace-addParentMethodsSending.st
Type: application/octet-stream
Size: 601 bytes
Desc: not available
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20200902/9605162d/attachment-0001.obj>
More information about the Squeak-dev
mailing list
|