arturobernalg commented on a change in pull request #729:
URL: https://github.com/apache/commons-lang/pull/729#discussion_r588486562
##########
File path: src/main/java/org/apache/commons/lang3/exception/ExceptionUtils.java
##########
@@ -586,6 +586,30 @@ public static int indexOfType(final Throwable throwable,
final Class<? extends T
return indexOf(throwable, type, fromIndex, true);
}
+ /**
+ * Return whether the given throwable is a checked exception:
+ * that is, neither a RuntimeException nor an Error.
+ *
+ * <pre>
+ * ExceptionUtils.isCheckedException(new Exception()) =
true
+ * ExceptionUtils.isCheckedException(new SQLException()) =
true
+ * ExceptionUtils.isCheckedException(new RuntimeException()) =
false
+ * ExceptionUtils.isCheckedException(new IllegalArgumentException("")) =
false
+ * ExceptionUtils.isCheckedException(new Throwable()) =
true
+ * ExceptionUtils.isCheckedException(null) =
false
+ * </pre>
+ *
+ * @param throwable the throwable to check
+ * @return whether the throwable is a checked exception
+ * @see java.lang.Exception
+ * @see java.lang.RuntimeException
+ * @see java.lang.Error
+ * @since 3.12.1
+ */
+ public static boolean isCheckedException(final Throwable throwable) {
+ return throwable != null && !(throwable instanceof RuntimeException ||
throwable instanceof Error);
Review comment:
No. Return true without the null check
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
