mirror of
https://github.com/ceph/ceph
synced 2025-03-25 03:37:38 +00:00
os/ObjectStore.cc: prefer prefix ++operator for non-primitive types
Prefer prefix ++operator for non-primitive types like iterators for performance reasons. Prefix ++/-- operators avoid creating a temporary copy. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
This commit is contained in:
parent
555e717795
commit
a8e10d3d0a
@ -504,7 +504,7 @@ int ObjectStore::collection_list(coll_t c, vector<hobject_t>& o)
|
|||||||
int ret = collection_list(c, go);
|
int ret = collection_list(c, go);
|
||||||
if (ret == 0) {
|
if (ret == 0) {
|
||||||
o.reserve(go.size());
|
o.reserve(go.size());
|
||||||
for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; i++)
|
for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; ++i)
|
||||||
o.push_back(i->hobj);
|
o.push_back(i->hobj);
|
||||||
}
|
}
|
||||||
return ret;
|
return ret;
|
||||||
@ -520,7 +520,7 @@ int ObjectStore::collection_list_partial(coll_t c, hobject_t start,
|
|||||||
if (ret == 0) {
|
if (ret == 0) {
|
||||||
*next = gnext.hobj;
|
*next = gnext.hobj;
|
||||||
ls->reserve(go.size());
|
ls->reserve(go.size());
|
||||||
for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; i++)
|
for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; ++i)
|
||||||
ls->push_back(i->hobj);
|
ls->push_back(i->hobj);
|
||||||
}
|
}
|
||||||
return ret;
|
return ret;
|
||||||
@ -534,7 +534,7 @@ int ObjectStore::collection_list_range(coll_t c, hobject_t start, hobject_t end,
|
|||||||
int ret = collection_list_range(c, gstart, gend, seq, &go);
|
int ret = collection_list_range(c, gstart, gend, seq, &go);
|
||||||
if (ret == 0) {
|
if (ret == 0) {
|
||||||
ls->reserve(go.size());
|
ls->reserve(go.size());
|
||||||
for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; i++)
|
for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; ++i)
|
||||||
ls->push_back(i->hobj);
|
ls->push_back(i->hobj);
|
||||||
}
|
}
|
||||||
return ret;
|
return ret;
|
||||||
|
Loading…
Reference in New Issue
Block a user