Wshadow: Do not shadow argument with a local variable #8981

pull paveljanik wants to merge 1 commits into bitcoin:master from paveljanik:20161020_Wshadow_rpcdump changing 1 files +2 −2
  1. paveljanik commented at 10:35 AM on October 20, 2016: contributor

    Current master adds new -Wshadow warning:

    wallet/rpcdump.cpp:801:44: warning: declaration shadows a local variable [-Wshadow]
    

    Use better name for the local variable, matching its vector type.

  2. Do not shadow local variable ff6639b0f5
  3. fanquake added the label Refactoring on Oct 21, 2016
  4. fanquake cross-referenced this on Nov 7, 2016 from issue Enable -Wshadow by default by paveljanik
  5. MarcoFalke commented at 9:56 AM on November 7, 2016: member

    same-binaries-ACK ff6639b

  6. MarcoFalke added this to the milestone 0.14.0 on Nov 7, 2016
  7. MarcoFalke assigned laanwj on Nov 7, 2016
  8. laanwj merged this on Nov 7, 2016
  9. laanwj closed this on Nov 7, 2016

  10. laanwj referenced this in commit 0b2322b144 on Nov 7, 2016
  11. codablock referenced this in commit 293333481e on Jan 15, 2018
  12. andvgal referenced this in commit b568159f35 on Jan 6, 2019
  13. CryptoCentric referenced this in commit 109d2a153c on Feb 24, 2019
  14. bitcoin locked this on Sep 8, 2021

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-19 06:55 UTC