Skip to content
Snippets Groups Projects
Commit 364d792b authored by Mark de Wever's avatar Mark de Wever
Browse files

Merge branch 'avoid-temporaries' into 'master'

Avoid tempories in a range-based for loop

See merge request !814
parents 3c11a04b 3fea9d2a
Branches
Tags
1 merge request!814Avoid tempories in a range-based for loop
Pipeline #38864 passed
......@@ -13,7 +13,7 @@ namespace common {
std::ostream& operator<<(std::ostream& os, const ParameterRecord& record) {
bool first = true;
os << '{';
for (const std::pair<std::string, ParameterValue>& entry : record) {
for (const std::pair<const std::string, ParameterValue>& entry : record) {
if (first) {
first = false;
} else {
......
......@@ -71,7 +71,7 @@ blob::BlobOStream& operator<<(blob::BlobOStream& stream,
const ParameterSet& set) {
stream.putStart("ParameterSet", 1);
stream << static_cast<std::uint32_t>(set.size());
for (const std::pair<std::string, ParameterValue>& entry : set) {
for (const std::pair<const std::string, ParameterValue>& entry : set) {
stream << entry.first << entry.second.get();
}
stream.putEnd();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment