« Return to Thread: Surrogate pair output support (unit test)

Re: Surrogate pair output support (unit test)

by Olivier Jaquemet :: Rate this Message:

Reply to Author | View in Thread

Jason Hunter wrote:
>> A simple test case for this bug is attached and end of this mail.
>> ...
> Excellent!  With that I'll feel safer checking it in.  It doesn't seem
> in big demand, but if we can do that's right w/ surrogate pairs then
> we should.
I'm glad I could help a bit with this test... Some "warnings" though :
- it is quite a high level unit test as I don't know the low level JDOM
API to test....
- it does not test the boundary condition (invalid surrogate pair) which
Dave's properly checked in his code.
- it does not perform regression testing on other basic character set
such as ASCII, ISO-8859-1 or UTF-8, but I suppose you already have valid
unit tests on this..

Also note that the issue you were raising a year ago regarding the
method signature change introduced by Dave's patch is still there... You
may wan't to have a deeper look at this if you don't want to break
people's code in your next release.

Regards,
Olivier.

_______________________________________________
To control your jdom-interest membership:
http://www.jdom.org/mailman/options/jdom-interest/youraddr@...

 « Return to Thread: Surrogate pair output support (unit test)