mirco-optimize control-flow clauses in net.cpp #1388

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:micro-optimise_net changing 1 files +8 −8
  1. Diapolo commented at 4:57 PM on May 24, 2012: none
    • add a variable re-use instead of double-calling vNodes.size()

    Put simple bool checks before function calls.

  2. mirco-optimize control-flow clauses in net.cpp / add a variable re-use instead of double-calling vNodes.size() a1ed2010b7
  3. gavinandresen commented at 7:29 PM on May 24, 2012: contributor

    NACK. Please don't micro-optimize unless you've benchmarked and are fixing some performance bottleneck; otherwise you're just contributing to a code-review-denial-of-service-on-the-core-developers attack.

  4. gavinandresen closed this on May 24, 2012

  5. lateminer referenced this in commit 6e6938d915 on Jan 22, 2019
  6. lateminer referenced this in commit c80688e02a on May 6, 2020
  7. 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: 2026-04-21 18:16 UTC

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