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
Lang 1482 #448
Lang 1482 #448
Conversation
…short class names
…ength should result ".." it was throwing exception. Tests are added
pull original master
apache master merged into current branch
throw rethrow(t); | ||
} | ||
}; | ||
return () -> call(pCallable); |
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 functionality as it was implemented the first hand is already there.
update master on fork
…ache-master # Conflicts: # src/test/java/org/apache/commons/lang3/ClassUtilsTest.java
Conflicts: src/main/java/org/apache/commons/lang3/Functions.java src/test/java/org/apache/commons/lang3/FunctionsTest.java
@@ -414,8 +412,8 @@ public void testAsPredicate() { | |||
assertNotNull(cause); | |||
assertTrue(cause instanceof SomeException); | |||
assertEquals("Odd Invocation: 1", cause.getMessage()); | |||
// assert that it does not throw exception | |||
final boolean instance = predicate.test(null); |
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.
assertNotNull()
makes no sense on primitive
And now all the cleanup, extra JavaDoc is just gone... |
mainly code clean up
extra tests that were missing
rethrow now corrects stack trace showing the location where the rethrow was called
(nobody cares that an exception was thrown from a utility function that does nothing else but throws the exception)
documentation was extended