<div dir="ltr">Yay! That sounds awesome!<div><br></div><div>I'll take it for a ride tonight and let you folks know how it worked.</div><div><br></div><div>Thanks!</div><div>Nico PM</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 27, 2020 at 10:55 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">
    On 10/26/2020 10:41 PM, Nicolás Papagna Maldonado via Cuis-dev
    wrote:
    <blockquote type="cite">
      <div dir="ltr">Hi folks!
        <div>Hope you're doing fine :)</div>
        <div>
          <div><br>
          </div>
          <div>Just found this issue while I was trying to add a new <font face="monospace">Preference</font> whose value is a block.</div>
          <div>Specifically, I want users to be able to configure the
            block that should be used for opening the class finder
            (either the default one or <font face="monospace">Cuis-Finder</font>).</div>
          <div><br>
          </div>
          <div>The issue starts when <font face="monospace">Preferences>>compileAccessMethodForPreference: </font>is
            evaluated: the message <font face="monospace">storeString</font>
            is sent to the preference value (in this case a block) to
            create the code for its accessor method.</div>
          <div>Turns out that if you send <font face="monospace">storeString</font> to
            a block (say <font face="monospace">[] storeString</font>)
            the image freezes and the only way to exit is to kill the
            process.</div>
          <div><br>
          </div>
          <div>The whole thing seems to be caused by the default
            implementation that is used (<font face="monospace">Object>>storeOn:</font>)
            when the first instance variable is retrieved.</div>
          <div>I'm not sure what a custom implementation of <font face="monospace">BlockClosure>>storeOn: </font>or if
            it should have one at all, but if we are unable to store
            blocks into strings maybe it will be better if we fail.</div>
          <div><br>
          </div>
          <div>What are your thoughts?</div>
          <div><br>
          </div>
          <div>Cheers,</div>
          <div>Nico PM</div>
        </div>
      </div>
    </blockquote>
    <br>
    Hi Nico, Folks,<br>
    <br>
    You are lucky today! This problem is not trivial, and our friend Ken
    Dickey complained for years about not being able to store closures
    in SmartRefStreams. I finally implemented the solution about a year
    ago. So, the only thing you need is <br>
    BlockClosure >> <br>
    storeOn: aStream<br>
        aStream nextPut: $(.<br>
        self asSerializable storeOn: aStream.<br>
        aStream nextPutAll: ' asEvaluable)'<br>
    <br>
    Please try it. If this works fine for everybody, I'll push it to
    GitHub.<br>
    <br>
    BTW, you can browse SerializableBlockClosure to see how I did it. It
    took me many hours to get it right! This has many applications, for
    example, the ability to store any object (including
    SortedCollections with sortBlock, live Morphs, etc) on files or even
    the clipboard, and materializing them in a different image.<br>
    <br>
    Cheers,<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><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><br>Nicolás Papagna</div>