diff options
author | Kurtis Rader <krader@skepticism.us> | 2016-04-04 14:34:28 -0700 |
---|---|---|
committer | Kurtis Rader <krader@skepticism.us> | 2016-04-04 14:34:28 -0700 |
commit | 47f1a92cc4c3c7056f4172974f10a8dd402cf55c (patch) | |
tree | 34bf83341006081f8a07bc6ca77cac03c603eb97 /src/parse_tree.cpp | |
parent | 0953590cca7cdc2dab75af5a85c8e2f15d4b8ab2 (diff) |
fixes for cppcheck lint warnings
Refine the linting behavior.
Fix several of the, mostly trivial, lint errors.
Diffstat (limited to 'src/parse_tree.cpp')
-rw-r--r-- | src/parse_tree.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/parse_tree.cpp b/src/parse_tree.cpp index 5939e270..51ae7720 100644 --- a/src/parse_tree.cpp +++ b/src/parse_tree.cpp @@ -690,7 +690,7 @@ void parse_ll_t::dump_stack(void) const } } - fprintf(stderr, "Stack dump (%lu elements):\n", symbol_stack.size()); + fprintf(stderr, "Stack dump (%zu elements):\n", symbol_stack.size()); for (size_t idx = 0; idx < lines.size(); idx++) { fprintf(stderr, " %ls\n", lines.at(idx).c_str()); @@ -1685,9 +1685,8 @@ enum parse_bool_statement_type_t parse_node_tree_t::statement_boolean_type(const bool parse_node_tree_t::job_should_be_backgrounded(const parse_node_t &job) const { assert(job.type == symbol_job); - bool result = false; const parse_node_t *opt_background = get_child(job, 2, symbol_optional_background); - result = opt_background != NULL && opt_background->tag == parse_background; + bool result = opt_background != NULL && opt_background->tag == parse_background; return result; } |