[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:

    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