You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ma...@apache.org on 2016/05/18 14:21:14 UTC
logging-log4j2 git commit: Update ReflectionUtil to ignore more
internal classes
Repository: logging-log4j2
Updated Branches:
refs/heads/master 7fca94f59 -> 407ec15d1
Update ReflectionUtil to ignore more internal classes
In JDK 1.9+, another internal reflection package jdk.internal.reflect has been added.
Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/407ec15d
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/407ec15d
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/407ec15d
Branch: refs/heads/master
Commit: 407ec15d14d604cf431b1dbcef8846e65cced9b9
Parents: 7fca94f
Author: Matt Sicker <bo...@gmail.com>
Authored: Wed May 18 09:21:04 2016 -0500
Committer: Matt Sicker <bo...@gmail.com>
Committed: Wed May 18 09:21:04 2016 -0500
----------------------------------------------------------------------
.../main/java/org/apache/logging/log4j/util/ReflectionUtil.java | 4 ++++
1 file changed, 4 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/407ec15d/log4j-api/src/main/java/org/apache/logging/log4j/util/ReflectionUtil.java
----------------------------------------------------------------------
diff --git a/log4j-api/src/main/java/org/apache/logging/log4j/util/ReflectionUtil.java b/log4j-api/src/main/java/org/apache/logging/log4j/util/ReflectionUtil.java
index b4a75b7..52f171e 100644
--- a/log4j-api/src/main/java/org/apache/logging/log4j/util/ReflectionUtil.java
+++ b/log4j-api/src/main/java/org/apache/logging/log4j/util/ReflectionUtil.java
@@ -181,6 +181,10 @@ public final class ReflectionUtil {
if (cn.startsWith("java.lang.reflect.") && (mn.equals("invoke") || mn.equals("newInstance"))) {
return false;
}
+ // ignore use of Java 1.9+ reflection classes
+ if (cn.startsWith("jdk.internal.reflect.")) {
+ return false;
+ }
// ignore Class.newInstance
if (cn.equals("java.lang.Class") && mn.equals("newInstance")) {
return false;