diff options
author | ridiculousfish <corydoras@ridiculousfish.com> | 2014-04-27 18:44:21 -0700 |
---|---|---|
committer | ridiculousfish <corydoras@ridiculousfish.com> | 2014-04-27 18:44:21 -0700 |
commit | 97c2ec8dcfe14882bafb2f2c56502427c0ffa1d0 (patch) | |
tree | 350d646b1cfc0db862eef4f27938aeffd2ab04c9 | |
parent | 4948508277e424264a1b6232a6c275dd03fd584d (diff) |
Fix a duplicated variable, and defeat some warnings in fish_tests
-rw-r--r-- | fish_tests.cpp | 4 | ||||
-rw-r--r-- | parse_execution.cpp | 3 |
2 files changed, 3 insertions, 4 deletions
diff --git a/fish_tests.cpp b/fish_tests.cpp index 7ec4ab8b..6e293514 100644 --- a/fish_tests.cpp +++ b/fish_tests.cpp @@ -2012,8 +2012,8 @@ static void test_autosuggest_suggest_special() // A single quote should defeat tilde expansion perform_one_autosuggestion_test(L"cd '~/test_autosuggest_suggest_specia'", wd, L"", __LINE__); - system("rm -Rf '/tmp/autosuggest_test/'"); - system("rm -Rf ~/test_autosuggest_suggest_special/"); + if (system("rm -Rf '/tmp/autosuggest_test/'")) err(L"rm failed"); + if (system("rm -Rf ~/test_autosuggest_suggest_special/")) err(L"rm failed"); } static void test_autosuggestion_combining() diff --git a/parse_execution.cpp b/parse_execution.cpp index d5c63aba..3c4dfe6c 100644 --- a/parse_execution.cpp +++ b/parse_execution.cpp @@ -503,7 +503,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); - parse_execution_result_t ret = parse_execution_success; const parse_node_t *matching_case_item = NULL; parse_execution_result_t result = parse_execution_success; @@ -530,7 +529,7 @@ parse_execution_result_t parse_execution_context_t::run_switch_statement(const p { /* Store the node that failed to expand */ report_error(switch_value_node, WILDCARD_ERR_MSG, switch_value.c_str()); - ret = parse_execution_errored; + result = parse_execution_errored; break; } |