Skip to content

Commit

Permalink
Fix minor anti-pattern issues
Browse files Browse the repository at this point in the history
  • Loading branch information
PabloNicolla committed Apr 26, 2024
1 parent e2290ea commit ac616bd
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Directory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ namespace fs
size_t Directory::getSize() const
{
size_t sum{};
for (auto& [k, v] : m_children)
for (const auto& [k, v] : m_children)
{
sum += v->getSize();
}
Expand Down
2 changes: 1 addition & 1 deletion Tokenizer.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ namespace shell

if (isBetweenQuotes)
{
return false;
return false; // skipcq: CXX-C2015
}
return true;
}
Expand Down
2 changes: 1 addition & 1 deletion UserManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,7 @@ namespace fs
<< "> ";
if (auto line = Utils::getLine())
{
password = Utils::trim(line.value());
password = Utils::trim(line.value()); // skipcq: CXX-A1004
isValid = true;
}
else if (Utils::promptToExitLoop())
Expand Down

0 comments on commit ac616bd

Please sign in to comment.