<div dir="ltr"><div><div><div>After my late changes, it is now possible to interact with a SyntaxError and manually correct source code in the MC loading/merging process.<br></div><div>Maybe the package won&#39;t be marked dirty as it should be, but it already is something...<br>
</div><div>I wanted to favour correction of code over workaround for keeping a statu quo.<br></div>So I did not propose yet any change of Preferences/settings.<br>If many methods are incorrect, the manual change will be too teddious, and won&#39;t be of much help.<br>
</div>IMO, the preferences change can come later by analyzing the kind of error and proposing some alternative actions thru the pop up menu, or other means (a line of buttons...).<br></div>Better could be an automatic correction (if it ALWAYS works, or at least knows when it works, there is nothing worse than automatic actions that sometimes work, sometimes don&#39;t...)<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/16 Nicolas Cellier <span dir="ltr">&lt;<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank">nicolas.cellier.aka.nice@gmail.com</a>&gt;</span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Ah, because SyntaxError&gt;&gt;contents:notifying: is trying to install the compiled method directly in target class...<br>
</div>That does not fit MC expectations at all...<br></div><div>MC expects the compiledMethod to just be compiled and stored in some MethodAddition, not to be installed that soon.<br>
</div><div><br></div><div>Then The SyntaxError #proceed in the SyntaxErrorNotification signalerContext...<br></div><div>There must be some way to handle that SyntaxErrorNotification differently...<br></div><div>What we want to do is to restart the compilation with a corrected source, that sounds possible.<span class="HOEnZb"><font color="#888888"><br>

<br></font></span></div><span class="HOEnZb"><font color="#888888"><div>Nicolas<br></div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/16 Nicolas Cellier <span dir="ltr">&lt;<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank">nicolas.cellier.aka.nice@gmail.com</a>&gt;</span><br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>I&#39;ve just tested what happens to a source with invalid syntax:<br></div>- from a File Browser fileIn action, correcting and accepting source in the SyntaxError pop up let the fileIn continue<br>


</div>- from a MC load action, it won&#39;t work because some MethodAddition compiledMethod isNil...<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/13 H. Hirzel <span dir="ltr">&lt;<a href="mailto:hannes.hirzel@gmail.com" target="_blank">hannes.hirzel@gmail.com</a>&gt;</span><div>

<div><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
Any ideas how to implement these suggestions/hints?<br>
<br>
for example a hint about the<br>
<br>
     underscore assignment?<br>
<span><font color="#888888"><br>
H.<br>
</font></span><div><div><br>
<br>
On 9/13/13, Bert Freudenberg &lt;<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>&gt; wrote:<br>
&gt; On 2013-09-13, at 10:20, Tobias Pape &lt;<a href="mailto:Das.Linux@gmx.de" target="_blank">Das.Linux@gmx.de</a>&gt; wrote:<br>
&gt;<br>
&gt;&gt; Hi<br>
&gt;&gt;<br>
&gt;&gt; Am 13.09.2013 um 10:11 schrieb Bert Freudenberg &lt;<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>&gt;:<br>
&gt;&gt;<br>
&gt;&gt;&gt; In particular if the syntax error notifier lets us easily fix an<br>
&gt;&gt;&gt; occurrence when trying to load older code - does it?<br>
&gt;&gt;<br>
&gt;&gt; Personally, I really hate it when those syntax error windows pop up.<br>
&gt;&gt; I have to abort the whole loading, fix the code by hand and try again,<br>
&gt;&gt; don&#39;t I?<br>
&gt;&gt;  Your comment seems to imply that I can change the respective code and<br>
&gt;&gt; save it?<br>
&gt;&gt; If so, we _really_ should make that syntax error stuff more accessible.<br>
&gt;&gt; And probably indicate what preferences to tune to make certain code load.<br>
&gt;&gt; I mean, Squeak hasn&#39;t enabled underscore selectors by default, which<br>
&gt;&gt; I tripped over several times, with said syntax error window. Would be<br>
&gt;&gt; gorgeous if the window _could_ indicate which preference to tune…<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; Best<br>
&gt;&gt;      -Tobias<br>
&gt;<br>
&gt; +1<br>
&gt;<br>
&gt; - Bert -<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
<br>
</div></div></blockquote></div></div></div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>