[trivial] Fixed -Wshadow warnings #10009

pull tjps wants to merge 1 commits into bitcoin:master from tjps:tjps_shadowing changing 3 files +23 −25
  1. tjps commented at 8:11 am on March 16, 2017: contributor
  2. [trivial] Fixed -Wshadow warnings 3df45c6748
  3. fanquake added the label Refactoring on Mar 16, 2017
  4. tjps renamed this:
    [trivial] Fixing -Wshadow warnings
    [trivial] Fixed -Wshadow warnings
    on Mar 16, 2017
  5. in src/test/checkqueue_tests.cpp: in 3df45c6748
     95@@ -96,7 +96,7 @@ struct MemoryCheck {
     96     };
     97     ~MemoryCheck(){
     98         fake_allocated_memory -= b;
     99-    
    100+
    


    practicalswift commented at 9:26 am on March 16, 2017:
    Intentional blank line? :-)
  6. in src/util.h: in 3df45c6748
    93@@ -94,9 +94,9 @@ template<typename... Args> std::string FormatStringFromLogArgs(const char *fmt,
    94 } while(0)
    95 
    96 template<typename... Args>
    97-bool error(const char* fmt, const Args&... args)
    98+bool error(const char* fmt, Args... args)
    


    laanwj commented at 10:16 am on March 16, 2017:
    Not sure what you’re doing here but I doubt it is required to “Fix Wshadow”.
  7. laanwj commented at 10:17 am on March 16, 2017: member
    Also may be duplicate of #9911.
  8. laanwj commented at 5:37 pm on March 16, 2017: member
    Closing in favor of #10010 which is cleaner and more targeted.
  9. laanwj closed this on Mar 16, 2017

  10. tjps deleted the branch on May 13, 2017
  11. DrahtBot 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: 2024-11-17 12:12 UTC

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