diff options
author | Tom Tromey <tromey@redhat.com> | 2007-03-30 04:09:35 +0000 |
---|---|---|
committer | Tom Tromey <tromey@gcc.gnu.org> | 2007-03-30 04:09:35 +0000 |
commit | cc5e1fab404f5c6c6f6f7e79c5626fc681eae97d (patch) | |
tree | 8b08d6e0dec0395e2de932fcd3c04af282a603c0 /libjava/java/util | |
parent | 1a54978869b510d5b712778ae5e32521459f3791 (diff) | |
download | gcc-cc5e1fab404f5c6c6f6f7e79c5626fc681eae97d.zip gcc-cc5e1fab404f5c6c6f6f7e79c5626fc681eae97d.tar.gz gcc-cc5e1fab404f5c6c6f6f7e79c5626fc681eae97d.tar.bz2 |
re PR libgcj/29869 (LogManager class loading failure with Tomcat)
libjava
PR libgcj/29869:
* java/util/logging/LogManager.java (readConfiguration): Handle
comma-separated 'handlers'. Don't try to add a non-existing
handler.
libgcj/classpath
PR libgcj/29869:
* gnu/java/util/jar/JarUtils.java (log): Commented out.
(readSFManifest): Don't log.
From-SVN: r123356
Diffstat (limited to 'libjava/java/util')
-rw-r--r-- | libjava/java/util/logging/LogManager.java | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/libjava/java/util/logging/LogManager.java b/libjava/java/util/logging/LogManager.java index 1e0192e..811310c 100644 --- a/libjava/java/util/logging/LogManager.java +++ b/libjava/java/util/logging/LogManager.java @@ -559,13 +559,21 @@ public class LogManager if ("handlers".equals(key)) { - StringTokenizer tokenizer = new StringTokenizer(value); + // In Java 5 and earlier this was specified to be + // whitespace-separated, but in reality it also accepted + // commas (tomcat relied on this), and in Java 6 the + // documentation was updated to fit the implementation. + StringTokenizer tokenizer = new StringTokenizer(value, + " \t\n\r\f,"); while (tokenizer.hasMoreTokens()) { String handlerName = tokenizer.nextToken(); Handler handler = (Handler) createInstance(handlerName, Handler.class, key); - Logger.root.addHandler(handler); + // Tomcat also relies on the implementation ignoring + // items in 'handlers' which are not class names. + if (handler != null) + Logger.root.addHandler(handler); } } |