Skip to content
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

Update NTTP detection macros #273

Closed
technic opened this issue May 4, 2022 · 2 comments
Closed

Update NTTP detection macros #273

technic opened this issue May 4, 2022 · 2 comments

Comments

@technic
Copy link

technic commented May 4, 2022

Use __cpp_nontype_template_args instead of __cpp_nontype_template_parameter_class

see also here foonathan/lexy#15

beached added a commit that referenced this issue May 4, 2022
* #273 Using __cpp_nontype_template_args in addition to
  __cpp_nontype_template_parameter_class(for older compilers) to allow
  for CNTTP member names.  Created a detection macro
  DAW_JSON_CNTTP_JSON_NAME that can be used elsewhere to simplify the
  process of knowing if it is enabled
@beached
Copy link
Owner

beached commented May 4, 2022

Should be fixed via b90116a for release. Just waiting on CI to confirm no mistakes/typos

beached added a commit that referenced this issue May 4, 2022
* #273 Using __cpp_nontype_template_args in addition to __cpp_nontype_template_parameter_class(for older compilers) to allow for CNTTP member names.  Created a detection macro DAW_JSON_CNTTP_JSON_NAME that can be used elsewhere to simplify the process of knowing if it is enabled
@beached
Copy link
Owner

beached commented May 4, 2022

Also, fixed in v3 branch via 75957ac

@beached beached closed this as completed May 4, 2022
beached added a commit that referenced this issue May 26, 2022
* Fixed bug in documentation of variant

* version bump

* Updated Release to use new string_view in header libraries

* Bugfix: Disconnected fetchcontent was enabled in release.

Bugfix: Disconnected fetchcontent was enabled in release.  Should only
be in debug

* Added status to non-error cmake messages #269

* Updated release of header_libraries and added missing headers

* Targetting header_libraries v2.13.1

* changed tag on header_libraries to v2.13.2

* Updated to header_libraries 2.14.0

* Updated to newer header_libraries 2.14.2

* Updated to header_libraries 2.14.3

* Updated to header_libraries 2.14.4

* Updated to header_libraries 2.14.5

* Updated to header_libraries 2.15.0

* Updated to header_libraries 2.15.1

* Added automated doxygen

* Missed doxygen.yaml in prev commit

* removed cookbook as main page in doxygen

* updated doxyfile

* updated doxygen style

* missing extra file in doxyfile

* Added doxygen theme

* Added cookbook to doxydocs

* doxyfile update

* doxyfile update

* doxyfile update

* doxyfile update

* updated cookbook docs with absolute links

* doxyfile update

* doxyfile update

* doxyfile update

* doxyfile update

* Added mainpage.md for doxygen

* Added logo to doxydocs

* Updated to latest header_libraries, fixed compiler warning

* Fixed updated header_libraries that fixes compiler warnings

* Updated to upstream header_libraries

* version bump

* Updated dependencies

* version bump

* fixed build warnings

* added target guards in extern

* Fixed bug in json_value_state and added example for variants based on
other properties of the submember

* BUGFIX: #273 Use __cpp_nontype_template_args

* #273 Using __cpp_nontype_template_args in addition to
  __cpp_nontype_template_parameter_class(for older compilers) to allow
  for CNTTP member names.  Created a detection macro
  DAW_JSON_CNTTP_JSON_NAME that can be used elsewhere to simplify the
  process of knowing if it is enabled

* Corrected some perf issues when DAW_JSON_DONT_USE_EXCEPTIONS was set

std::uncaught_exceptions was showing how in some profiling.  It
shouldn't be needed if there are no exceptions.  This can result in
termination if multiple exceptions are throwing in some cases.

* Updated to latest header_libraries

* Updated to use newer header_libraries to fix -fno-exception issue

* Updated dependency version

* Version 3 Merge to Release Branch (#277)

* type deduction in mappings.  This allows integration with Reflection and reflection like libraries(e.g. Boost.Describe)
* Pretty printing/output formatting options
* json_nullable/readable trait to separate nullability from mappers
* json_raw/json_custom for things that don't fit
* Json Schema generation via to_json_schema
* Minimized symbol names
* Using a single Options value for member mappings, serialization options, and parse options
* Using better testing in CI
* added `json_type_alias` mapping type to allow classes to be parsed as other types e.g json_string/json_number/...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants