<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

</head>

<body>

<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>

<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">

<p>Hi Marcel, thanks for the review!</p>

<div id="Signature">

<div id="divtagdefaultwrapper" dir="ltr" style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;">

<div name="divtagdefaultwrapper" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:; margin:0">

<div>

<div class="_rp_T4" id="Item.MessagePartBody">

<div class="_rp_U4 ms-font-weight-regular ms-font-color-neutralDark rpHighlightAllClass rpHighlightBodyClass" id="Item.MessageUniqueBody" style="font-family:wf_segoe-ui_normal,"Segoe UI","Segoe WP",Tahoma,Arial,sans-serif,serif,EmojiFont">

<div dir="ltr">

<div id="divtagdefaultwrapper"><font face="Calibri,Helvetica,sans-serif,EmojiFont,Apple Color Emoji,Segoe UI Emoji,NotoColorEmoji,Segoe UI Symbol,Android Emoji,EmojiSymbols">

<div id="Signature">

<div style="margin:0px"><font style="font-family:Calibri,Arial,Helvetica,sans-serif,serif,EmojiFont">

<div><font size="3" color="black"><span style="font-size:12pt"><a href="http://www.hpi.de/" target="_blank" rel="noopener noreferrer" id="LPNoLP"><font size="2"><span id="LPlnk909538"><font color="#757B80"></font></span></font></a></span></font></div>

</font></div>

</div>

</font></div>

</div>

</div>

</div>

<div class="_rp_T4" id="Item.MessagePartBody"><br>

</div>

<div class="_rp_T4" id="Item.MessagePartBody">> <span>Please note that you changed the overall style of the method from single to multiple returns.</span></div>

<div class="_rp_T4" id="Item.MessagePartBody"><span><br>

</span></div>

<div class="_rp_T4" id="Item.MessagePartBody"><span>Do you mean this impedes the readability? It was an attempt to apply the Guard Clause.</span></div>

<div class="_rp_T4" id="Item.MessagePartBody"><span><br>

</span></div>

<div class="_rp_T4" id="Item.MessagePartBody"><span>> </span><span style="font-size: 12pt;">Btw: Do you know about Object >> #perform:orSendTo:? :-)</span><span>

<div><br>

</div>

<div>Where would you use that in this method? :-)</div>

<div><br>

</div>

<div>Best,</div>

<div>Christoph</div>

</span></div>

</div>

<div><font size="2" color="#808080"></font></div>

</div>

</div>

</div>

</div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel<br>

<b>Gesendet:</b> Montag, 2. März 2020 10:39:42<br>

<b>An:</b> John Pfersich via Squeak-dev<br>

<b>Betreff:</b> Re: [squeak-dev] The Inbox: Morphic-ct.1632.mcz</font>

<div> </div>

</div>

<div>

<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000">

Hi Christoph.

<div><br>

</div>

<div>Please note that you changed the overall style of the method from single to multiple returns. You should mention that in your comment message.</div>

<div><br>

</div>

<div>Btw: Do you know about Object >> #perform:orSendTo:? :-)</div>

<div><br>

</div>

<div>Best,</div>

<div>Marcel</div>

<div class="mb_sig"></div>

<blockquote class="history_container" type="cite" style="border-left-style:solid;border-width:1px; margin-top:20px; margin-left:0px;padding-left:10px;">

<p style="color: #AAAAAA; margin-top: 10px;">Am 29.02.2020 11:29:45 schrieb commits@source.squeak.org <commits@source.squeak.org>:</p>

<div style="font-family:Arial,Helvetica,sans-serif">Christoph Thiede uploaded a new version of Morphic to project The Inbox:<br>

http://source.squeak.org/inbox/Morphic-ct.1632.mcz<br>

<br>

==================== Summary ====================<br>

<br>

Name: Morphic-ct.1632<br>

Author: ct<br>

Time: 29 February 2020, 11:29:22.475896 am<br>

UUID: 2d941b38-6604-cb43-bfbb-b7751aa2d55c<br>

Ancestors: Morphic-mt.1631<br>

<br>

Proposal: In Morph>>#balloonText, also try to invoke balloonSelector on model.<br>

<br>

=============== Diff against Morphic-mt.1631 ===============<br>

<br>

Item was changed:<br>

----- Method: Morph>>balloonText (in category 'accessing') -----<br>

balloonText<br>

+ "Answer balloon help text or nil, if no help is available."<br>

+ "NB: subclasses may override such that they programatically construct the text, for economy's sake, such as model phrases in a Viewer."<br>

- "Answer balloon help text or nil, if no help is available. <br>

- NB: subclasses may override such that they programatically <br>

- construct the text, for economy's sake, such as model phrases in <br>

- a Viewer"<br>

<br>

+ | balloonSelector |<br>

+ extension ifNil: [^ nil].<br>

- | result |<br>

- extension ifNil: [^nil].<br>

<br>

extension balloonText<br>

+ ifNotNil: [:balloonText | ^ balloonText].<br>

+ balloonSelector := extension balloonTextSelector<br>

+ ifNil: [^ nil].<br>

+ (ScriptingSystem helpStringOrNilFor: balloonSelector)<br>

+ ifNotNil: [:result | ^ result].<br>

+ balloonSelector == #methodComment<br>

+ ifTrue: [^ self methodCommentAsBalloonHelp].<br>

+ balloonSelector isUnary<br>

+ ifTrue: [<br>

+ (self respondsTo: balloonSelector)<br>

+ ifTrue: [^ self perform: balloonSelector].<br>

+ (self model respondsTo: balloonSelector)<br>

+ ifTrue: [^ self model perform: balloonSelector]].<br>

+ ^ nil!<br>

- ifNotNil: [:balloonText | result := balloonText]<br>

- ifNil: [extension balloonTextSelector<br>

- ifNotNil: [:balloonSelector |<br>

- result := ScriptingSystem helpStringOrNilFor: balloonSelector.<br>

- (result isNil and: [balloonSelector == #methodComment]) <br>

- ifTrue: [result := self methodCommentAsBalloonHelp].<br>

- ((result isNil and: [balloonSelector numArgs = 0]) <br>

- and: [self respondsTo: balloonSelector]) <br>

- ifTrue: [result := self perform: balloonSelector]]].<br>

- ^ result!<br>

<br>

<br>

</div>

</blockquote>

</div>

</div>

</body>

</html>