So add isLargeInteger to Number and LargePosiitveInteger and LargeNegativeInteger and use that instead. It may even be faster ;)<br><br><div class="gmail_quote">On Tue, Jul 12, 2011 at 8:05 AM, Chris Muller <span dir="ltr"><<a href="mailto:asqueaker@gmail.com">asqueaker@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I need to test it, but this looks like it will hurt me. I have<br>
subclassed LargePositiveInteger but I need it to compare equally - but<br>
with this hard class-check it will now fail....<br>
<div><div></div><div class="h5"><br>
On Sun, Jul 10, 2011 at 9:36 AM, <<a href="mailto:commits@source.squeak.org">commits@source.squeak.org</a>> wrote:<br>
> Levente Uzonyi uploaded a new version of Kernel to project The Trunk:<br>
> <a href="http://source.squeak.org/trunk/Kernel-ul.604.mcz" target="_blank">http://source.squeak.org/trunk/Kernel-ul.604.mcz</a><br>
><br>
> ==================== Summary ====================<br>
><br>
> Name: Kernel-ul.604<br>
> Author: ul<br>
> Time: 10 July 2011, 4:36:20.467 pm<br>
> UUID: f758876f-a507-404b-aa2f-2c1038504b48<br>
> Ancestors: Kernel-bf.603<br>
><br>
> Optimized Integer >> #= for integer-integer comparisons. Return false for integers from different classes without further checking.<br>
><br>
> =============== Diff against Kernel-bf.603 ===============<br>
><br>
> Item was changed:<br>
> ----- Method: Integer>>= (in category 'comparing') -----<br>
> = aNumber<br>
> +<br>
> + aNumber isInteger ifTrue: [<br>
> + aNumber class == self class ifFalse: [ ^false ].<br>
> + ^(self digitCompare: aNumber) = 0 ].<br>
> + aNumber isNumber ifFalse: [ ^false ].<br>
> + ^aNumber adaptToInteger: self andCompare: #=!<br>
> - aNumber isNumber ifFalse: [^ false].<br>
> - aNumber isInteger ifTrue:<br>
> - [aNumber negative == self negative<br>
> - ifTrue: [^ (self digitCompare: aNumber) = 0]<br>
> - ifFalse: [^ false]].<br>
> - ^ aNumber adaptToInteger: self andCompare: #=!<br>
><br>
><br>
><br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>best,<div>Eliot</div><br>