<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 20, 2017 at 5:54 PM, Chris Cunningham <span dir="ltr"><<a href="mailto:cunningham.cb@gmail.com" target="_blank">cunningham.cb@gmail.com</a>></span> wrote:<br><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"><div class="gmail_extra"><div><div class="gmail-h5"><br><div class="gmail_quote">On Thu, Apr 20, 2017 at 8:34 AM, Bert Freudenberg <span dir="ltr"><<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>></span> wrote:<br><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"><div class="gmail_extra"><div class="gmail_quote"><span>On Thu, Apr 20, 2017 at 4:57 AM, Chris Cunningham <span dir="ltr"><<a href="mailto:cunningham.cb@gmail.com" target="_blank">cunningham.cb@gmail.com</a>></span> wrote:<br><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">Actually, Elliot, did you want to keep it in the image?  I was removing it pursuant to Bert's suggestion.<div><br></div><div>After removing it myself, and then doing a Smalltalk garbageCollect, it is no longer anywhere in the image (or Undeclared, either).  (Although BlockContext is, but that's another issue.)</div><div><br></div><div>I have added a new package for it if you want to keep it around.  We went here because the ExternalForm was removed from FFI and put into EToys, and then Bert said it wasn't used in EToys (and it isn't).</div><div><br></div><div>Now in package: <a href="http://source.squeak.org/inbox/GraphicsExternal-cbc.1.mcz" rel="noreferrer" style="font-size:12.8px" target="_blank">http://source.squeak.<wbr>org/inbox/GraphicsExternal-cbc<wbr>.1.mcz</a></div></div></blockquote><div><br></div></span><div>Nice. I think you missed some extension methods, e.g. #isExternalForm,  #isFillAccelerated:for:, #isBltAccelerated:for:, #displayScreen in class Form. If you move these into a *graphicsexternal category, they will become part of that package. </div><span class="gmail-m_6850268827510339795HOEnZb"><font color="#888888"><div><br></div><div>- Bert - </div></font></span></div></div></div>
</blockquote></div></div></div>Ok.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I can see that I probably should have noticed #isExternalForm, but I wouldn't have had a chance to find the other ones. </div></div></blockquote><div><br></div><div>I just gave the code a cursory look ... these are implemented in ExternalForm but there is a base implementation.</div><div> </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"><div class="gmail_extra"> Also, these weren't part of the Etoys nor FFI packages - they were in the base Graphics package.</div></div></blockquote><div><br></div><div>Yeah, isn't it fun to untangle a monolithic system? ;)</div><div> </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"><div class="gmail_extra">That said, I'll moved them over.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Anything else lying around?</div></div></blockquote><div><br></div><div>There is also the stuff with hasNonStandardPalette and convertARGB etc which is only used if there are external forms, but that seems a bit harder to remove because the hooks need to be there. I think it's fine to leave that as it is, unless you notice something that's easily removable.</div><div><br></div><div>- Bert -</div></div></div></div>