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

introduce parsing for 'long options' #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rzerres
Copy link
Contributor

@rzerres rzerres commented Mar 8, 2019

[v1-v4]

  • enable getopt_long supporting long option style
    (e.g --video-path, --fps, etc.)
  • try to be consistent which 'v4l2-compliance' binary when
    choosing option names
  • adapt help message
  • to adopt the slice-path you need to explicitly prepend an
    argument-option (-s or --slice-path) with the path as an option-string
  • adopt summary output to be in line with long options
  • typo corrections
  • update code style (tabify vs whitespace, alingement)

[v5]

  • typo corrections

Signed-off-by: Ralf Zerres [email protected]

@rzerres rzerres force-pushed the wip-longoptions branch 4 times, most recently from 9f27054 to c505fd9 Compare March 8, 2019 18:15
- enable getopt_long supporting long option style
  (e.g --video-path, --fps, etc.)
- try to be consistent which 'v4l2-compliance' binary when
  choosing option names
- adapt help message
- to adopt the slice-path you need to explicitly prepend an
  argument-option (-s or --slice-path) with the path as an option-string
- adopt summary output to be in line with long options
- typo corrections
- update code style (tabify vs whitespace, alingement)

Signed-off-by: Ralf Zerres <[email protected]>
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 this pull request may close these issues.

1 participant