Remove superfluous "has" function

[?]
Nov 2, 2015, 1:29 PM
7CCLJLW7KARUC2ZVJPFALT6SKXEJV4LZE326MVH5TLUBBX7WY3LQC

Dependencies

  • [2] RND7XFNH getQueuedBuilds(): Periodically stop to handle priority bumps
  • [3] HUUZFPPK Fix race between the queue monitor and the builder threads
  • [4] HJOEIMLR Refactor
  • [5] 24BMQDZA Start of single-process hydra-queue-runner
  • [6] OG3Z3QGC Namespace cleanup
  • [7] T5BIOVJE Add support for tracking custom metrics
  • [8] MHVIT4JY Split hydra-queue-runner.cc more
  • [9] ENXUSMSV Make concurrency more robust

Change contents

  • replacement in src/hydra-queue-runner/hydra-queue-runner.cc at line 252
    [3.13628][3.2475:2513]()
    if (has(steps, step)) return;
    [3.13628]
    [3.2513]
    if (steps.count(step)) return;
  • replacement in src/hydra-queue-runner/queue-monitor.cc at line 92
    [2.419][3.22345:22390](),[3.22345][3.22345:22390]()
    if (has(*builds_, id)) continue;
    [2.419]
    [3.22390]
    if (builds_->count(id)) continue;
  • edit in src/hydra-queue-runner/state.hh at line 442
    [3.9202][3.33277:33379]()
    template <class C, class V>
    bool has(const C & c, const V & v)
    {
    return c.find(v) != c.end();
    }