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

Reject final primitive fields annotated with @Option or @Parameters #68

Closed
remkop opened this issue Mar 9, 2017 · 0 comments
Closed

Reject final primitive fields annotated with @Option or @Parameters #68

remkop opened this issue Mar 9, 2017 · 0 comments
Labels
type: enhancement ✨
Milestone
2.1.0

Comments

@remkop
Copy link
Owner

remkop commented Mar 9, 2017

Final primitive fields are replaced with constant values (inlined), and programs will not be able to detect that the value was modified reflectively. JCommander solves this by throwing an exception if a final field is annotated with @parameter.
@remkop remkop added this to the 0.4.0 user manual milestone Apr 12, 2017
@remkop remkop changed the title Reject final fields annotated with @Option or @Parameters Reject final primitive fields annotated with @Option or @Parameters Apr 12, 2017
@remkop remkop modified the milestones: 0.5.0 advanced option parsing, 0.4.0 user manual Apr 12, 2017
remkop added a commit that referenced this issue Apr 17, 2017
@remkop
Added failing (ignored) test for Reject final primitive fields annota…
fb4445d 
…ted with @option or @parameters #68
@remkop remkop modified the milestones: backlog, 2.1.0 Oct 20, 2017
@remkop remkop closed this as completed in 3438b4e Nov 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: enhancement ✨
Projects
None yet
Milestone
2.1.0
Development

No branches or pull requests
1 participant
@remkop