<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 22 Sep 2016, at 20:28, Eliot Miranda &lt;<a href="mailto:eliot.miranda@gmail.com" class="">eliot.miranda@gmail.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">Hi Bert, Hi All,<div class="gmail_extra"><br class=""><div class="gmail_quote">On Thu, Sep 15, 2016 at 2:55 PM,  <span dir="ltr" class="">&lt;<a href="mailto:commits@source.squeak.org" target="_blank" class="">commits@source.squeak.org</a>&gt;</span> wrote:</div><div class="gmail_quote">[snip]<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><a href="http://lists.squeakfoundation.org/pipermail/packages/2016-September/068930.html" rel="noreferrer" target="_blank" class="">http://lists.squeakfoundation.<wbr class="">org/pipermail/packages/2016-<wbr class="">September/068930.html</a><br class="">
<br class="">
Name: System-bf.916<br class="">
Ancestors: System-bf.915<br class="">
<br class="">
Replace VM-level ImageSegment loading with a Smalltalk implementation for old (interpreter-era) projects.<br class="">
<br class="">
Also removes support for writing segments.<br class="">
<br class="">
This overrides the Spur support introduced in System-eem.758.<br class=""></blockquote><div class=""><br class=""></div><div class="">&nbsp;So one question is should we delete VM support for ImageSegment from the Spur VM?&nbsp; There's at least 1.5k of generated source for the Spur ImageSegment load and save support, some 2% of the interpreter/primitives source code.&nbsp; That's a lot, and the code is complex and ugly.&nbsp; If it never really worked before IMO we should nuke it asap.&nbsp; If it worked in some fashion perhaps we can schedule its demise for the 6.0 release's VM.</div><div class=""><br class=""></div><div class="">What do others think?</div></div></div></div></div></blockquote><div><br class=""></div><div>As long as you don’t remove it from the Cog VM’s until I no longer need it I’m fine with that.</div><div><br class=""></div><div>Max</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""><br class=""></div></div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr" class=""><div class=""><span style="font-size:small;border-collapse:separate" class=""><div class="">_,,,^..^,,,_<br class=""></div><div class="">best,&nbsp;Eliot</div></span></div></div></div>
</div></div>
</div></blockquote></div><br class=""></body></html>