diff options
| author |  ridiculousfish <corydoras@ridiculousfish.com> | 2016-02-19 15:45:12 -0800 |
|---|---|---|
| committer | ridiculousfish <corydoras@ridiculousfish.com> | 2016-02-19 15:45:31 -0800 |
| commit | 947f659f969b318eb7757656e28188e5d2d0f5cf (patch) | |
| tree | 866487c6f6473f86d765b40362fc4d1725eb2862 /src/parse_execution.cpp | |
| parent | 8703c5bc00aa7349d65078f8b2c0c5a7a8ed9c74 (diff) | |
Miscellaneous cleanup and dead code removal
Noticed by cppcheck
Diffstat (limited to 'src/parse_execution.cpp')
| -rw-r--r-- | src/parse_execution.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/parse_execution.cpp b/src/parse_execution.cpp index 0ea0aba9..061ca03e 100644 --- a/src/parse_execution.cpp +++ b/src/parse_execution.cpp @@ -536,7 +536,6 @@ parse_execution_result_t parse_execution_context_t::run_for_statement(const pars parse_execution_result_t parse_execution_context_t::run_switch_statement(const parse_node_t &statement) { assert(statement.type == symbol_switch_statement); - const parse_node_t *matching_case_item = NULL; parse_execution_result_t result = parse_execution_success; @@ -590,6 +589,7 @@ parse_execution_result_t parse_execution_context_t::run_switch_statement(const p const parse_node_t *case_item_list = get_child(statement, 3, symbol_case_item_list); /* Loop while we don't have a match but do have more of the list */ + const parse_node_t *matching_case_item = NULL; while (matching_case_item == NULL && case_item_list != NULL) { if (should_cancel_execution(sb)) @@ -990,7 +990,6 @@ parse_execution_result_t parse_execution_context_t::determine_arguments(const pa { this->report_errors(errors); return parse_execution_errored; - break; } case EXPAND_WILDCARD_NO_MATCH: |