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.

Sign up for GitHub

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

Jump to bottom

Remove whitespace after operator"" #39

Merged

davidchall merged 2 commits into davidchall:master from MichaelChirico:patch-1

Mar 13, 2025

Contributor

MichaelChirico commented Mar 12, 2025

The space after "" is deprecated since it creates ambiguity with reserved _-initial names per:

https://en.cppreference.com/w/cpp/language/user_literal

This might start throwing compiler warnings under clang++-20.

MichaelChirico and others added 2 commits

March 12, 2025 09:19


          Remove whitespace after operator""

dbc6c07


          Fix deprecation warning from nlohmann/json.hpp

975fbcc

Owner

davidchall commented Mar 13, 2025

Thanks a lot for the PR, @MichaelChirico! FYI: I found a couple of other instances of this issue and pushed a commit to your branch.

davidchall merged commit 985c220 into davidchall:master

11 of 12 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet