Bug 353194

Description Karsten Thoms 2011-07-27 06:40:47 EDT

In AbstractDeclarativeValidator#invoke() the target of a InvocationTargetException is thrown as a RuntimeException. It is not necessary to wrap it when the target is already a RuntimeException, in this case just throw that instance.



        catch (InvocationTargetException e) {
          // ignore GuardException, check is just not evaluated if
          // guard is false
          // ignore NullPointerException, as not having to check for
          // NPEs all the time is a convenience feature
          Throwable targetException = e.getTargetException();
          if (!(targetException instanceof GuardException)
              && !(targetException instanceof NullPointerException))
            if (targetException instanceof RuntimeException)
               throw targetException;
            else
               throw new RuntimeException(targetException);
        }