Merge #17971: refactor: Remove redundant conditional

e80317b refactor: Remove redundant conditional (Bushstar)

Pull request description:

  Conditional check against fMaster is now redundant as it is already checked as true. This originally made sense as the outer conditional was:

  https://github.com/bitcoin/bitcoin/blob/f9cae832e6f56c6abe89b3bf05d1f176c2a7c913/src/checkqueue.h#L86

  Removal of fQuit happened in the commit below.

  bitcoin/bitcoin@30ded3e#diff-88316c9aa9514c038c9304297e672da5

ACKs for top commit:
  theStack:
    ACK bitcoin/bitcoin@e80317b
  hebasto:
    ACK e80317b, I have reviewed the code, and it looks OK, I agree it can be merged.
  promag:
    ACK e80317b.
  emilengler:
    re-ACK e80317b
  practicalswift:
    ACK e80317b
  Empact:
    ACK bitcoin/bitcoin@e80317b

Tree-SHA512: 136ea1d02e3d65100a8758730617ccede7864e08e8404e42e65d45d4bf95a3bfea2ab9895c6e8833abd654557d3efbba02b25297a2a5eefc36a11e97bbe9134f

Author: fanquake

src/checkqueue.h

@@ -90,8 +90,7 @@ class CCheckQueue
                         nTotal--;
                         bool fRet = fAllOk;
                         // reset the status for new work later
-                        if (fMaster)
-                            fAllOk = true;
+                        fAllOk = true;
                         // return the current status
                         return fRet;
                     }