ignore bom at the beginning of the UTF-8 text

add a testcase about bom.

Author: dota17

src/lib_json/json_reader.cpp

@@ -18,6 +18,7 @@
 #include <memory>
 #include <set>
 #include <sstream>
+#include <string.h>
 #include <utility>
 
 #include <cstdio>
@@ -939,6 +940,7 @@ class OurReader {
 
   bool readToken(Token& token);
   void skipSpaces();
+  void skipBom(Location& begin);
   bool match(const Char* pattern, int patternLength);
   bool readComment();
   bool readCStyleComment(bool* containsNewLineResult);
@@ -1010,6 +1012,7 @@ bool OurReader::parse(const char* beginDoc, const char* endDoc, Value& root,
     collectComments = false;
   }
 
+  skipBom(beginDoc);
   begin_ = beginDoc;
   end_ = endDoc;
   collectComments_ = collectComments;
@@ -1021,7 +1024,6 @@ bool OurReader::parse(const char* beginDoc, const char* endDoc, Value& root,
   while (!nodes_.empty())
     nodes_.pop();
   nodes_.push(&root);
-
   bool successful = readValue();
   nodes_.pop();
   Token token;
@@ -1268,6 +1270,12 @@ void OurReader::skipSpaces() {
   }
 }
 
+void OurReader::skipBom(Location& begin) {
+  if (strncmp(begin, "\xEF\xBB\xBF", 3) == 0) {
+    begin += 3;
+  }
+}
+
 bool OurReader::match(const Char* pattern, int patternLength) {
   if (end_ - current_ < patternLength)
     return false;

src/test_lib_json/main.cpp

@@ -3578,6 +3578,20 @@ JSONTEST_FIXTURE_LOCAL(BuilderTest, settings) {
   }
 }
 
+struct BomTest : JsonTest::TestCase {};
+
+JSONTEST_FIXTURE_LOCAL(BomTest, withBom) {
+  const std::string with_bom = u8"\xEF\xBB\xBF{}";
+  Json::Value root;
+  JSONCPP_STRING errs;
+  std::istringstream iss(with_bom);
+  bool ok = parseFromStream(Json::CharReaderBuilder(), iss, &root, &errs);
+  JSONTEST_ASSERT(ok);
+  if (!ok) {
+    std::cout << "errs: " << errs << std::endl;
+  }
+}
+
 struct IteratorTest : JsonTest::TestCase {};
 
 JSONTEST_FIXTURE_LOCAL(IteratorTest, convert) {