[Glass] MAStringReader>>visitNumberDescription broken?

[Dale Henrichs via Glass]

Iwan,

Okay, then I need to dive into the Magritte implementation directly and 
try to figure things out from there ... I naively thought that the 
problem was the direct use of NumberParser in the code, instead of 
SqNumberParser - it would be relatively easy to change NumberParser to 
redirect instance creation to SqNumberParser, but It sounds like that 
isn't the source ofthe problem ...

BTW are Magritte tests failing on GemStone or is this an untested chunk 
of code?

Dale

On 09/08/2015 02:50 AM, Iwan Vosloo via Glass wrote:
> On 07/09/2015 16:56, Dale Henrichs via Glass wrote:
>> It turns out that in the GemStone/GsDevKit version of NumberParser 
>> (actually SqNumberParser), isNumber: is not implemented ... was there 
>> a major rewrite of NumberParser in Pharo4.0 (it's not in Pharo3.0 
>> either) or is it a simple extension method?
>>
>> I might be able to squeeze out some time and take a bigger look at 
>> this one:)
>
> From what I can see, Magritte-Model adds isNumber in the Pharo and 
> GemStone case. The real problem is perhaps how isNumber: is 
> implemented OR perhaps the implementation of parse:onError: which it 
> sends.
>
> Regards
> - Iwan
>