From f07808a8f90bd36f25e40b93a23e8360f9c5e7f6 Mon Sep 17 00:00:00 2001 From: Kevin Van Brunt Date: Fri, 27 Sep 2019 17:43:28 -0400 Subject: Changed some logic and added comment why --- cmd2/argparse_completer.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'cmd2/argparse_completer.py') diff --git a/cmd2/argparse_completer.py b/cmd2/argparse_completer.py index c1200ae5..66ee89f6 100644 --- a/cmd2/argparse_completer.py +++ b/cmd2/argparse_completer.py @@ -331,8 +331,9 @@ class AutoCompleter(object): # Check if we have a positional to consume this token if pos_arg_state is not None: - if not update_mutex_groups(pos_arg_state.action): - return [] + # No need to check for an error since we remove a completed group's positional from + # remaining_positionals which means this action can't belong to a completed mutex group + update_mutex_groups(pos_arg_state.action) consume_argument(pos_arg_state) -- cgit v1.2.1