<div dir="ltr">Martin,<div>Thanks for the very detailed explanation! I went ahead and replaced '@-' with '@ -' in the image and the tests. There are still 10 tests producing errors because the decompiler decompiles it without the space, tho.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 9, 2022 at 5:04 PM Martin McClure <<a href="mailto:martin@hand2mouse.com">martin@hand2mouse.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>
    <div>The ANSI Smalltalk grammar says that
      "@-" should be a valid binary selector, and would require
      whitespace in "1@ -1" to interpret the argument as a literal
      negative one.<br>
      <br>
      The Blue Book grammar, though, does not allow "@-" as a binary
      selector. The Blue Book grammar, as printed, also does not allow
      "," as a binary selector, even though that was clearly used as a
      binary selector in Smalltalk-80. The Blue Book grammar also does
      not talk about whitespace at all, even though that is clearly
      required in some places, so the Blue Book is a bit fuzzier than
      one would want as a grammar reference.<br>
      <br>
      Going with ANSI-like grammar and putting a space in expressions
      like "1 @- 1" makes more sense to me. Allows more valid selectors,
      and the code is more readable with the whitespace.<br>
      <br>
      Regards,<br>
      -Martin <br>
      <br>
      On 5/9/22 03:48, Luciano Notarfrancesco via Cuis-dev wrote:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="ltr">After this change '1@-1' is parsed as a binary
        message #@-, and this causes some base image tests to fail.
        Should we add a space between @ and -1 or should we fix the
        parser?</div>
      <br>
      <div class="gmail_quote">
        <div dir="ltr" class="gmail_attr">On Fri, May 6, 2022 at 1:56 PM
          Luciano Notarfrancesco <<a href="mailto:luchiano@gmail.com" target="_blank">luchiano@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="auto">That was quick. Thanks a lot!</div>
          <div><br>
            <div class="gmail_quote">
              <div dir="ltr" class="gmail_attr">On Fri, 6 May 2022 at
                8:55 PM Juan Vuletich <<a href="mailto:JuanVuletich@zoho.com" target="_blank">JuanVuletich@zoho.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 bgcolor="#ffffff"> On 5/6/2022 10:48 AM, Luciano
                  Notarfrancesco via Cuis-dev wrote:
                  <blockquote type="cite">
                    <div>I’m not sure I was clear, I meant a method
                      implementing those messages, not sending those
                      messages.</div>
                    <div><br>
                    </div>
                    <div>
                      <div>
                        <div>On Fri, 6 May 2022 at 1:22 PM Luciano
                          Notarfrancesco <<a href="mailto:luchiano@gmail.com" target="_blank">luchiano@gmail.com</a>>
                          wrote:<br>
                        </div>
                        <blockquote>I’m getting “Argument name expected”
                          when trying to compile a method with selector
                          <|. On the other hand, |>, | and || all
                          compile fine as binary selectors, but ||
                          breaks syntax highlighting. </blockquote>
                      </div>
                    </div>
                  </blockquote>
                  <br>
                </div>
                <div bgcolor="#ffffff"> Sure. Fixes at GitHub now.<br>
                  <br>
                  A small detail: Now an empty temporal declaration `||`
                  is parsed as a binary selector, and therefore marked
                  as invalid. Not a big deal. We'd just remove any such.<br>
                  <br>
                  Cheers,<br>
                  <pre cols="72" style="font-family:monospace">-- 
Juan Vuletich
<a href="http://www.cuis-smalltalk.org" style="font-family:monospace" target="_blank">www.cuis-smalltalk.org</a>
<a href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" style="font-family:monospace" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a>
<a href="https://github.com/jvuletich" style="font-family:monospace" target="_blank">https://github.com/jvuletich</a>
<a href="https://www.linkedin.com/in/juan-vuletich-75611b3" style="font-family:monospace" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a>
<a href="https://independent.academia.edu/JuanVuletich" style="font-family:monospace" target="_blank">https://independent.academia.edu/JuanVuletich</a>
<a href="https://www.researchgate.net/profile/Juan-Vuletich" style="font-family:monospace" target="_blank">https://www.researchgate.net/profile/Juan-Vuletich</a>
<a href="https://patents.justia.com/inventor/juan-manuel-vuletich" style="font-family:monospace" target="_blank">https://patents.justia.com/inventor/juan-manuel-vuletich</a>
<a href="https://twitter.com/JuanVuletich" style="font-family:monospace" target="_blank">https://twitter.com/JuanVuletich</a></pre>
                </div>
              </blockquote>
            </div>
          </div>
        </blockquote>
      </div>
      <br>
      <fieldset></fieldset>
    </blockquote>
    <br>
  </div>

</blockquote></div>