From 1e60fcfa67d0f57df6386c097390f5c4e115a9c0 Mon Sep 17 00:00:00 2001 From: Danny Al-Gaaf Date: Thu, 14 Mar 2013 19:21:45 +0100 Subject: [PATCH] mds/CDir.cc: prefer prefix ++operator for iterators Signed-off-by: Danny Al-Gaaf --- src/mds/CDir.cc | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/mds/CDir.cc b/src/mds/CDir.cc index a1ed05cea263d..231630e08aff6 100644 --- a/src/mds/CDir.cc +++ b/src/mds/CDir.cc @@ -644,7 +644,7 @@ void CDir::remove_null_dentries() { CDir::map_t::iterator p = items.begin(); while (p != items.end()) { CDentry *dn = p->second; - p++; + ++p; if (dn->get_linkage()->is_null() && !dn->is_projected()) remove_dentry(dn); } @@ -687,7 +687,7 @@ void CDir::purge_stale_snap_data(const set& snaps) CDir::map_t::iterator p = items.begin(); while (p != items.end()) { CDentry *dn = p->second; - p++; + ++p; if (dn->last == CEPH_NOSNAP) continue; @@ -931,7 +931,7 @@ void CDir::merge(list& subs, list& waiters, bool replay) int stale_rstat = 0; int stale_fragstat = 0; - for (list::iterator p = subs.begin(); p != subs.end(); p++) { + for (list::iterator p = subs.begin(); p != subs.end(); ++p) { CDir *dir = *p; dout(10) << " subfrag " << dir->get_frag() << " " << *dir << dendl; assert(!dir->is_auth() || dir->is_complete() || replay); @@ -1744,7 +1744,7 @@ CDir::map_t::iterator CDir::_commit_full(ObjectOperation& m, const set map_t::iterator p = items.begin(); while (p != items.end() && bl.length() < max_write_size) { CDentry *dn = p->second; - p++; + ++p; if (dn->linkage.is_null()) continue; // skip negative entries @@ -2055,7 +2055,7 @@ void CDir::_committed(version_t v, version_t lrv) for (map_t::iterator it = items.begin(); it != items.end(); ) { CDentry *dn = it->second; - it++; + ++it; // inode? if (dn->linkage.is_primary()) { @@ -2400,7 +2400,7 @@ void CDir::verify_fragstat() for (map_t::iterator it = items.begin(); it != items.end(); - it++) { + ++it) { CDentry *dn = it->second; if (dn->is_null()) continue; -- 2.39.5