From 79c7ccb903343b52640c6346050d444ba14fb923 Mon Sep 17 00:00:00 2001 From: Ning Yao Date: Tue, 15 Dec 2015 23:38:53 +0800 Subject: [PATCH] osd: combine map.count() and map.find() into one searching operation Signed-off-by: Ning Yao --- src/osd/OSDMap.h | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/osd/OSDMap.h b/src/osd/OSDMap.h index e929b72ce1743..e8e39af08d01b 100644 --- a/src/osd/OSDMap.h +++ b/src/osd/OSDMap.h @@ -751,14 +751,16 @@ public: return p->second.get_size(); } int get_pg_type(pg_t pg) const { - assert(pools.count(pg.pool())); - return pools.find(pg.pool())->second.get_type(); + map::const_iterator p = pools.find(pg.pool()); + assert(p != pools.end()); + return p->second.get_type(); } pg_t raw_pg_to_pg(pg_t pg) const { - assert(pools.count(pg.pool())); - return pools.find(pg.pool())->second.raw_pg_to_pg(pg); + map::const_iterator p = pools.find(pg.pool()); + assert(p != pools.end()); + return p->second.raw_pg_to_pg(pg); } // pg -> acting primary osd -- 2.39.5