Issue 17250: argparse: Issue 15906 patch; positional with nargs='*' and string default (original) (raw)

The production argparse applies the type conversion to a string default whether it is needed or not. With the 12776 and 15906 patch, that conversion is postponed, so that it is applied only once. However, for a positional argument with nargs='*', that conversion is never applied.

For example, with: add_argument('foo', type=FileType('r'), default='anyfile', nargs='*')

the development version, with parse_args([]) produces Namespace(foo='anyfile')

The previous code tested this default, raising an error if there was an IOError. But even if it successfully opened the file, the namespace would get the string value, not the opened file.

With nargs = '?', the result is an IOError, or an opened file.

It is evident from the code (but not the documentation) that the best default for the '*' positional is a list of appropriate type of objects. In the case of FileTypes, about the only choices, without opening a file, are: [] and [sys.stdin].