GothenburgBitFactory/taskwarrior
1
0
Fork 0
mirror of https://github.com/GothenburgBitFactory/taskwarrior.git synced 2025-08-19 09:53:08 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

clang-tidy: use const references

Browse source 
Found with performance-for-range-copy

Signed-off-by: Rosen Penev <rosenp@gmail.com>
This commit is contained in:
Rosen Penev 2021-02-28 19:03:38 -08:00 • committed by Tomas Babej
parent e84930be8d
commit 15f0ab87e0
2 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

8
src/Task.cpp
View file

@ -200,7 +200,7 @@ bool Task::has (const std::string& name) const
std::vector <std::string> Task::all ()
{
std::vector <std::string> all;
for (auto i : data)
for (const auto& i : data)
all.push_back (i.first);
return all;
@ -836,7 +836,7 @@ std::string Task::composeF4 () const
std::string ff4 = "[";
bool first = true;
for (auto it : data)
for (const auto& it : data)
{
// Orphans have no type, treat as string.
std::string type = Task::attributes[it.first];
@ -931,7 +931,7 @@ std::string Task::composeJSON (bool decorate /*= false*/) const
out << "\"tags\":[";
int count = 0;
for (auto i : tags)
for (const auto& i : tags)
{
if (count++)
out << ',';
@ -970,7 +970,7 @@ std::string Task::composeJSON (bool decorate /*= false*/) const
out << "\"depends\":[";
int count = 0;
for (auto i : deps)
for (const auto& i : deps)
{
if (count++)
out << ',';

2
src/feedback.cpp
View file

@ -48,7 +48,7 @@ std::string taskIdentifiers (const std::vector <Task>& tasks)
{
std::vector <std::string> identifiers;
identifiers.reserve(tasks.size());
for (auto task: tasks)
for (const auto& task: tasks)
identifiers.push_back (task.identifier (true));
return join (", ", identifiers);