[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Handle special characters in Junit report

From: Branko Čibej <brane_at_xbc.nu>
Date: Fri, 04 Dec 2009 15:29:56 +0100

Bhuvaneswaran A wrote:
> On Fri, 2009-12-04 at 12:36 +0000, Julian Foad wrote:
>
>> Bhuvaneswaran A wrote:
>>
>>> Please find attached the revised patch. I incorporated following
>>> feedback:
>>> a) Fix the array slicing part
>>> b) Escape using ord() instead of removing those characters
>>> c) Handle "]]>" in CDATA section
>>> d) Define the ascii table globally (once) and re-use
>>>
>>> I also verified this fix by generating the junit files for tests having
>>> special characters and simulating a test that has "]]>" in failure text.
>>> With this patch, it generates valid junit file.
>>>
>> It looks great. You could also move the definition of 'chars_to_remove'
>> out of the function, but either way it's fine. Go on, commit it!
>>
>
> Branko, Julian: Thank you for the review comments.
>
> Incorporated the above suggestion and committed in r887178.

Wait, you committed a script that does URL quoting on XML contents? Did
you look at the output?

-- Brane
Received on 2009-12-04 15:30:30 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.