
<html>

  <head>

    <meta content="text/html; charset=ISO-8859-1"

      http-equiv="Content-Type">

  </head>

  <body text="#000000" bgcolor="#FFFFFF">

    <font face="Georgia">Looks a bit odd. If byte is less than zero,

      then exit. Followed by another test for byte less than zero? Looks

      like old and new code mixed.<br>

      <br>

      Cheers,<br>

      Bob<br>

      <br>

      <br>

    </font>

    <div class="moz-cite-prefix">On 11/2/13 6:30 PM, Nicolas Cellier

      wrote:<br>

    </div>

    <blockquote

cite="mid:CAKnRiT66gQhpUz68zmi6q-m9Ghkpzdo4VQZEVwySX+NnxW94Pw@mail.gmail.com"

      type="cite">

      <pre wrap=""> </pre>

      <br>

      <fieldset class="mimeAttachmentHeader"></fieldset>

      <br>

      <div dir="ltr">

        <div>In recent COG, I see:<br>

          <br>

          LittleEndianBitmap&gt;&gt;byteAt: byteAddress put: byte<br>

          &nbsp;&nbsp;&nbsp; "Insert a byte into a Bitmap (little-endian version).&nbsp;

          N.B. Signedness will be lost since byteAt: answers

          non-negative values.<br>

          &nbsp;&nbsp;&nbsp; &nbsp;We can add a signedByteAt: if needed."<br>

          &nbsp;&nbsp;&nbsp; | value longWord shift lowBits longAddr |<br>

          &nbsp;&nbsp;&nbsp; (byte &lt; 0 or:[byte &gt; 255]) ifTrue:[^self

          errorImproperStore].<br>

          &nbsp;&nbsp;&nbsp; value := byte &lt; 0<br>

          &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; ifTrue: [byte &lt; 128 ifTrue:<br>

          &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; [self errorImproperStore].<br>

          <br>

        </div>

        It looks suspect to me, wouldn't the intention be <br>

        <br>

        &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; ifTrue: [byte &lt; -128 ifTrue:<br>

        &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; [self errorImproperStore].</div>

    </blockquote>

    <br>

  </body>

</html>