right,

another route to go would be automatically xml escaping the characters (eg. 
'\n' becomes '
') and leaving it to the reader to interpret them or not.

..ede

On 16.12.2016 13:35, Michaël Michaud wrote:
> Hi Ede,
> 
> Whoops, thanks !
> 
> Yes, GML writer is somewhat archaic. But changing the whole class to use 
> a xml library would be a bit of work I think. Using 
> StringEscapeUtils.escapeXml(str) would be easier, but I could not 
> confirm it escapes non-printable characters like \u0001, \u0002...
> 
> Michaël
> 
> 
> Le 16/12/2016 à 12:24, [email protected] a écrit :
>> hey Mike,
>>
>> On 16.12.2016 09:03, [email protected] wrote:
>>> +            if ((int)c < 20 && c != '\t' && c != '\n' && c != 'r') {
>>> +                sb.replace(t, t + 1, "");
>>> +            }
>> c!='r' should probably be '\r' ?
>>
>> it would be cleaner to use an xml library to escape strings properly (we've 
>> got Xerces in OJ, you could use it's XMLSerializer).
>>
>> easier would be to use apache's StringEscapeUtils.escapeXml(str) .
>>
>> ..ede
>>
>> ------------------------------------------------------------------------------
>> Check out the vibrant tech community on one of the world's most
>> engaging tech sites, SlashDot.org! http://sdm.link/slashdot
>> _______________________________________________
>> Jump-pilot-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>
> 
> 
> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most 
> engaging tech sites, SlashDot.org! http://sdm.link/slashdot
> _______________________________________________
> Jump-pilot-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Jump-pilot-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to