Remove redundant should_process check.
AFAICT - the code never reaches this point because we already check should process before adding it to the queue. http://shortn/_PWFae17qw4 - this might be necessary if the value of should_process changes between adding it to the queue and running it. But I don't see that happening anywhere. should_process is only ever set and modified here: http://shortn/_eirozAoIm4 which happens during in the parent's run() before the deps get added to the queue. The should_process check that happens at the beginning of run() was added here: https://codereview.chromium.org/3124017/diff/2001/gclient.py The should_process check before a deps gets enqueued was added later here: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/559150/ where several should_process checks were added and I think the author just didn't realize that, while the other methods might need them, run() already had a should_process check. Alternatively, we could remove the should_process check before enqueue-ing. But that would result in enqueuing things that we currently don't enqueue, even if we ultimately don't do anything with it. Change-Id: Ib3be61310c09a681988baa75fd562c7596e9e044 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3721600Reviewed-by:Josip Sokcevic <sokcevic@google.com> Commit-Queue: Joanna Wang <jojwang@chromium.org>
Showing
Please
register
or
sign in
to comment