This repository has been archived by the owner on Aug 6, 2024. It is now read-only.
Add man page #106
Open
Add man page #106
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Error is always shown because an error will halt the application. I'm going to quickly rewrite how command line options are handled so I'll probably be updating how the debug flags are handled. I'll ping you when it's done. |
|
I've re-implemented CLI options. The |
* New log levels * Updated date * Updated features
vinszent
force-pushed
the
master
branch
2 times, most recently
from
7d02b63 to
11b742b
Compare
# for free
to subscribe to this conversation on GitHub.
Already have an account?
#.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I had originally written this man page for Debian (because every executable is required to have a man page in Debian), and while working on the new package I asked myself why I didn't submit it upstream earlier, so here I am. :)
One thing I'm not sure about are the log levels, because they seem to be
-1 (Error)
0 (Critical)
1 (Warning)
2 (Message)
3 (Info)
4 (Debug)
I didn't document the "Error" level (-1) because it seems weird, so maybe it wasn't intended to be specified by the user. It also seems to be used only once in the mpv backend (which doesn't really exist yet). But maybe it is the numbering that is wrong and Error was supposed to be 0 and so on, if that's the case I can update the PR after that is fixed.