diff mbox series

[1/3] cooker: remove some redundant control flow

Message ID 20250806145308.2052128-1-chris.laplante@agilent.com
State Accepted, archived
Commit c2eb4336fe10b1bf8bbc6291c32ca362840f39e1
Headers show
Series [1/3] cooker: remove some redundant control flow | expand

Commit Message

chris.laplante@agilent.com Aug. 6, 2025, 2:53 p.m. UTC 
From: Chris Laplante <chris.laplante@agilent.com>

Prior to commit aa84a900e ("cooker: Ensure delays are accurately transfered to
the idle loops from runqueue"), this was necessary. But now retval is returned
directly.

Signed-off-by: Chris Laplante <chris.laplante@agilent.com>
---
 lib/bb/cooker.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
diff mbox series

Patch

diff --git a/lib/bb/cooker.py b/lib/bb/cooker.py
index 9e4000b63..fb87368f1 100644
--- a/lib/bb/cooker.py
+++ b/lib/bb/cooker.py
@@ -1434,8 +1434,7 @@  You can also remove the BB_HASHSERVE_UPSTREAM setting, but this may result in si
                 if quietlog:
                     bb.runqueue.logger.setLevel(rqloglevel)
                 return bb.server.process.idleFinish(msg)
-            if retval is True:
-                return True
+
             return retval
 
         self.idleCallBackRegister(buildFileIdle, rq)
@@ -1504,8 +1503,6 @@  You can also remove the BB_HASHSERVE_UPSTREAM setting, but this may result in si
                     bb.event.disable_heartbeat()
                 return bb.server.process.idleFinish(msg)
 
-            if retval is True:
-                return True
             return retval
 
         self.reset_mtime_caches()