Skip to content

Commit

Permalink
Cleanup: Missed a few more warnings in Command.cc
Browse files Browse the repository at this point in the history
  • Loading branch information
dburkart committed Dec 20, 2023
1 parent 13635bb commit ad8838e
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/AST/Validation/Command.cc
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,7 @@ bool Command::_validateRedirectCommand(const ASTNode *node) {
std::vector<sieve::ASTNode *> children = command->children();
const size_t size = children.size();

int numArguments = 1;
size_t numArguments = 1;

if (command->find(ASTTag(":copy")) != command->children().end() ||
command->find(ASTTag(":list")) != command->children().end()) {
Expand All @@ -337,8 +337,8 @@ bool Command::_validateSetCommand(const ASTNode *node) {
std::vector<sieve::ASTNode *> children = command->children();
const size_t size = children.size();

int numArguments = 2;
int stringArguments = 0;
size_t numArguments = 2;
size_t stringArguments = 0;

for (auto it : children) {
const ASTTag *tagChild = dynamic_cast<ASTTag *>(it);
Expand Down Expand Up @@ -388,7 +388,7 @@ bool Command::_validateVacationCommand(const ASTNode *node) {
std::vector<sieve::ASTNode *> children = command->children();
const size_t size = children.size();

int numArguments = 1;
size_t numArguments = 1;

if (command->find(ASTTag(":days")) != command->children().end()) {
numArguments += 2;
Expand Down Expand Up @@ -509,7 +509,7 @@ bool Command::_validateNotifyCommand(const ASTNode *node) {
const auto *command = dynamic_cast<const ASTCommand*>(node);
const size_t size = command->children().size();

int numArguments = 1;
size_t numArguments = 1;

if (!nodeIsType<ASTString>(command->children()[size-1]))
return false;
Expand Down

0 comments on commit ad8838e

Please sign in to comment.