From 25865863f8e741af73817f431aecf8e94bde697c Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Wed, 13 May 2026 12:32:33 +0800 Subject: [PATCH] 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 --- src/crimson/osd/osd_operations/pg_advance_map.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/crimson/osd/osd_operations/pg_advance_map.cc b/src/crimson/osd/osd_operations/pg_advance_map.cc index ee809cc53a93..e3d8b66487b5 100644 --- 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( -- 2.47.3