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
Simplify boolean expressions with an simpler and equivalent. #104
Conversation
arturobernalg
commented
Aug 8, 2021
•
edited
edited
- Simplify boolean expression with equivalent.
@@ -38,7 +38,7 @@ | |||
* | |||
* @param numValues Number of values of the function to compute. | |||
* @param cache Cached values. | |||
* @throw IllegalArgumentException if {@code n < 0}. | |||
* @throw CombinatoricsException if {@code n < 0}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CombinatoricsException
is package private; it should not be exposed in the javadoc. It extends IllegalArgumentException so the javadoc is still valid.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
primeNumbers.add(Integer.valueOf(5)); | ||
primeNumbers.add(Integer.valueOf(7)); | ||
primeNumbers.add(Integer.valueOf(11)); | ||
primeNumbers.add(2); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current code makes the boxing explicit. Please avoid auto-boxing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
@@ -41,7 +41,7 @@ | |||
* | |||
* @param numValues Number of values of the function to compute. | |||
* @param cache Cached values. | |||
* @throw IllegalArgumentException if {@code n < 0}. | |||
* @throw CombinatoricsException if {@code n < 0}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have left this CombinatoricsException
exposed in the API.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
TY
This is now only simplifying boolean expressions. Can you update the commit message to reflect that and then correct the formatting as per #103 |
You changed the commit message but did not fix formatting to change 8 spaces to 4. |