Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Enhancement: The library should defensively check input arguments #70

Closed
bbottema opened this issue Mar 11, 2017 · 1 comment
Closed

Enhancement: The library should defensively check input arguments #70

bbottema opened this issue Mar 11, 2017 · 1 comment
Assignees
@bbottema
Labels
enhancement Priority-Low
Milestone
4.2.0

Comments

@bbottema
Copy link
Owner

A lot of places now have @nonnull and @nullable annotations to provide hints to static analyses tools, but this doesn't add any value in runtime conditions.

It would be nice if Simple Java Mail implemented defensive null/empty checks on all required parameters, so we can Fail Fast.
Annotations
@bbottema bbottema self-assigned this Mar 11, 2017
@bbottema bbottema added this to the 4.2.1 milestone Mar 11, 2017
bbottema added a commit that referenced this issue Mar 11, 2017
@bbottema
#70 Added defensive null/empty checks to Fail Fast (http://wiki.c2.co…
a99f74c 
…m/?FailFast)
bbottema added a commit that referenced this issue Mar 11, 2017
@bbottema
#70 Added more defensive null/empty checks to Fail Fast (http://wiki.…
9d89c30 
…c2.com/?FailFast)
@bbottema
Copy link
Owner Author

Released in v4.2.0.

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@bbottema bbottema

Labels
enhancement Priority-Low
Projects
None yet
Milestone
4.2.0
Development

No branches or pull requests
1 participant
@bbottema