<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p>Hi Levente,</p>
<p><br>
</p>
<p>I'm very much with you that we should get rid of #ifNotNilDo:, but unless we actually deprecate this selector, I fear that this is an unstable change which someone else might revert in any later commit out of ignorance. Do we want to deprecate #ifNotNilDo:
& Co. before the next major release? :-)</p>
<p><br>
</p>
<p>Best,</p>
<p>Christoph</p>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von commits@source.squeak.org <commits@source.squeak.org><br>
<b>Gesendet:</b> Freitag, 3. September 2021 16:22:05<br>
<b>An:</b> squeak-dev@lists.squeakfoundation.org; packages@lists.squeakfoundation.org<br>
<b>Betreff:</b> [squeak-dev] The Trunk: EToys-ul.448.mcz</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Levente Uzonyi uploaded a new version of EToys to project The Trunk:<br>
<a href="http://source.squeak.org/trunk/EToys-ul.448.mcz">http://source.squeak.org/trunk/EToys-ul.448.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: EToys-ul.448<br>
Author: ul<br>
Time: 3 September 2021, 3:50:47.825369 pm<br>
UUID: 32822614-0663-4add-b1f3-cbbf4c906a6d<br>
Ancestors: EToys-nice.447<br>
<br>
- ifNotNilDo: -> ifNotNil:<br>
<br>
=============== Diff against EToys-nice.447 ===============<br>
<br>
Item was changed:<br>
----- Method: ScriptEditorMorph>>dismiss (in category 'buttons') -----<br>
dismiss<br>
"Dismiss the scriptor, usually nondestructively. Possibly animate the dismissal."<br>
<br>
| endPoint aForm startPoint topRend |<br>
owner ifNil: [^ self].<br>
scriptName ifNil: [^ self delete]. "ad hoc fixup for bkwrd compat"<br>
<br>
+ endPoint := self viewerTile ifNotNil: [:tile | tile topLeft] ifNil: [owner topRight].<br>
- endPoint := self viewerTile ifNotNilDo: [:tile | tile topLeft] ifNil: [owner topRight].<br>
aForm := (topRend := self topRendererOrSelf) imageForm offset: (0@0).<br>
handWithTile := nil.<br>
startPoint := topRend topLeft.<br>
topRend topRendererOrSelf delete.<br>
(playerScripted isExpendableScript: scriptName) ifTrue: [<br>
^ playerScripted removeScript: scriptName fromWorld: Project current world].<br>
<br>
Project current world displayWorld.<br>
aForm slideFrom: startPoint to: endPoint nSteps: 4 delay: 30.<br>
"The OLPC Virtual Screen wouldn't notice the last update here."<br>
Display forceToScreen: (endPoint extent: aForm extent).!<br>
<br>
Item was changed:<br>
----- Method: TileMorph>>layoutChanged (in category 'layout') -----<br>
layoutChanged<br>
| vpanel hpanel popArrows |<br>
super layoutChanged.<br>
self labelMorph<br>
ifNil: [^ self].<br>
+ popArrows := self activeHand ifNil: [^ self] ifNotNil: [:ac |<br>
- popArrows := self activeHand ifNil: [^ self] ifNotNilDo: [:ac |<br>
ac valueOfProperty: #popArrows<br>
ifAbsent: [^ self]].<br>
popArrows first == self<br>
ifFalse: [^ self].<br>
vpanel := popArrows second.<br>
hpanel := popArrows third.<br>
vpanel<br>
ifNotNil: [vpanel openInWorld.<br>
vpanel center: self labelMorph center.<br>
vpanel right: self labelMorph left - 2].<br>
hpanel<br>
ifNotNil: [hpanel openInWorld.<br>
hpanel center: self labelMorph center.<br>
hpanel left: self labelMorph right + 2]!<br>
<br>
<br>
</div>
</span></font>
</body>
</html>