<div dir="ltr">Juan,<div><br></div><div>Fair enough.  I wasn't sure if you were intentionally trying to minimize image size on every save and was trying to offer an alternative if that was the case.</div><div><br></div><div>Thanks,</div><div>Phil</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 8, 2020 at 11:04 AM Juan Vuletich <<a href="mailto:juan@jvuletich.org">juan@jvuletich.org</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"><u></u>

  
    
  
  <div bgcolor="#ffffff">
    Hi Phil,<br>
    <br>
    On 5/8/2020 2:32 AM, Phil B via Cuis-dev wrote:
    <blockquote type="cite">
      <div dir="ltr">I just ran a quick test... yes, let's please have a
        fast save option!  When I eliminated the
        #releaseClassCachedState sends from the save process (and the
        corresponding #initClassCachedState on the other side of the
        save), the save took *maybe* 1/4 of a second.</div>
      <br>
      <div class="gmail_quote">
        <div dir="ltr" class="gmail_attr">On Fri, May 8, 2020 at 1:00 AM
          Phil B <<a href="mailto:pbpublist@gmail.com" target="_blank">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">I noticed that this method is significantly
            slowing down image saves... taking upwards of 10 seconds. 
            Partially it's due to my Morphs now wiping and recreating a
            bunch of state each save.  But it's also partially due just
            to the overhead of Morph allSubInstancesDo: [:ea| ] taking
            ~3-4 seconds to execute per save. (in an image with ~2k
            morphs, which isn't that hard to get to)
            <div><br>
              <div>If the goal is to minimize saved image file size,
                could we have options for a 'fast save' vs a 'small
                save'?  Unless I'm going to email/publish an image, I'd
                prefer the save to be fast rather than small.</div>
            </div>
          </div>
        </blockquote>
      </div>
    </blockquote>
    <br>
    We already have that. When you do a 'save as new version',
    additional, more expensive cleanup is done.<br>
    <br>
    The problem here is that recently I added a 'Morph
    allSubInstancesDo: [:ea| ]' on every image save. Apologies. Fix now
    on GitHub.<br>
    <br>
    Thanks,<br>
    <pre cols="72">-- 
Juan Vuletich
<a href="http://www.cuis-smalltalk.org" target="_blank">www.cuis-smalltalk.org</a>
<a href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a>
<a href="https://github.com/jvuletich" target="_blank">https://github.com/jvuletich</a>
<a href="https://www.linkedin.com/in/juan-vuletich-75611b3" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a>
@JuanVuletich</pre>
  </div>

</blockquote></div>