From 0372cdbe7eba761d6fce1abccdf70c33c956968e Mon Sep 17 00:00:00 2001 From: Kevin Ballard Date: Mon, 13 Oct 2014 17:27:56 -0700 Subject: [PATCH] Standardize unmatched wildcard errors Unmatched wildcard errors during parsing are normally only reported when run interactively. The switch command was unconditionally reporting them anyway (and not setting the status to 124). Fix it so switch goes through the same code path as everything else. --- parse_execution.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/parse_execution.cpp b/parse_execution.cpp index 18d419ff9..a77ba5796 100644 --- a/parse_execution.cpp +++ b/parse_execution.cpp @@ -552,9 +552,7 @@ parse_execution_result_t parse_execution_context_t::run_switch_statement(const p case EXPAND_WILDCARD_NO_MATCH: { - /* Store the node that failed to expand */ - report_error(switch_value_node, WILDCARD_ERR_MSG, switch_value.c_str()); - result = parse_execution_errored; + result = report_unmatched_wildcard_error(switch_value_node); break; }