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

[Trivial] Remove nonsense #undef foreach #9732

Merged
merged 1 commit into from
Feb 28, 2017

Conversation

jnewbery
Copy link
Contributor

@jnewbery jnewbery commented Feb 9, 2017

This PR removes #undef foreach from txmempool.h. That preprocessor
directive was added in 34628a1
(#6654) but there doesn't appear to be a
#define foreach anywhere in the codebase. The #undef directive doesn't
actually do anything.

This commit removes a preprocessor #undef for a macro that was never
defined. Should make no difference to compiled code.
@maflcko
Copy link
Member

maflcko commented Feb 9, 2017

gives same binaries for me ACK 467df39

@jtimon
Copy link
Contributor

jtimon commented Feb 9, 2017

utACK 467df39

@sipa
Copy link
Member

sipa commented Feb 10, 2017

Obvious ACK.

This was added in #6654, without explanation.

@paveljanik
Copy link
Contributor

ACK 467df39

@laanwj
Copy link
Member

laanwj commented Feb 10, 2017

but there doesn't appear to be a #define foreach anywhere in the codebase.

IIRC Qt defines this, and there was a conflict with that and some other dependency. But if it's no longer necessary that's great.

@fanquake
Copy link
Member

utACK 467df39

@laanwj laanwj merged commit 467df39 into bitcoin:master Feb 28, 2017
laanwj added a commit that referenced this pull request Feb 28, 2017
467df39 Remove nonsense #undef foreach (John Newbery)

Tree-SHA512: b8c3c6654eb8a9e5008a1923bee46f7ef9f8216c8f4f1b4662d82c26410ca8d8afe41e24e48a3e2e8b2a25c81c33795a4073dabfe357caf0aa820519d4fffc71
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jan 5, 2019
467df39 Remove nonsense #undef foreach (John Newbery)

Tree-SHA512: b8c3c6654eb8a9e5008a1923bee46f7ef9f8216c8f4f1b4662d82c26410ca8d8afe41e24e48a3e2e8b2a25c81c33795a4073dabfe357caf0aa820519d4fffc71
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jan 23, 2019
467df39 Remove nonsense #undef foreach (John Newbery)

Tree-SHA512: b8c3c6654eb8a9e5008a1923bee46f7ef9f8216c8f4f1b4662d82c26410ca8d8afe41e24e48a3e2e8b2a25c81c33795a4073dabfe357caf0aa820519d4fffc71
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jan 25, 2019
467df39 Remove nonsense #undef foreach (John Newbery)

Tree-SHA512: b8c3c6654eb8a9e5008a1923bee46f7ef9f8216c8f4f1b4662d82c26410ca8d8afe41e24e48a3e2e8b2a25c81c33795a4073dabfe357caf0aa820519d4fffc71
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants