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.

Sign up for GitHub

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

Jump to bottom

src: add CheckOptions to Options classes #22943

Closed
wants to merge 2 commits into from
Closed

src: add CheckOptions to Options classes #22943

wants to merge 2 commits into from

Commits on Sep 19, 2018

  1. src: add CheckOptions to Options classes 

    This commit adds a CheckOptions function that the options classes can
optionally implement to check that options specified are correct
(dependencies between options are met or options that are mutually
exclusive).

In the process of doing this the error pointer passed to Parse was
changed to be of type vector so that potentially multiple options check
failures can be reported.
    @danbev
    danbev committed Sep 19, 2018
    Configuration menu
    Copy the full SHA
    cc9e954 View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2018

  1. squash: make CheckOptions delegate

    @danbev
    danbev committed Sep 20, 2018
    Configuration menu
    Copy the full SHA
    0c61a6d View commit details
    Browse the repository at this point in the history