Bugs item #3005593, was opened at 2010-05-22 03:35
Message generated for change (Tracker Item Submitted) made by nobody
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=462816&aid=3005593&group_id=51305
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: wxJSON
Group: last released version
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: wxJSONReader - multiline C-style comment error
Initial Comment:
Multiline C-style comments (/*....*/) was supported correctly in wxJSON 1.0
But it's now not correctly parsed with current release (wxJSON 1.2)
The bug is in wxJSONReader::SkipComment() method.
while ( ch >= 0 ) {
// check the END-COMMENT chars ('*/')
if ( ch == '*' ) {
ch = PeekChar( is );
if ( ch == '/' ) {
ch = ReadChar( is ); <--- The comment close '/' char will be return instead of the next char
utf8Buff.AppendData( "*/", 2 );
break;
}
}
// store the char in the UTF8 temporary buffer
c = (unsigned char) ch;
utf8Buff.AppendByte( c );
ch = ReadChar( is );
}
The following should fix this bug:
while ( ch >= 0 ) {
// check the END-COMMENT chars ('*/')
if ( ch == '*' ) {
ch = PeekChar( is );
if ( ch == '/' ) {
ch = ReadChar( is ); //read pass the comment close char
ch = ReadChar( is );// read the next char that will be returned
utf8Buff.AppendData( "*/", 2 );
break;
}
}
// store the char in the UTF8 temporary buffer
c = (unsigned char) ch;
utf8Buff.AppendByte( c );
ch = ReadChar( is );
}
Thank you for this great library!
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=462816&aid=3005593&group_id=51305
|
