-
Notifications
You must be signed in to change notification settings - Fork 232
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
xml test fail:Uncaught (in promise) RuntimeError: memory access out of bounds #192
Comments
Is this the old version? iguana::xml has been refactored recently. Can you give it a try? |
First of all, "../frozen/string.h" should be "frozen/string.h", just pay attention to the path. Secondly, there is a conflicting redefinition between YAML、JSON and XML, and it has not been resolved yet.
…------------------ 原始邮件 ------------------
发件人: "Mr ***@***.***>;
发送时间: 2023年7月12日(星期三) 下午5:57
收件人: ***@***.***>;
抄送: ***@***.***>; ***@***.***>;
主题: Re: [qicosmos/iguana] xml test fail:Uncaught (in promise) RuntimeError: memory access out of bounds (Issue #192)
afer update to latest version ,got some other error:
[ 33%] Building CXX object CMakeFiles/CPPHX.dir/test/TestCase.cpp.o In file included from /home/rock/tools/CppHx/test/TestCase.cpp:43: In file included from /home/rock/tools/CppHx/Public/Json.h:4: In file included from /home/rock/tools/CppHx/Public/iguana/json_reader.hpp:5: In file included from /home/rock/tools/CppHx/Public/iguana/json_util.hpp:7: /home/rock/tools/CppHx/Public/iguana/reflection.hpp:20:10: error: '../frozen/string.h' file not found, did you mean 'frozen/string.h'? 20 | #include "../frozen/string.h" | ^~~~~~~~~~~~~~~~~~~~ | "frozen/string.h" In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:19:9: error: redefinition of 'char_t' 19 | concept char_t = std::same_as < std::decay_t<T>, | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:22:9: note: previous definition is here 22 | concept char_t = std::same_as < std::decay_t<T>, | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:25:9: error: redefinition of 'bool_t' 25 | concept bool_t = std::same_as < std::decay_t<T>, | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:28:9: note: previous definition is here 28 | concept bool_t = std::same_as < std::decay_t<T>, | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:29:9: error: redefinition of 'int_t' 29 | concept int_t = | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:32:9: note: previous definition is here 32 | concept int_t = | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:33:9: error: redefinition of 'float_t' 33 | concept float_t = std::floating_point<std::decay_t<T>>; | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:36:9: note: previous definition is here 36 | concept float_t = std::floating_point<std::decay_t<T>>; | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:36:9: error: redefinition of 'num_t' 36 | concept num_t = std::floating_point<std::decay_t<T>> || int_t<T>; | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:39:9: note: previous definition is here 39 | concept num_t = std::floating_point<std::decay_t<T>> || int_t<T>; | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:39:9: error: redefinition of 'enum_t' 39 | concept enum_t = std::is_enum_v<std::decay_t<T>>; | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:42:9: note: previous definition is here 42 | concept enum_t = std::is_enum_v<std::decay_t<T>>; | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:41:45: error: redefinition of 'is_basic_string_view' 41 | template <typename T> constexpr inline bool is_basic_string_view = false; | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:47:45: note: previous definition is here 47 | template <typename T> constexpr inline bool is_basic_string_view = false; | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:44:23: error: redefinition of 'is_basic_string_view<std::basic_string_view<T>>' 44 | constexpr inline bool is_basic_string_view<std::basic_string_view<T>> = true; | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:50:23: note: previous definition is here 50 | constexpr inline bool is_basic_string_view<std::basic_string_view<T>> = true; | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:47:9: error: redefinition of 'str_view_t' 47 | concept str_view_t = is_basic_string_view<std::remove_reference_t<T>>; | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:96:9: note: previous definition is here 96 | concept str_view_t = is_basic_string_view<std::remove_reference_t<T>>; | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:50:9: error: redefinition of concept 'str_t' with different template parameters or requirements 50 | concept str_t = | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:100:9: note: previous definition is here 100 | concept str_t = | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:57:9: error: redefinition of 'tuple_t' 57 | concept tuple_t = is_tuple<std::remove_cvref_t<T>>::value; | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:143:9: note: previous definition is here 143 | concept tuple_t = is_tuple<std::remove_cvref_t<T>>::value; | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:60:9: error: redefinition of 'sequence_container_t' 60 | concept sequence_container_t = | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:139:9: note: previous definition is here 139 | concept sequence_container_t = | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:64:9: error: redefinition of 'unique_ptr_t' 64 | concept unique_ptr_t = requires(Type ptr) { | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:149:9: note: previous definition is here 149 | concept unique_ptr_t = requires(Type ptr) { | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:79:9: error: redefinition of 'container' 79 | concept container = requires(Type container) { | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:69:9: note: previous definition is here 69 | concept container = requires(Type container) { | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:86:9: error: redefinition of 'map_container' 86 | concept map_container = container<Type> && requires(Type container) { | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:77:9: note: previous definition is here 77 | concept map_container = container<Type> && requires(Type container) { | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:95:9: error: redefinition of 'c_array' 95 | concept c_array = std::is_array_v<std::remove_cvref_t<T>> && | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:82:9: note: previous definition is here 82 | concept c_array = std::is_array_v<std::remove_cvref_t<T>> && | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:100:9: error: redefinition of 'array' 100 | concept array = requires(Type arr) { | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:87:9: note: previous definition is here 87 | concept array = requires(Type arr) { | ^ In file included from /home/rock/tools/CppHx/test/TestCase.cpp:47: In file included from /home/rock/tools/CppHx/Public/iguana/xml_reader.hpp:4: /home/rock/tools/CppHx/Public/iguana/xml_util.hpp:106:9: error: redefinition of 'tuple' 106 | concept tuple = !array<Type> && requires(Type tuple) { | ^ /home/rock/tools/CppHx/Public/iguana/json_util.hpp:105:9: note: previous definition is here 105 | concept tuple = !array<Type> && requires(Type tuple) { | ^ fatal error: too many errors emitted, stopping now [-ferror-limit=]
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: ***@***.***>
|
I am planing to resolve these conflicts while supporting 17, and if necessary, you can also resolve these conflicts yourself. If you want me to solve it, it's also my pleasure |
171930433
pushed a commit
to 171930433/iguana
that referenced
this issue
Jun 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
platform: emscripten wasm
iguana::from_xml(t1, ss.data());//will throw Uncaught (in promise) RuntimeError: memory access out of bounds
The text was updated successfully, but these errors were encountered: