git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] commons-cli issue #25: CLI-284: Fix inconsistent behaviour for short and lon...


Github user sfuhrm commented on the issue:

    https://github.com/apache/commons-cli/pull/25
  
    Hi @kinow 
    
    thanks for your review! Oops, my maven build had a hickup and all tests were green.
    
    There's a design problem in the code you pointed me to. Every Option instance may be illegal (short and long == null) until first use. OptionBuilder (line 356) insists on creating an Option instance even if he long parameter is missing. Option.Builder does something similar.
    The setting happens after Option instance construction.



---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@xxxxxxxxxxxxxxxxxx
For additional commands, e-mail: dev-help@xxxxxxxxxxxxxxxxxx




( ! ) Warning: include(msgfooter.php): failed to open stream: No such file or directory in /var/www/git/apache-commons-developers/msg06507.html on line 88
Call Stack
#TimeMemoryFunctionLocation
10.0007363704{main}( ).../msg06507.html:0

( ! ) Warning: include(): Failed opening 'msgfooter.php' for inclusion (include_path='.:/var/www/git') in /var/www/git/apache-commons-developers/msg06507.html on line 88
Call Stack
#TimeMemoryFunctionLocation
10.0007363704{main}( ).../msg06507.html:0