<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">
                                        Hi David.<div><br></div><div>Thanks for working on this! :-)</div><div><br></div><div>That special case in "Delay" reminds me of the suggestion to just put "DoItFirst" at a more sensible place in the start-up list. But then it would not be "do it first" anymore. :-D Same for FileDirectory.</div><div><br></div><div>What about a little bit more "double dispatch"?</div><div><br></div><div><span style="font-size: 13.3333px">(Smalltalk classNamed: #DoItFirst) ifNotNil: [ :doit | doit startUpFromDelay ].</span><br></div><div>(Smalltalk classNamed: #DoItFirst) ifNotNil: [ :doit | doit startUpFromFileDirectory ].<span style="font-size: 13.3333px"><br></span></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;min-width: 500px">
                        <p style="color: #AAAAAA; margin-top: 10px;">Am 09.12.2020 06:39:15 schrieb Eliot Miranda <eliot.miranda@gmail.com>:</p><div style="font-family:Arial,Helvetica,sans-serif">Hi David,<br><br>> On Dec 8, 2020, at 1:13 PM, commits@source.squeak.org wrote:<br>> <br>> David T. Lewis uploaded a new version of Kernel to project The Trunk:<br>> http://source.squeak.org/trunk/Kernel-dtl.1363.mcz<br>> <br>> ==================== Summary ====================<br>> <br>> Name: Kernel-dtl.1363<br>> Author: dtl<br>> Time: 8 December 2020, 4:13:03.533263 pm<br>> UUID: dc65867e-37e6-4295-aa44-ef26c62ba250<br>> Ancestors: Kernel-eem.1362<br>> <br>> Let Delay class>>startup invoke DoItFirst class>>reevaluateDebug. If a DoItFirst command line option has requested a debugger, then invoke it now after Delay startUp processing..<br>> <br>> =============== Diff against Kernel-eem.1362 ===============<br>> <br>> Item was changed:<br>>  ----- Method: Delay class>>startUp (in category 'snapshotting') -----<br>>  startUp<br>>      "Restart active delay, if any, when resuming a snapshot."<br>> <br>>      DelaySuspended ifFalse:[^self error: 'Trying to activate Delay twice'].<br>>      DelaySuspended := false.<br>>      self restoreResumptionTimes.<br>>      AccessProtect signal.<br>> +    (Smalltalk classNamed: #DoItFirst) ifNotNil: [ :doit | doit perform: #reevaluateDebug].<br>> + <br>> + <br>>  !<br><br>I don’t understand why this is necessary/what this code does.  A comment in the startUp method would go a long way.  Also do we really need two lines of white space tacked onto the end of the method?<br><br><br><br></div></blockquote></div>