This patch modified usage() so that it does not always exit 1. However, actual error condition that result is displaying usage do now exit 1 where appropriate.
The reason for this change is so that simple build/test CI jobs do not encounter a failure condition when legitimately requesting ./quickget --help.
Type of change
[x] Bug fix (non-breaking change which fixes an issue)
Checklist:
[x] I have performed a self-review of my code
[x] I have tested my code in common scenarios and confirmed there are no regressions
Description
This patch modified
usage()so that it does not alwaysexit 1. However, actual error condition that result is displaying usage do nowexit 1where appropriate.The reason for this change is so that simple build/test CI jobs do not encounter a failure condition when legitimately requesting
./quickget --help.Type of change
Checklist: