Skip to content

build: Add -Wundef #88

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
Merged
merged 1 commit into from
Jun 12, 2024
Merged

build: Add -Wundef #88

merged 1 commit into from
Jun 12, 2024

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Jun 10, 2024

This PR addresses bitcoin/bitcoin#29876 (comment):

Steps to reproduce an issue:

$ g++ -Wundef -Wno-unused-parameter -o src/fields/libminisketch_a-generic_4bytes.o -c src/fields/generic_4bytes.cpp
In file included from src/fields/generic_common_impl.h:12,
                 from src/fields/generic_4bytes.cpp:12:
src/fields/../int_utils.h:162:7: warning: "HAVE_CLZ" is not defined, evaluates to 0 [-Wundef]
  162 | #elif HAVE_CLZ
      |       ^~~~~~~~

@hebasto hebasto mentioned this pull request Jun 10, 2024
Merged
theuni
theuni approved these changes Jun 11, 2024
View reviewed changes
Copy link
Contributor

@theuni theuni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 978a3d8

@sipa
Copy link
Collaborator

sipa commented Jun 12, 2024

utACK 978a3d8

@sipa sipa merged commit fe65576 into bitcoin-core:master Jun 12, 2024
fanquake added a commit to fanquake/bitcoin that referenced this pull request Jun 12, 2024
@fanquake
Squashed 'src/minisketch/' changes from 3472e2f5ec..eb37a9b8e7
cb59af3 
eb37a9b8e7 Merge bitcoin-core/minisketch#87: Avoid copy in self-assign
fe6557642e Merge bitcoin-core/minisketch#88: build: Add `-Wundef`
8ea298bfa7 Avoid copy in self-assign
978a3d8869 build: Add `-Wundef`
3387044179 Merge bitcoin-core/minisketch#86: doc: fix typo in sketch_impl.h
15c2d13b60 doc: fix typo in sketch_impl.h
7be08b8a46 Merge bitcoin-core/minisketch#85: Fixes for integer precision loss
00fb4a4d83 Avoid or make integer precision conversion explicit
9d62a4d27c Avoid the need to cast/convert to size_t for vector operations
19e06cc7af Prevent overflows from large capacity/max_elements

git-subtree-dir: src/minisketch
git-subtree-split: eb37a9b8e79f9e49d73b96a49bf97a96d9eb676c
@fanquake fanquake mentioned this pull request Jun 12, 2024
Merged
fanquake added a commit to bitcoin/bitcoin that referenced this pull request Jun 13, 2024
@fanquake
Merge #30270: Update minisketch subtree to eb37a9b8e79f9e49d73b96a49b…
080a47c 
…f97a96d9eb676c

cb59af3 Squashed 'src/minisketch/' changes from 3472e2f5ec..eb37a9b8e7 (fanquake)

Pull request description:

  Includes bitcoin-core/minisketch#87 which is used in #30234.
  Includes bitcoin-core/minisketch#88 which is used in #29876.

ACKs for top commit:
  sipa:
    utACK 89464ad
  theuni:
    utACK 89464ad

Tree-SHA512: 838a8c60856bfdf714da7d5d97e31d458290849ba5007d5c5bb7abb83d413ada6b4c16e45b0e060ff892b5785e6b664be9b6a666d04f0a414b0e359d64d3ad44
@hebasto hebasto deleted the 240610-undef branch June 16, 2024 09:07
janus pushed a commit to BitgesellOfficial/bitgesell that referenced this pull request Jul 26, 2024
@fanquake @janus
Squashed 'src/minisketch/' changes from 3472e2f5ec..eb37a9b8e7
f4bfa21 
eb37a9b8e7 Merge bitcoin-core/minisketch#87: Avoid copy in self-assign
fe6557642e Merge bitcoin-core/minisketch#88: build: Add `-Wundef`
8ea298bfa7 Avoid copy in self-assign
978a3d8869 build: Add `-Wundef`
3387044179 Merge bitcoin-core/minisketch#86: doc: fix typo in sketch_impl.h
15c2d13b60 doc: fix typo in sketch_impl.h
7be08b8a46 Merge bitcoin-core/minisketch#85: Fixes for integer precision loss
00fb4a4d83 Avoid or make integer precision conversion explicit
9d62a4d27c Avoid the need to cast/convert to size_t for vector operations
19e06cc7af Prevent overflows from large capacity/max_elements

git-subtree-dir: src/minisketch
git-subtree-split: eb37a9b8e79f9e49d73b96a49bf97a96d9eb676c
vmta added a commit to umkoin/umkoin that referenced this pull request Oct 6, 2024
@vmta
Squashed 'src/minisketch/' changes from a223557ce..eb37a9b8e
21eb7a8 
eb37a9b8e Merge bitcoin-core/minisketch#87: Avoid copy in self-assign
fe6557642 Merge bitcoin-core/minisketch#88: build: Add `-Wundef`
8ea298bfa Avoid copy in self-assign
978a3d886 build: Add `-Wundef`
338704417 Merge bitcoin-core/minisketch#86: doc: fix typo in sketch_impl.h
15c2d13b6 doc: fix typo in sketch_impl.h
7be08b8a4 Merge bitcoin-core/minisketch#85: Fixes for integer precision loss
00fb4a4d8 Avoid or make integer precision conversion explicit
9d62a4d27 Avoid the need to cast/convert to size_t for vector operations
19e06cc7a Prevent overflows from large capacity/max_elements
3472e2f5e Merge bitcoin-core/minisketch#81: Avoid overflowing shift by special casing inverse of 1
653d8b2e2 Avoid overflowing shift by special casing inverse of 1
33b7c200b Merge bitcoin-core/minisketch#80: Add c++20 version of CountBits
4a48f31a3 Merge bitcoin-core/minisketch#83: ci: Fix "s390x (big-endian)" task
82b6488ac Add c++20 version of CountBits
0498084d3 ci: Fix "s390x (big-endian)" task
71709dca9 Merge bitcoin-core/minisketch#82: ci: Fix `x86_64-w64-mingw32` task
9e6127fa9 Merge bitcoin-core/minisketch#74: Avoid >> above type width in BitWriter
ed420bc17 ci: Fix `x86_64-w64-mingw32` task
fe1040f22 Drop -Wno-shift-count-overflow compile flag
154bcd43b Avoid >> above type width in BitWriter
67b87acdb Merge bitcoin-core/minisketch#78: ci: Update macOS image for CI
7de725041 ci: Update macOS image for CI
83d812ea9 Merge bitcoin-core/minisketch#73: ci: Use correct variable to designate C++ compiler
e051a7d69 ci: Install wine32 package for Windows tests
2d2c695d7 build: Drop unused `CC` variable
1810fcbd1 ci: Use correct variable to designate C++ compiler
022b95904 Merge bitcoin-core/minisketch#77: Add missing include
08443c489 Add missing include
a571ba20f Merge bitcoin-core/minisketch#68: Add missed `#include <string>`
b9a7f7e2b Merge bitcoin-core/minisketch#69: refactor: Drop unused `total` local variables
8a5af94ed Merge bitcoin-core/minisketch#70: build: Remove `-Qunused-arguments` workaround for clang + ccache
c36f1f03a Merge bitcoin-core/minisketch#72: Fix MSVC implementation of `CountBits()` function
0078bedda Ignore `HAVE_CLZ` macro when building with MSVC
1c772918c Fix MSVC implementation of `CountBits()` function
98f87c55f build: Remove `-Qunused-arguments` workaround for clang + ccache
11a1e25c8 refactor: Drop unused `total` local variables
ed6c8fcfd Add missed `#include <string>`
47f0a2d26 Merge bitcoin-core/minisketch#66: msvc: remove direct Bitcoin Core `compat.h` include
64f17584c msvc: remove Core compat.h include

git-subtree-dir: src/minisketch
git-subtree-split: eb37a9b8e79f9e49d73b96a49bf97a96d9eb676c
kwvg added a commit to kwvg/dash that referenced this pull request Oct 29, 2024
@kwvg
Squashed 'src/minisketch/' changes from 3472e2f5ec..eb37a9b8e7
ef10e83 
eb37a9b8e7 Merge bitcoin-core/minisketch#87: Avoid copy in self-assign
fe6557642e Merge bitcoin-core/minisketch#88: build: Add `-Wundef`
8ea298bfa7 Avoid copy in self-assign
978a3d8869 build: Add `-Wundef`
3387044179 Merge bitcoin-core/minisketch#86: doc: fix typo in sketch_impl.h
15c2d13b60 doc: fix typo in sketch_impl.h
7be08b8a46 Merge bitcoin-core/minisketch#85: Fixes for integer precision loss
00fb4a4d83 Avoid or make integer precision conversion explicit
9d62a4d27c Avoid the need to cast/convert to size_t for vector operations
19e06cc7af Prevent overflows from large capacity/max_elements

git-subtree-dir: src/minisketch
git-subtree-split: eb37a9b8e79f9e49d73b96a49bf97a96d9eb676c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@theuni theuni theuni approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hebasto @sipa @theuni