Thread: [Jsoncpp-devel] A little bug in parsing array with comment after last item.

SourceForge Headquarters 225 Broadway Suite 1600 San Diego, CA 92101 +1 (858) 422-6466

<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
</head>
<body bgcolor="#ffffff" text="#000000">
Hi, 
 
Example : 
 
{ 
&nbsp;&nbsp;&nbsp; "test": 
&nbsp;&nbsp;&nbsp; [ 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp; { "a" : "aaa" }, // Comment for a 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp; { "b" : "bbb" }, // Comment for b 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp; { "c" : "ccc" } // Comment for c 
&nbsp;&nbsp;&nbsp; ] 
} 
 
With this example you get an error : "Missing ',' or ']' in array
declaration" for "// Comment for c" 
 
I fixed the problem by reading tokens in the same way readObject() does. 
 
Fixed version of&nbsp; readArray() : 
 
<tt>bool 
Reader::readArray( Token &amp;tokenStart ) 
{ 
&nbsp;&nbsp;&nbsp; &nbsp;currentValue() = Value( arrayValue ); 
&nbsp;&nbsp;&nbsp; &nbsp;skipSpaces(); 
&nbsp;&nbsp;&nbsp; &nbsp;if ( *current_ == ']' ) // empty array 
&nbsp;&nbsp;&nbsp; &nbsp;{ 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; Token endArray; 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; readToken( endArray ); 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; return true; 
&nbsp;&nbsp;&nbsp; &nbsp;} 
&nbsp;&nbsp;&nbsp; &nbsp;int index = 0; 
&nbsp;&nbsp;&nbsp; &nbsp;while ( true ) 
&nbsp;&nbsp;&nbsp; &nbsp;{ 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; Value &amp;value = currentValue()[ index++ ]; 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; nodes_.push( &amp;value ); 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; bool ok = readValue(); 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; nodes_.pop(); 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if ( !ok ) // error already set 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;return recoverFromError( tokenArrayEnd ); 
 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; Token token; 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp; // Accept Comment after last item
in the array. 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;ok = readToken( token ); 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;while ( token.type_ == tokenComment &amp;&amp; ok ) 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp; ok = readToken( token ); 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;if( ! ok 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp; ||&nbsp; ( token.type_ !=
tokenArraySeparator&nbsp; &amp;&amp; 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;token.type_ != tokenArrayEnd ) ) 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; { 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;return addErrorAndRecover( "Missing ',' or ']' in
array declaration", 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;
token, 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;
tokenArrayEnd ); 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; } 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if ( token.type_ == tokenArrayEnd ) 
&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;break; 
&nbsp;&nbsp;&nbsp; &nbsp;} 
&nbsp;&nbsp;&nbsp; &nbsp;return true; 
} 
</tt> 
 
Regards, 
Fr&eacute;d&eacute;ric SURLEAU. 
 
</body>
</html>

Thread: [Jsoncpp-devel] A little bug in parsing array with comment after last item.

C++ JSON parser (Now in GitHub)

jsoncpp-devel