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

CmdLineParser: mark createOptionHandler() as non-deprecated #112

Merged
merged 1 commit into from
Jan 30, 2016
Merged

CmdLineParser: mark createOptionHandler() as non-deprecated #112

merged 1 commit into from
Jan 30, 2016

Commits on May 11, 2015

  1. CmdLineParser: mark createOptionHandler() as non-deprecated

    Since b1aa90f the createOptionHandler() was
    deprecated and the user was routed to OptionHandlerRegistry method
    instead. As a side effect the overridding of the now deprecated method
    createOptionHandler() doesn't have any effect, because this method is
    not get called any more from the addOption() and addArgument() methods.
    
    The documented OptionHandlerRegistry.registerHandler() machinery cannot
    be used, when a DI framework is used and the instances must be created by
    the injector.
    
    Mark createOptionHandler as non deprecated again and use it from addOption()
    and addArgument() methods.
    
    This commit partial reverts b1aa90f
    davido committed May 11, 2015
    Configuration menu
    Copy the full SHA
    d0a9de1 View commit details
    Browse the repository at this point in the history