-
-
Notifications
You must be signed in to change notification settings - Fork 252
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
jsoncons trait longer compatible with latest release #371
Comments
I would strongly encourage you to update the trait definition to be compatible with the newest release. If younise that library you'll be best suited for it. It does seem the range needs set since there's a breaking change |
I'm on master of jwt-cpp, and on the latest release of jsoncons 1.2.0. The Version 0.7.0 in the issue is forced by template, but I use the commit 482f4f1c050fb59db7f824636d8ab70ef8525544 Do you want me to do any particular tests? |
Well the logs you posted show it does not compile, looks like the internals of jsocons have changed. Maybe update the trait implementation to work with the new version? |
First reported in #366 |
What happened?
A bug happened!
With master version of jwt-cpp and jsoncons at 1.2.0
traits.h(93,28): error C2039: 'object_value': non è un membro di 'jsoncons::basic_json<char,jsoncons::sorted_policy,std::allocator<char>>'
traits.h(98,27): error C2039: 'array_value': non è un membro di 'jsoncons::basic_json<char,jsoncons::sorted_policy,std::allocator<char>>'
P.S.
updated
static_assert
fromstatic_assert(jsoncons::version().minor >= 167, "A higher version of jsoncons is required!");
to
static_assert(jsoncons::version().minor >= 167 || (jsoncons::version().major > 0), "A higher version of jsoncons is required!");
How To Reproduce?
Version
0.7.0
What OS are you seeing the problem on?
Windows
What compiler are you seeing the problem on?
MSVC
Relevant log output
Code of Conduct
The text was updated successfully, but these errors were encountered: