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

Immutable Exceptions #810

Merged
merged 1 commit into from Oct 4, 2015

Conversation

Projects
None yet
2 participants
@hansjoachim
Copy link
Contributor

commented Oct 3, 2015

Made all fields in exception and errors final so that they cannot be changed after a problem occured.

SlimException checks on line 90 if tag is null or empty, so should be equivalent behaviour to assign an empty string if it is not passed in from the constructor.

Immutable Exceptions. Made all fields in exception and errors final s…
…o that they cannot be changed after a problem occured.

@amolenaar amolenaar added this to the Next release milestone Oct 4, 2015

amolenaar added a commit that referenced this pull request Oct 4, 2015

@amolenaar amolenaar merged commit 2ad5c0e into unclebob:master Oct 4, 2015

@hansjoachim hansjoachim deleted the hansjoachim:exceptions branch Oct 4, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.