tree 189043cc790f1e6604ee3e67ced23c045565b6ed
parent 0632c1198715079880735e7aab761a94fbf115a2
author Guenter Roeck <groeck@chromium.org> 1589394804 -0700
committer Commit Bot <commit-bot@chromium.org> 1589400735 +0000

findmissing: gerrit_interface: Remove self-assignment

Python's "a = b if <condition> else c" is a bit odd if used for a
conditional assignment. We get

    reviewers = cl_reviewers if cl_reviewers else reviewers

which is not easy to read. Replace it with

    if cl_reviewers:
        reviewers = cl_reviewers

to improve readability.

BUG=None
TEST=Run on GCE

Change-Id: Ia82cddb3e63e5daa4d10d2e7ef6ea9c7b8691142
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/dev-util/+/2199609
Reviewed-by: Curtis Malainey <cujomalainey@chromium.org>
Commit-Queue: Guenter Roeck <groeck@chromium.org>
Tested-by: Guenter Roeck <groeck@chromium.org>