SqNumberParser!

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

SqNumberParser!

Göran Krampe
Howdy folks!

In working on Tirade I stumbled upon SqNumberParser and its history:

http://bugs.squeak.org/view.php?id=3512

...it was added to the image in 3.9 but not hooked into Scanner -
Scanned still uses Number class>>readFrom:.

Now... In Tirade I can use whichever - but using SqNumberParser makes it
more correct, and quite faster - although "less portable" since I would
depend on it. I will now make Tirade use SqNumberParser *if present* in
the image, otherwise fall back on Number class>>readFrom:. This opens up
some slight potential incompatibility issues, but whatever.

Question: Is there anything stopping 3.9+ including Pharo to start using
SqNumberParser from Scanner>>xDigit?

And oh, the class name is not so good IMHO, why the Sq prefix?

And yeah, class var not used in SqNumberParser: BelllerophonBase10 :)

regards, Göran


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: SqNumberParser!

Stéphane Ducasse
if nicolas do it we will integrate it I imagine :)

On Mar 20, 2009, at 11:22 AM, Göran Krampe wrote:

> Howdy folks!
>
> In working on Tirade I stumbled upon SqNumberParser and its history:
>
> http://bugs.squeak.org/view.php?id=3512
>
> ...it was added to the image in 3.9 but not hooked into Scanner -
> Scanned still uses Number class>>readFrom:.
>
> Now... In Tirade I can use whichever - but using SqNumberParser  
> makes it
> more correct, and quite faster - although "less portable" since I  
> would
> depend on it. I will now make Tirade use SqNumberParser *if present*  
> in
> the image, otherwise fall back on Number class>>readFrom:. This  
> opens up
> some slight potential incompatibility issues, but whatever.
>
> Question: Is there anything stopping 3.9+ including Pharo to start  
> using
> SqNumberParser from Scanner>>xDigit?
>
> And oh, the class name is not so good IMHO, why the Sq prefix?
>
> And yeah, class var not used in SqNumberParser: BelllerophonBase10 :)
>
> regards, Göran
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project