This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/master by this push:
     new 77eb2a7  Remove unnecessary code
77eb2a7 is described below

commit 77eb2a72cc89b2d9dd2ca76b395f4671b2e35ac3
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Mon Apr 19 14:27:53 2021 +0100

    Remove unnecessary code
---
 java/org/apache/jasper/compiler/Generator.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/java/org/apache/jasper/compiler/Generator.java 
b/java/org/apache/jasper/compiler/Generator.java
index 7af2ccc..0f47b54 100644
--- a/java/org/apache/jasper/compiler/Generator.java
+++ b/java/org/apache/jasper/compiler/Generator.java
@@ -1346,8 +1346,10 @@ class Generator {
                     // and (for Java 9+) in an exported module
                     int modifiers = bean.getModifiers();
                     JreCompat jreCompat = JreCompat.getInstance();
+                    // No need to test for being an interface here as the
+                    // getConstructor() call above will have already failed for
+                    // any interfaces.
                     if (!Modifier.isPublic(modifiers) ||
-                            Modifier.isInterface(modifiers) ||
                             Modifier.isAbstract(modifiers) ||
                             !jreCompat.canAccess(null, constructor) ) {
                         throw new 
Exception(Localizer.getMessage("jsp.error.invalid.bean",

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to