crimson: consolidate the return paths of get_segment_manager()

before this change, two branches both return `BlockSegmentManager`,
which is redundant. in this change, consolidate them so that the
`HAVE_ZNS` path becomes an early return. this improves readability.

Signed-off-by: Kefu Chai <k.chai@proxmox.com>

diff --git a/src/crimson/os/seastore/segment_manager.cc b/src/crimson/os/seastore/segment_manager.cc
index 90a95be0a4308eeecf6bb961e552ab30cbde713d..0672b84a1374e445394c8f19521fba5395487757 100644 (file)
--- a/src/crimson/os/seastore/segment_manager.cc
+++ b/src/crimson/os/seastore/segment_manager.cc
@@ -44,12 +44,9 @@ SegmentManager::get_segment_manager(
   INFO("Found {} zones.", nr_zones);
   if (nr_zones != 0) {
     co_return std::make_unique<segment_manager::zbd::ZBDSegmentManager>(device + "/block");
-  } else {
-    co_return std::make_unique<segment_manager::block::BlockSegmentManager>(device + "/block", dtype);
   }
-#else
-  co_return std::make_unique<segment_manager::block::BlockSegmentManager>(device + "/block", dtype);
 #endif
+  co_return std::make_unique<segment_manager::block::BlockSegmentManager>(device + "/block", dtype);
 }
 
 } // namespace crimson::os::seastore