paul j3 added the comment:

These three changes end up testing for the same thing. The initial 'if' catches 
different test cases.  'subparsers' or 'remainder' might 'confuse' the 'O' 
test.  The zero-width test ends up weeding out everything but the test cases 
added for this issue.

    # if we haven't hit the end of the command line strings,
    if start_index + sum(arg_counts) != len(arg_strings_pattern):
        while arg_counts and arg_counts[-1] == 0: 
            arg_counts.pop()

    # same test using selected_pattern (= arg_strings_pattern[start_index:])
    if len(selected_pattern) != sum(arg_counts):
        while arg_counts and arg_counts[-1] == 0: 
            arg_counts.pop()

    # alt test: test for optional in the remaining pattern
    if 'O' in selected_pattern:
        while arg_counts and arg_counts[-1] == 0: 
            arg_counts.pop()

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue15112>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to