<div dir="ltr">(keyboard failure)<br><div class="gmail_quote"><div dir="ltr">On Sun, Oct 28, 2018 at 3:41 PM Chris Cunningham <<a href="mailto:cunningham.cb@gmail.com">cunningham.cb@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr">Looking at LargeIntegers (I'm 64 bit, so these are big):<div><div>{</div><div>1152921504606846977 digitCompare:  -1152921504606846977.</div><div>1152921504606846977 digitCompare:  -1152921504606846978.</div><div>1152921504606846978 digitCompare:  -1152921504606846977.</div><div>}  "#(0 -1 1)"</div></div><div><br></div><div><div>{</div><div>1249 digitCompare: -1249.</div><div>1249 digitCompare: -1250.</div><div>1250 digitCompare: -1249</div></div></div></div></div></blockquote><div> }   "#(1 1 1)"</div><div><br></div><div>The method is actively different for SmallIntegers and LargeIntegers.</div><div><br></div><div>I'm not sure how much this matters - besides the fact that it is lying around waiting for someone to use it.  For comparisons of integers, the callers check to see if one (or more) of the numbers are negative, then handles the issues here.</div><div><br></div><div>It is also currently used in DateAndTime class methods (#nowWithOffset: [deprecated] and #now:offset:), which is probably safe because now is always after the epoch.  And we hopefully wouldn't use #now:withOffset: for pre-epoch times.</div><div><br></div><div>Is this worth fixing?</div><div><br></div><div>-cbc</div><div><br></div><div><br></div></div></div>