🔨 hid the test lambda #489

The is_contiguous lambda is only used in an assertion, so it can be removed if NDEBUG is defined.
nlohmann · Mar 11, 2017 · 9e232ec · 9e232ec
1 parent a801d76
commit 9e232ec
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/json.hpp b/src/json.hpp
@@ -6969,6 +6969,7 @@ class basic_json
     {
         // assertion to check that the iterator range is indeed contiguous,
         // see http://stackoverflow.com/a/35008842/266378 for more discussion
+#ifndef NDEBUG
         const auto is_contiguous = [&]()
         {
             bool test = true;
@@ -6979,6 +6980,7 @@ class basic_json
             }
             return test;
         };
+#endif
         assert(is_contiguous());
 
         // assertion to check that each element is 1 byte long

diff --git a/src/json.hpp.re2c b/src/json.hpp.re2c
@@ -6969,6 +6969,7 @@ class basic_json
     {
         // assertion to check that the iterator range is indeed contiguous,
         // see http://stackoverflow.com/a/35008842/266378 for more discussion
+#ifndef NDEBUG
         const auto is_contiguous = [&]()
         {
             bool test = true;
@@ -6979,6 +6980,7 @@ class basic_json
             }
             return test;
         };
+#endif
         assert(is_contiguous());
 
         // assertion to check that each element is 1 byte long