gui: use odb::Rect method to simplify return statement

AcKoucher · AcKoucher · commit cbea0602a232 · 2025-07-29T11:15:53.000-03:00
Signed-off-by: Arthur Koucher &lt;arthurkoucher@precisioninno.com&gt;
diff --git a/src/gui/src/dbDescriptors.cpp b/src/gui/src/dbDescriptors.cpp
@@ -4685,16 +4685,14 @@ bool DbScanListDescriptor::getBBox(std::any object, odb::Rect& bbox) const
   auto scan_list = getObject(object);
   bbox.mergeInit();
 
-  bool has_placed_insts = false;
   for (odb::dbScanInst* scan_inst : scan_list->getScanInsts()) {
     odb::dbInst* inst = scan_inst->getInst();
     if (inst->getPlacementStatus().isPlaced()) {
-      has_placed_insts = true;
       bbox.merge(inst->getBBox()->getBox());
     }
   }
 
-  return has_placed_insts;
+  return !bbox.isInverted();
 }
 
 void DbScanListDescriptor::highlight(std::any object, Painter& painter) const

Original file line number	Diff line number	Diff line change
`@@ -4685,16 +4685,14 @@ bool DbScanListDescriptor::getBBox(std::any object, odb::Rect& bbox) const`
`4685`	`4685`	`auto scan_list = getObject(object);`
`4686`	`4686`	`bbox.mergeInit();`
`4687`	`4687`
`4688`		`- bool has_placed_insts = false;`
`4689`	`4688`	`for (odb::dbScanInst* scan_inst : scan_list->getScanInsts()) {`
`4690`	`4689`	`odb::dbInst* inst = scan_inst->getInst();`
`4691`	`4690`	`if (inst->getPlacementStatus().isPlaced()) {`
`4692`		`- has_placed_insts = true;`
`4693`	`4691`	`bbox.merge(inst->getBBox()->getBox());`
`4694`	`4692`	`}`
`4695`	`4693`	`}`
`4696`	`4694`
`4697`		`- return has_placed_insts;`
	`4695`	`+ return !bbox.isInverted();`
`4698`	`4696`	`}`
`4699`	`4697`
`4700`	`4698`	`void DbScanListDescriptor::highlight(std::any object, Painter& painter) const`