Replace some insert/push_back expr with emplace/emplace_back

This commit is contained in:
Yonas Habteab 2022-08-26 16:30:39 +02:00
parent 52b1b7ead3
commit d8445ca41a
12 changed files with 17 additions and 17 deletions

View File

@ -131,7 +131,7 @@ static Array::Ptr ArrayMap(const Function::Ptr& function)
ObjectLock olock(self);
for (const Value& item : self) {
result.push_back(function->Invoke({ item }));
result.emplace_back(function->Invoke({ item }));
}
return new Array(std::move(result));

View File

@ -262,7 +262,7 @@ Object::Ptr Array::Clone() const
ObjectLock olock(this);
for (const Value& val : m_Data) {
arr.push_back(val.Clone());
arr.emplace_back(val.Clone());
}
return new Array(std::move(arr));

View File

@ -63,7 +63,7 @@ static Array::Ptr NamespaceValues()
ArrayData values;
ObjectLock olock(self);
for (const Namespace::Pair& kv : self) {
values.push_back(kv.second.Val);
values.emplace_back(kv.second.Val);
}
return new Array(std::move(values));
}

View File

@ -455,7 +455,7 @@ Array::Ptr ScriptUtils::GetTemplates(const Type::Ptr& type)
for (const ConfigItem::Ptr& item : ConfigItem::GetItems(type)) {
if (item->IsAbstract())
result.push_back(GetTargetForTemplate(item));
result.emplace_back(GetTargetForTemplate(item));
}
return new Array(std::move(result));

View File

@ -41,7 +41,7 @@ std::vector<String> icinga::GetBashCompletionSuggestions(const String& type, con
if (lstat(path.CStr(), &statbuf) >= 0) {
if (S_ISDIR(statbuf.st_mode)) {
result.clear(),
result.push_back(path + "/");
result.emplace_back(path + "/");
}
}
}

View File

@ -634,7 +634,7 @@ int DaemonCommand::Run(const po::variables_map& vm, const std::vector<std::strin
else if (!vm.count("no-config")) {
/* The implicit string assignment is needed for Windows builds. */
String configDir = Configuration::ConfigDir;
configs.push_back(configDir + "/icinga2.conf");
configs.emplace_back(configDir + "/icinga2.conf");
}
if (vm.count("dump-objects")) {

View File

@ -422,7 +422,7 @@ bool ConfigItem::CommitNewItems(const ActivationContext::Ptr& context, WorkQueue
for (const ConfigItem::Ptr& item : m_UnnamedItems) {
if (item->m_ActivationContext != context) {
newUnnamedItems.push_back(item);
newUnnamedItems.emplace_back(item);
continue;
}
@ -456,7 +456,7 @@ bool ConfigItem::CommitNewItems(const ActivationContext::Ptr& context, WorkQueue
for (const Type::Ptr& type : Type::GetAllTypes()) {
if (ConfigObject::TypeInstance->IsAssignableFrom(type))
types.insert(type);
types.emplace(type);
}
while (types.size() != completed_types.size()) {
@ -507,7 +507,7 @@ bool ConfigItem::CommitNewItems(const ActivationContext::Ptr& context, WorkQueue
itemsCount += committed_items;
completed_types.insert(type);
completed_types.emplace(type);
#ifdef I2_DEBUG
if (committed_items > 0)
@ -571,7 +571,7 @@ bool ConfigItem::CommitNewItems(const ActivationContext::Ptr& context, WorkQueue
{
std::unique_lock<std::mutex> lock(item->m_Mutex);
item->m_IgnoredItems.push_back(item->m_DebugInfo.Path);
item->m_IgnoredItems.emplace_back(item->m_DebugInfo.Path);
}
}
});
@ -580,7 +580,7 @@ bool ConfigItem::CommitNewItems(const ActivationContext::Ptr& context, WorkQueue
}
}
completed_types.insert(type);
completed_types.emplace(type);
#ifdef I2_DEBUG
if (notified_items > 0)

View File

@ -45,7 +45,7 @@ Value MacroProcessor::ResolveMacros(const Value& str, const ResolverList& resolv
EscapeCallback(), resolvedMacros, useResolvedMacros, recursionLevel + 1);
if (value.IsObjectType<Array>())
resultArr.push_back(Utility::Join(value, ';'));
resultArr.emplace_back(Utility::Join(value, ';'));
else
resultArr.push_back(value);
}

View File

@ -41,7 +41,7 @@ bool DeleteObjectHandler::HandleRequest(
}
QueryDescription qd;
qd.Types.insert(type->GetName());
qd.Types.emplace(type->GetName());
qd.Permission = "objects/delete/" + type->GetName();
params->Set("type", type->GetName());

View File

@ -39,7 +39,7 @@ bool ModifyObjectHandler::HandleRequest(
}
QueryDescription qd;
qd.Types.insert(type->GetName());
qd.Types.emplace(type->GetName());
qd.Permission = "objects/modify/" + type->GetName();
params->Set("type", type->GetName());

View File

@ -115,7 +115,7 @@ bool ObjectQueryHandler::HandleRequest(
}
QueryDescription qd;
qd.Types.insert(type->GetName());
qd.Types.emplace(type->GetName());
qd.Permission = "objects/query/" + type->GetName();
Array::Ptr uattrs, ujoins, umetas;
@ -174,7 +174,7 @@ bool ObjectQueryHandler::HandleRequest(
if (ujoins) {
ObjectLock olock(ujoins);
for (const String& ujoin : ujoins) {
userJoinAttrs.insert(ujoin.SubStr(0, ujoin.FindFirstOf(".")));
userJoinAttrs.emplace(ujoin.SubStr(0, ujoin.FindFirstOf(".")));
}
}

View File

@ -102,7 +102,7 @@ bool TemplateQueryHandler::HandleRequest(
}
QueryDescription qd;
qd.Types.insert(type->GetName());
qd.Types.emplace(type->GetName());
qd.Permission = "templates/query/" + type->GetName();
qd.Provider = new TemplateTargetProvider();