help needed for xmlparser

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

help needed for xmlparser

Alexandre Bergel
Hi!

There has been a yellow test for some times already.
XMLParserTest>>#testDecodingCharacters

I am not really sure how to fix it. It would be _really_ cool to have a character encoding guru to have a look at it...
If you fixed it and happen to be at esug, I invite you for a beer :-)

Cheers,
Alexandre

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
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: help needed for xmlparser

Henrik Sperre Johansen
Unauthorized :/

Basically, the two last asserts should be removed, as in both Squeak and Pharo, leadingChar for Unicode has been changed from 255 to 0.
Doesn't really make sense to test for leadingChar anyways, what you really care about is the charCode...

Cheers,
Henry

On Sep 14, 2010, at 9:42 39AM, Alexandre Bergel wrote:

> Hi!
>
> There has been a yellow test for some times already.
> XMLParserTest>>#testDecodingCharacters
>
> I am not really sure how to fix it. It would be _really_ cool to have a character encoding guru to have a look at it...
> If you fixed it and happen to be at esug, I invite you for a beer :-)
>
> Cheers,
> Alexandre
>
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>
>
> _______________________________________________
> 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
Reply | Threaded
Open this post in threaded view
|

Re: help needed for xmlparser

Alexandre Bergel
In reply to this post by Alexandre Bergel
Henrik will get the beer.

All tests of XML-Support are green again!

ConfigurationOfXMLSupport has been updated accordingly.

thanks Henrik!
Alexandre


On 14 Sep 2010, at 09:42, Alexandre Bergel wrote:

> Hi!
>
> There has been a yellow test for some times already.
> XMLParserTest>>#testDecodingCharacters
>
> I am not really sure how to fix it. It would be _really_ cool to have a character encoding guru to have a look at it...
> If you fixed it and happen to be at esug, I invite you for a beer :-)
>
> Cheers,
> Alexandre
>
> --
> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
> Alexandre Bergel  http://www.bergel.eu
> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>
>
>
>
>

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
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: help needed for xmlparser

Alexandre Bergel
In reply to this post by Henrik Sperre Johansen
I added you in the project.

Cheers,
Alexandre


On 14 Sep 2010, at 10:06, Henrik Johansen wrote:

> Unauthorized :/
>
> Basically, the two last asserts should be removed, as in both Squeak and Pharo, leadingChar for Unicode has been changed from 255 to 0.
> Doesn't really make sense to test for leadingChar anyways, what you really care about is the charCode...
>
> Cheers,
> Henry
>
> On Sep 14, 2010, at 9:42 39AM, Alexandre Bergel wrote:
>
>> Hi!
>>
>> There has been a yellow test for some times already.
>> XMLParserTest>>#testDecodingCharacters
>>
>> I am not really sure how to fix it. It would be _really_ cool to have a character encoding guru to have a look at it...
>> If you fixed it and happen to be at esug, I invite you for a beer :-)
>>
>> Cheers,
>> Alexandre
>>
>> --
>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>> Alexandre Bergel  http://www.bergel.eu
>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>>
>>
>>
>>
>>
>>
>> _______________________________________________
>> 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

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






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