Fix incorrect handling of scalar documents with trailing content (#92)

* Remove use of document_reference type

It is not suitable for use here since the behaviour deviates from the
regular document type when it comes to trailing content.

* Add tests for scalar value with trailing content

* Move invalid test case to invalid sub folder

Author: tobil4sk

jsonexamples/invalid/bool_trailing.json

@@ -0,0 +1 @@
+true trailing

spec/compile_spec.lua

@@ -105,3 +105,21 @@ if tonumber(major) >= 5 and tonumber(minor) >= 3 then
         end)
     end)
 end
+
+local invalid_files = {
+    "bool_trailing.json"
+}
+
+describe("Make sure invalid files are not accepted", function()
+    for _, file in ipairs(invalid_files) do
+        it("should fail to parse: " .. file, function()
+            local fileContents = loadFile("jsonexamples/invalid/" .. file)
+            local cjsonValue, cjsonError = pcall(function() cjson.decode(fileContents) end)
+            local simdjsonValue, simdjsonError = pcall(function() simdjson.parse(fileContents) end)
+            assert.is.False(cjsonValue)
+            assert.is.False(simdjsonValue)
+            assert(cjsonError)
+            assert(simdjsonError)
+        end)
+    end
+end)

src/luasimdjson.cpp

@@ -43,8 +43,8 @@ ondemand::parser ondemand_parser;
 simdjson::padded_string jsonbuffer;
 
 template<typename T>
-void convert_ondemand_element_to_table(lua_State *L, T element) {
-  static_assert(std::is_base_of<ondemand::document_reference, T>::value || std::is_base_of<ondemand::value, T>::value, "type parameter must be document_reference or value");
+void convert_ondemand_element_to_table(lua_State *L, T& element) {
+  static_assert(std::is_base_of<ondemand::document, T>::value || std::is_base_of<ondemand::value, T>::value, "type parameter must be document or value");
 
   switch (element.type()) {
 
@@ -168,7 +168,7 @@ static int parse(lua_State *L)
     try {
         // makes a padded_string_view for a bit of quickness!
         doc = ondemand_parser.iterate(get_padded_string_view(json_str, json_str_len, jsonbuffer));
-        convert_ondemand_element_to_table(L, ondemand::document_reference(doc));
+        convert_ondemand_element_to_table(L, doc);
     } catch (simdjson::simdjson_error &error) {
         luaL_error(L, error.what());
     }
@@ -186,7 +186,7 @@ static int parse_file(lua_State *L)
     try {
         json_string = padded_string::load(json_file);
         doc = ondemand_parser.iterate(json_string);
-        convert_ondemand_element_to_table(L, ondemand::document_reference(doc));
+        convert_ondemand_element_to_table(L, doc);
     } catch (simdjson::simdjson_error &error) {
         luaL_error(L, error.what());
     }