setComment() will assert if comment does not start with / (or if it were NULL, which would have seg-faulted before).

cdunn2001 · cdunn2001 · commit ca2125639515 · 2007-03-23T07:05:19.000Z
diff --git a/include/json/value.h b/include/json/value.h
@@ -317,8 +317,10 @@ namespace Json {
 //      EnumValues enumValues() const;
 //# endif
 
+      /// Comments must be //... or /* ... */
       void setComment( const char *comment,
                        CommentPlacement placement );
+      /// Comments must be //... or /* ... */
       void setComment( const std::string &comment,
                        CommentPlacement placement );
       bool hasComment( CommentPlacement placement ) const;
diff --git a/src/lib_json/json_value.cpp b/src/lib_json/json_value.cpp
@@ -1,3 +1,4 @@
+#include <iostream>
 #include <json/value.h>
 #include <json/writer.h>
 #include <utility>
@@ -149,6 +150,9 @@ Value::CommentInfo::setComment( const char *text )
 {
    if ( comment_ )
       valueAllocator()->releaseStringValue( comment_ );
+   JSON_ASSERT( text );
+   JSON_ASSERT_MESSAGE( text[0]==NULL || text[0]=='/', "Comments must start with /");
+   // It seems that /**/ style comments are acceptable as well.
    comment_ = valueAllocator()->duplicateStringValue( text );
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+#include <iostream>`
`1`	`2`	`#include <json/value.h>`
`2`	`3`	`#include <json/writer.h>`
`3`	`4`	`#include <utility>`
`@@ -149,6 +150,9 @@ Value::CommentInfo::setComment( const char *text )`
`149`	`150`	`{`
`150`	`151`	`if ( comment_ )`
`151`	`152`	`valueAllocator()->releaseStringValue( comment_ );`
	`153`	`+ JSON_ASSERT( text );`
	`154`	`+ JSON_ASSERT_MESSAGE( text[0]==NULL \|\| text[0]=='/', "Comments must start with /");`
	`155`	`+ // It seems that /**/ style comments are acceptable as well.`
`152`	`156`	`comment_ = valueAllocator()->duplicateStringValue( text );`
`153`	`157`	`}`
`154`	`158`