MantisBT - CDash
View Issue Details
0006917CDashpublic2008-04-28 16:242008-05-14 17:58
François Bertel 
Julien Jomier 
normalminoralways
closedfixed 
0.8 
1.0.2 
0006917: cvs comments are report wrong if they include some simple quote
This is a regression compared to dart1.

Dart1 OK:
http://www.paraview.org/ParaView3/Testing/Sites/amber5.kitware/Linux-g++-cont/20080428-1733-Continuous/Update.html [^]

Cdash wrong:
http://www.cdash.org/CDash/viewUpdate.php?buildid=64400 [^]

"when it shouldn't." on dart1 becomes "when it shouldn't." on cdash.
No tags attached.
Issue History
2008-04-28 16:24François BertelNew Issue
2008-04-28 16:26François BertelNote Added: 0011573
2008-04-28 17:46Julien JomierStatusnew => assigned
2008-04-28 17:46Julien JomierAssigned To => Julien Jomier
2008-04-28 17:48Julien JomierStatusassigned => closed
2008-04-28 17:48Julien JomierNote Added: 0011576
2008-04-28 17:48Julien JomierResolutionopen => fixed
2008-04-28 17:48Julien JomierFixed in Version => 1.0
2008-05-12 23:14François BertelStatusclosed => feedback
2008-05-12 23:14François BertelResolutionfixed => reopened
2008-05-12 23:14François BertelNote Added: 0011847
2008-05-12 23:32Julien JomierStatusfeedback => closed
2008-05-12 23:32Julien JomierNote Added: 0011848
2008-05-12 23:32Julien JomierResolutionreopened => fixed
2008-05-12 23:32Julien JomierFixed in Version1.0 => 1.0.1
2008-05-13 10:54François BertelStatusclosed => feedback
2008-05-13 10:54François BertelResolutionfixed => reopened
2008-05-13 10:54François BertelNote Added: 0011858
2008-05-14 17:58Julien JomierStatusfeedback => closed
2008-05-14 17:58Julien JomierNote Added: 0011925
2008-05-14 17:58Julien JomierResolutionreopened => fixed

Notes
(0011573)
François Bertel   
2008-04-28 16:26   
My comment did not pass well on mantis because mantis corrected the syntax...

Just look at the links to see the difference.
(0011576)
Julien Jomier   
2008-04-28 17:48   
Fixed in SVN. A javascript function to convert the encoded XML back to ascii has been added. Thanks for the report.
Feel free to reopen if some characters are still now showing up correctly.
(0011847)
François Bertel   
2008-05-12 23:14   
I reopen the bug: the simple quote is fixed but not the double quote.

Example:
http://www.cdash.org/CDash/viewUpdate.php?buildid=74019 [^]
(0011848)
Julien Jomier   
2008-05-12 23:32   
Fixed in 1.0.1 and in trunk. Thanks for the report. As usual, feel free to reopen as needed.
(0011858)
François Bertel   
2008-05-13 10:54   
I reopen again for bad translation of >

Real cvs comment is:

ENH: all ctype function have the same issue: char can be signed or unsigned, sin
ce isspace only deal with >=0 value (except EOF) one has to first cast it to uns
igned char


Compare that to the wrong result with cdash:
http://www.cdash.org/CDash/viewUpdate.php?buildid=74808 [^]
(0011925)
Julien Jomier   
2008-05-14 17:58   
Fixed in 1.0.1 (1.0 branch). Thanks for the report.