[clang-tidy] convert several loops to range based ones

Found with modernize-loop-convert Signed-off-by: 's avatarRosen Penev <rosenp@gmail.com>
parent bc6eca21
...@@ -246,8 +246,8 @@ static CommandResult ...@@ -246,8 +246,8 @@ static CommandResult
PrintAvailableCommands(Response &r, const Partition &partition, PrintAvailableCommands(Response &r, const Partition &partition,
unsigned permission) noexcept unsigned permission) noexcept
{ {
for (unsigned i = 0; i < num_commands; ++i) { for (const auto & i : commands) {
const struct command *cmd = &commands[i]; const struct command *cmd = &i;
if (cmd->permission == (permission & cmd->permission) && if (cmd->permission == (permission & cmd->permission) &&
command_available(partition, cmd)) command_available(partition, cmd))
...@@ -260,8 +260,8 @@ PrintAvailableCommands(Response &r, const Partition &partition, ...@@ -260,8 +260,8 @@ PrintAvailableCommands(Response &r, const Partition &partition,
static CommandResult static CommandResult
PrintUnavailableCommands(Response &r, unsigned permission) noexcept PrintUnavailableCommands(Response &r, unsigned permission) noexcept
{ {
for (unsigned i = 0; i < num_commands; ++i) { for (const auto & i : commands) {
const struct command *cmd = &commands[i]; const struct command *cmd = &i;
if (cmd->permission != (permission & cmd->permission)) if (cmd->permission != (permission & cmd->permission))
r.Format("command: %s\n", cmd->cmd); r.Format("command: %s\n", cmd->cmd);
......
...@@ -516,8 +516,8 @@ parse_xing(struct xing *xing, struct mad_bitptr *ptr, int *oldbitlen) noexcept ...@@ -516,8 +516,8 @@ parse_xing(struct xing *xing, struct mad_bitptr *ptr, int *oldbitlen) noexcept
if (xing->flags & XING_TOC) { if (xing->flags & XING_TOC) {
if (bitlen < 800) if (bitlen < 800)
return false; return false;
for (unsigned i = 0; i < 100; ++i) for (unsigned char & i : xing->toc)
xing->toc[i] = mad_bit_read(ptr, 8); i = mad_bit_read(ptr, 8);
bitlen -= 800; bitlen -= 800;
} }
......
...@@ -84,8 +84,8 @@ NeighborGlue::Open() ...@@ -84,8 +84,8 @@ NeighborGlue::Open()
void void
NeighborGlue::Close() noexcept NeighborGlue::Close() noexcept
{ {
for (auto i = explorers.begin(), end = explorers.end(); i != end; ++i) for (auto & explorer : explorers)
i->explorer->Close(); explorer.explorer->Close();
} }
NeighborGlue::List NeighborGlue::List
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment