JavaScriptExpert
diff --git a/‎.drone.yml
Lines changed: 22 additions & 0 deletions b/‎.drone.yml
Lines changed: 22 additions & 0 deletions
diff --git a/‎benchmark/bench_dom_api.cpp
Lines changed: 35 additions & 31 deletions b/‎benchmark/bench_dom_api.cpp
Lines changed: 35 additions & 31 deletions
diff --git a/‎benchmark/benchmarker.h
Lines changed: 2 additions & 2 deletions b/‎benchmark/benchmarker.h
Lines changed: 2 additions & 2 deletions
diff --git a/‎benchmark/distinctuseridcompetition.cpp
Lines changed: 1 addition & 1 deletion b/‎benchmark/distinctuseridcompetition.cpp
Lines changed: 1 addition & 1 deletion
diff --git a/‎benchmark/parse_stream.cpp
Lines changed: 2 additions & 2 deletions b/‎benchmark/parse_stream.cpp
Lines changed: 2 additions & 2 deletions
diff --git a/‎fuzz/fuzz_dump.cpp
Lines changed: 4 additions & 1 deletion b/‎fuzz/fuzz_dump.cpp
Lines changed: 4 additions & 1 deletion
@@ -178,6 +178,28 @@ steps:
     - make -j
     - ctest --output-on-failure
 ---
+  kind: pipeline
+  name: amd64_clang_cmake_no_exceptions
+  
+  platform:
+    os: linux
+    arch: amd64
+  
+  steps:
+  - name: Build and Test
+    image: ubuntu:18.04
+    environment:
+      CC: clang
+      CXX: clang++
+      CMAKE_FLAGS: -DSIMDJSON_BUILD_STATIC=OFF
+    commands:
+      - apt-get update -qq
+      - apt-get install -y clang make cmake
+      - $CC --version
+      - mkdir build && cd build
+      - cmake $CMAKE_FLAGS ..
+      - make -j
+      - ctest --output-on-failure
 kind: pipeline
 name: amd64_clang_cmake_static
 
 
@@ -12,6 +12,8 @@ using namespace std;
 
 const padded_string EMPTY_ARRAY("[]", 2);
 
+#if SIMDJSON_EXCEPTIONS
+
 static void twitter_count(Stat
E864
e& state) {
   // Prints the number of results in twitter.json
   document doc = document::load(JSON_TEST_PATH);
@@ -22,17 +24,6 @@ static void twitter_count(State& state) {
 }
 BENCHMARK(twitter_count);
 
-static void error_code_twitter_count(State& state) noexcept {
-  // Prints the number of results in twitter.json
-  document doc = document::load(JSON_TEST_PATH);
-  for (auto _ : state) {
-    auto [value, error] = doc["search_metadata"]["count"];
-    if (error) { return; }
-    if (uint64_t(value) != 100) { return; }
-  }
-}
-BENCHMARK(error_code_twitter_count);
-
 static void iterator_twitter_count(State& state) {
   // Prints the number of results in twitter.json
   document doc = document::load(JSON_TEST_PATH);
@@ -65,6 +56,39 @@ static void twitter_default_profile(State& state) {
 }
 BENCHMARK(twitter_default_profile);
 
+static void twitter_image_sizes(State& state) {
+  // Count unique image sizes
+  document doc = document::load(JSON_TEST_PATH);
+  for (auto _ : state) {
+    set<tuple<uint64_t, uint64_t>> image_sizes;
+    for (document::object tweet : doc["statuses"].as_array()) {
+      auto [media, not_found] = tweet["entities"]["media"];
+      if (!not_found) {
+        for (document::object image : media.as_array()) {
+          for (auto [key, size] : image["sizes"].as_object()) {
+            image_sizes.insert({ size["w"], size["h"] });
+          }
+        }
+      }
+    }
+    if (image_sizes.size() != 15) { return; };
+  }
+}
+BENCHMARK(twitter_image_sizes);
+
+#endif // SIMDJSON_EXCEPTIONS
+
+static void error_code_twitter_count(State& state) noexcept {
+  // Prints the number of results in twitter.json
+  document doc = document::load(JSON_TEST_PATH);
+  for (auto _ : state) {
+    auto [value, error] = doc["search_metadata"]["count"].as_uint64_t();
+    if (error) { return; }
+    if (value != 100) { return; }
+  }
+}
+BENCHMARK(error_code_twitter_count);
+
 static void error_code_twitter_default_profile(State& state) noexcept {
   // Count unique users with a default profile.
   document doc = document::load(JSON_TEST_PATH);
@@ -127,26 +151,6 @@ static void iterator_twitter_default_profile(State& state) {
 }
 BENCHMARK(iterator_twitter_default_profile);
 
-static void twitter_image_sizes(State& state) {
-  // Count unique image sizes
-  document doc = document::load(JSON_TEST_PATH);
-  for (auto _ : state) {
-    set<tuple<uint64_t, uint64_t>> image_sizes;
-    for (document::object tweet : doc["statuses"].as_array()) {
-      auto [media, not_found] = tweet["entities"]["media"];
-      if (!not_found) {
-        for (document::object image : media.as_array()) {
-          for (auto [key, size] : image["sizes"].as_object()) {
-            image_sizes.insert({ size["w"], size["h"] });
-          }
-        }
-      }
-    }
-    if (image_sizes.size() != 15) { return; };
-  }
-}
-BENCHMARK(twitter_image_sizes);
-
 static void error_code_twitter_image_sizes(State& state) noexcept {
   // Count unique image sizes
   document doc = document::load(JSON_TEST_PATH);
 
@@ -304,8 +304,8 @@ struct benchmarker {
     // Run it once to get hot buffers
     if(hotbuffers) {
       auto result = parser.parse((const uint8_t *)json.data(), json.size());
-      if (result.error) {
-        exit_error(string("Failed to parse ") + filename + string(":") + error_message(result.error));
+      if (result.error()) {
+        exit_error(string("Failed to parse ") + filename + string(":") + error_message(result.error()));
       }
     }
 
 
@@ -67,7 +67,7 @@ simdjson_compute_stats(const simdjson::padded_string &p) {
 
 __attribute__((noinline)) bool
 simdjson_just_parse(const simdjson::padded_string &p) {
-  return simdjson::document::parse(p).error != simdjson::SUCCESS;
+  return simdjson::document::parse(p).error() != simdjson::SUCCESS;
 }
 
 void sajson_traverse(std::vector<int64_t> &answer, const sajson::value &node) {
 
@@ -82,7 +82,7 @@ int main (int argc, char *argv[]){
                 auto start = std::chrono::steady_clock::now();
                 count = 0;
                 for (auto result : parser.parse_many(p, 4000000)) {
-                    error = result.error;
+                    error = result.error();
                     count++;
                 }
                 auto end = std::chrono::steady_clock::now();
@@ -121,7 +121,7 @@ int main (int argc, char *argv[]){
             auto start = std::chrono::steady_clock::now();
             // TODO this includes allocation of the parser; is that intentional?
             for (auto result : parser.parse_many(p, 4000000)) {
-                error = result.error;
+                error = result.error();
             }
             auto end = std::chrono::steady_clock::now();
 
 
@@ -47,7 +47,10 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t *Data, size_t Size) {
 
   try {
     auto pj = simdjson::build_parsed_json(Data, Size);
-    simdjson::ParsedJson::Iterator pjh(pj);
+    if (!pj.is_valid()) {
+      throw 1;
+    }
+    simdjson::ParsedJson::Iterator pjh(pj.doc);
     if (pjh.is_ok()) {
       compute_dump(pjh);
     }