crimson/osd: drop redundant trailing co_return in pg_advance_map

check_for_splits() and split_pg() both ended with a bare co_return
that the compiler inserts implicitly for a coroutine returning
seastar::future<>. Remove the redundant statements.

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

diff --git a/src/crimson/osd/osd_operations/pg_advance_map.cc b/src/crimson/osd/osd_operations/pg_advance_map.cc
index ee809cc53a93348670868a90a733271d90b226c8..e3d8b66487b58d14f157f48ea6d4ca1c680f76f2 100644 (file)
--- a/src/crimson/osd/osd_operations/pg_advance_map.cc
+++ b/src/crimson/osd/osd_operations/pg_advance_map.cc
@@ -151,7 +151,6 @@ seastar::future<> PGAdvanceMap::check_for_splits(
       co_await split_pg(children, next_map);
     }
   }
-  co_return;
 }
 
 
@@ -201,7 +200,6 @@ seastar::future<> PGAdvanceMap::split_pg(
   }
 
   split_stats(split_pgs, children_pgids);
-  co_return;
 }
 
 seastar::future<> PGAdvanceMap::handle_split_pg_creation(