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. shaavan cross-referenced this on Jul 27, 2022 from issue tidy: Fix clang-tidy errors in src/leveldb by shaavan
  8. 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-20 06:53 UTC