Closed
Description
parse_intermixed_args()
and parse_known_intermixed_args()
are implemented by parsing command lines twice -- first only optional arguments, then the remaining is parsed as positional arguments. This approach has some issues.
- The parser is temporary modified to suppress positional or required optional arguments in these two stages. This is not good, because the parser can no longer be used concurrently. This also smells bad in general, this can hide some bugs.
- Default values are handled twice.
- Unknown options in
parse_known_intermixed_args()
cannot be intermixed with positional arguments. Well, "parsing only known arguments" is a dubious feature, but still...
I tried to rewrite the implementation by moving the code deeper. No longer parser patching, defaults are handled only once, unknown options are excluded from positionals parsing. @hpaulj, @bitdancer, as the authors of the original implementation, could you please review this code?
Linked PRs
Metadata
Metadata
Assignees
Projects
Status
Doc issues