<div dir="ltr">Just did a quick look at senders of #on:do: and found these:<div><br><div>AndreasSystemProfiler>canWork</div><div>Browser>contents:notifying:</div><div>FeatureRequirement>require</div><div>FeatureRequirement>satisfyRequirementsAndInstall</div><div>MethodNode>generatePreSpur:using:</div><div>MethodNode>generateSpur:using:</div><div>RefactoringApplier>createAndValueHandlingExceptions:</div><div>SmalltalkEditor>renameTemporary:</div><div>SmalltalkEditor>withClassDefinitionNodeAndClassDo:ifErrorsParsing:</div><div>SmalltalkEditor>withMethodNodeAndClassDo:ifErrorsParsing:</div><div>SystemDictionary>okayToSave</div><div><br></div><div>That's ~15% of #on:do: senders in the base image *directly* returning within the block.  (i.e. I didn't check any of the blocks passed as parameters which might also do so or any of the other ways this might occur)  I don't think this change is fully baked...</div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 13, 2019 at 8:00 PM Phil B <<a href="mailto:pbpublist@gmail.com">pbpublist@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">It looks like we have at least some changes in the core image to resolve.  World menu->Save as... now fails.  Has a scan of the image been done to identify where these cases might be buried?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 13, 2019 at 9:48 AM Juan Vuletich via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st" target="_blank">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Folks,<br>
<br>
Some time ago, Andrés (with just a tad of help from me) fixed a problem <br>
in Exception handling. If exception handler blocks do non-local return <br>
(^stuff), they will skip execution of part of the Exception handling <br>
system code, breaking exception return values and possibly other 'bad <br>
things'.<br>
<br>
I just pushed to GitHub a few updates with this work. Now, if an <br>
exception handler does a non-local return, an Error will be raised. We <br>
also fixed a couple of places in the image where this was being done. <br>
BaseImageTests pass.<br>
<br>
This updates have some risk of breaking your code. If you you get this <br>
error: 'Exception handler blocks must not do non local returns', then <br>
you need to adjust your code. See updates #3917 to #3922 for the changes <br>
done to the base image.<br>
<br>
Thanks,<br>
<br>
-- <br>
Juan Vuletich<br>
<a href="http://www.cuis-smalltalk.org" rel="noreferrer" target="_blank">www.cuis-smalltalk.org</a><br>
<a href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" rel="noreferrer" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a><br>
<a href="https://github.com/jvuletich" rel="noreferrer" target="_blank">https://github.com/jvuletich</a><br>
<a href="https://www.linkedin.com/in/juan-vuletich-75611b3" rel="noreferrer" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a><br>
@JuanVuletich<br>
<br>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div>
</blockquote></div>