tidy: Fix clang-tidy errors in src/leveldb #25715

pull shaavan wants to merge 3 commits into bitcoin:master from shaavan:220726-fix_clang_warnings changing 9 files +39 −54
  1. shaavan commented at 5:53 PM on July 26, 2022: contributor

    This PR fixes all the errors listed when running the clang-tidy operation as documented in the https://github.com/bitcoin/bitcoin/blob/master/doc/developer-notes.md#coding-style-c-named-arguments

    Steps:

    ./autogen.sh && ./configure CC=clang CXX=clang++
    make clean && bear -- make -j $(nproc)
    ( cd ./src/ && run-clang-tidy  -j $(nproc) )
    
  2. tidy: fix modernize-use-nullptr warning 837de6470a
  3. tidy: fix modernize-use-default-member-init 3d7bc58dab
  4. tidy: Fix modernize-use-default-member-init, variable might be redundant error 4d5bb06fd8
  5. fanquake commented at 8:49 PM on July 26, 2022: member

    You cannot modify the src/leveldb code in this repository. If you want to change it, you'll need to open a PR in our bitcoin-core/leveldb subtree, and the changes will be pulled here next time we update the subtree.

  6. fanquake closed this on Jul 26, 2022

  7. bitcoin locked this on Jul 31, 2023
Contributors

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-02 06:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me