Add a few std::moves to fix Coverity warnings

This commit is contained in:
Tim Wojtulewicz 2025-05-21 12:46:34 -07:00
parent ff7c908460
commit e4ab1a3b1d
3 changed files with 3 additions and 3 deletions

View file

@ -113,7 +113,7 @@ void DebugLogger::EnableStreams(const char* s) {
if ( util::starts_with(ltok, "plugin-") ) {
// Cannot verify this at this time, plugins may not
// have been loaded.
enabled_streams.insert(ltok);
enabled_streams.insert(std::move(ltok));
goto next;
}

View file

@ -866,7 +866,7 @@ static std::set<EventGroupPtr> get_func_groups(const std::vector<AttrPtr>& attrs
}
auto group = event_registry->RegisterGroup(EventGroupKind::Attribute, v->AsStringVal()->ToStdStringView());
groups.insert(group);
groups.insert(std::move(group));
}
return groups;

View file

@ -811,7 +811,7 @@ bool Manager::TraverseRecord(Stream* stream, Filter* filter, RecordType* rt, Tab
}
// Alright, we want this field.
filter->indices.push_back(new_indices);
filter->indices.push_back(std::move(new_indices));
// Static cast this to void* to avoid a clang-tidy warning about converting from the
// double-pointer to void*