[squeak-dev] Color fromString: broken

Bert Freudenberg bert at freudenbergs.de
Thu Dec 9 01:21:05 UTC 2010


On 08.12.2010, at 16:36, Levente Uzonyi wrote:

> On Wed, 8 Dec 2010, Bert Freudenberg wrote:
> 
>> 
>> On 08.12.2010, at 15:36, Levente Uzonyi wrote:
>> 
>>> On Wed, 8 Dec 2010, Bert Freudenberg wrote:
>>> 
>>>> Test case:
>>>> 
>>>> 	Color fromString: 'yellow'
>>>> 
>>>> used to return yellow. Now it returns a very dark gray.
>>>> 
>>>> The culprit is in #asNumber:
>>>> 
>>>> 	'16rYE' asNumber
>>>> 
>>>> used to raise an error. Now it silently returns 16.
>>> 
>>> It returns 0 since Squeak 3.9.
>> 
>> No, it returns 16 now. It ignores everything after the "6".
> 
> I see. It did return 0 in Squeak 3.9. Seems like ExtendedNumberParser is not fully backwards compatible, but this is an edge case.
> 
>> 
>>> Relying on such an error is bad idea and fixing this method is easy.
>> 
>> Agreed.
>> 
>>> Though it may have been hard to track the problem down.
>> 
>> Not *that* hard, fortunately ;)
>> 
>>> I wonder why is ColorNames an OrderedCollection instead of an IdentitySet.
>> 
>> No idea. Historical I guess.
> 
> Okay, I'll change it. :)

Set should be fine, no?

- Bert -





More information about the Squeak-dev mailing list