<div dir="ltr"><div dir="ltr"><div dir="ltr">The following are resumable, so the resume: in defaultAction just needs to be replaced by a return from the method:<div>EnvironmentRequest</div><div>FTPConnectionException¬†¬†<br></div><div>InMidstOfFileinNotification</div><div>MCProxyMaterialization</div><div>OutOfScopeNotification</div><div>ProgressInitiationException</div><div>ProgressTargetRequestNotification</div><div>ProjectEntryNotification</div><div>ProjectPasswordNotification</div><div>ProjectViewOpenNotification</div><div>RequestAlternateSyntaxSetting<br></div><div>WebAuthRequired</div><div><br></div><div>Some of their defaultActions could rather answer their defaultResumeValue, so it is not duplicated.</div><div><br></div><div>The only one remaining that needs more thinking about is NameLookupFailure.</div><div><br></div><div>The hint that one must/should not control the signal handling from defaultAction could be added to Exception>defaultAction and Notification>defaultAction.</div></div></div></div><br><div class="gmail_quote"><div dir="ltr">Am Do., 3. Jan. 2019 um 06:06¬†Uhr schrieb tim Rowledge <<a href="mailto:tim@rowledge.org">tim@rowledge.org</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Exception handling is fun!<br>
<br>
After some discussions at todays' SOB meeting, Eliot pointed out that we should be worrying about what the J20 standard says about exceptions. Interestingly, it explicitly calls out the sort of thing that NameLookupFailure>defaultAction does as wrong, with capital W and extra '!!'. So, even though I found a slightly tacky work-around that appears to make it function most of the time, what we actually need to do is fix the real problem, ie bad code in some implementations of #defaultAction. And, maybe, comment a few places to help future-us get it right later.<br>
<br>
Looking at implementations of #defaultAction in the image I see the following in need of fixing - <br>
<br>
EnvironmentRequest>>#defaultAction<br>
FTPConnectionException>>#defaultAction<br>
InMidstOfFileinNotification>>#defaultAction<br>
MCProxyMaterialization>>#defaultAction<br>
NameLookupFailure>>#defaultAction<br>
OutOfScopeNotification>>#defaultAction<br>
ProgressInitiationException>>#defaultAction<br>
ProgressTargetRequestNotification>>#defaultAction<br>
ProjectEntryNotification>>#defaultAction<br>
ProjectPasswordNotification>>#defaultAction<br>
ProjectViewOpenNotification>>#defaultAction<br>
RequestAlternateSyntaxSetting>>#defaultAction (note that this was related to an ancient syntax experiment and the involved preference is no longer implemented and should be finally excised - see #printAlternateSyntax)<br>
WebAuthRequired>>#defaultAction<br>
<br>
tim<br>
--<br>
tim Rowledge; <a href="mailto:tim@rowledge.org" target="_blank">tim@rowledge.org</a>; <a href="http://www.rowledge.org/tim" rel="noreferrer" target="_blank">http://www.rowledge.org/tim</a><br>
Strange OpCodes: START: Cancel preceding jobs in queue<br>
<br>
<br>
<br>
</blockquote></div>