Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OptionsSingleInSingleOut does not warn or give error on excess positional parameters #842

Closed
MarkCallow opened this issue Jan 20, 2024 · 0 comments · Fixed by #843
Closed
Assignees

Comments

@MarkCallow
Copy link
Collaborator

With the command ktx foo <options ...> file1 file2 file3 ... where foo uses OptionsSingleInSingleOut, OptionsSingleInSingleOut will use file1 as input, file2 as output and ignore file3. As the last positional parameter is usually the output file sloppy commands could easily lead to unintentional overwrites of file2.

Note that I spotted this in a new command I am developing. Is the command expected to check for excess parameters? I can't find any check in command_encode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants