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

C++20 fixes #130

Merged
merged 1 commit into from
Oct 20, 2023
Merged

C++20 fixes #130

merged 1 commit into from
Oct 20, 2023

Conversation

ccotter
Copy link
Contributor

@ccotter ccotter commented Oct 18, 2023

Describe your changes
[A clear and concise description of the changes you have made.
](std::not1 is removed from C++20. Since this code is expected to be built on C++03 as well as C++11 and newer, this patch embeds an adhoc version of 'not1' instead of, e.g., using a lambda expression.)

Testing performed
I ran the internal DPKG build for bmqbrkr.

@ccotter ccotter requested a review from a team as a code owner October 18, 2023 04:48
@pniedzielski
Copy link
Collaborator

Hi @ccotter,

Thanks for your contribution! We require all our contributors to sign off on their commits acknowledging that they indeed have the right to convey their contribution to us under our license. Could you git commit --amend --signoff and force push your PR branch?

std::not1 is removed from C++20. Since this code is expected to be built
on C++03 as well as C++11 and newer, this patch embeds an adhoc version
of 'not1' instead of, e.g., using a lambda expression.

Signed-off-by: Chris Cotter <[email protected]>
@hallfox hallfox merged commit 708d49b into bloomberg:main Oct 20, 2023
6 checks passed
@ccotter ccotter deleted the cxx20 branch October 30, 2023 20:34
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

Successfully merging this pull request may close these issues.

3 participants