Update of /var/cvs/src/org/mmbase/util/logging
In directory james.mmbase.org:/tmp/cvs-serv10340

Modified Files:
      Tag: MMBase-1_8
        LoggerWrapper.java 
Log Message:
it's very bad to wrap null, so simply absolutely refuse to do that


See also: http://cvs.mmbase.org/viewcvs/src/org/mmbase/util/logging


Index: LoggerWrapper.java
===================================================================
RCS file: /var/cvs/src/org/mmbase/util/logging/LoggerWrapper.java,v
retrieving revision 1.6
retrieving revision 1.6.2.1
diff -u -b -r1.6 -r1.6.2.1
--- LoggerWrapper.java  30 Aug 2006 17:54:00 -0000      1.6
+++ LoggerWrapper.java  11 Feb 2008 16:06:32 -0000      1.6.2.1
@@ -17,7 +17,7 @@
  *
  * @author Michiel Meeuwissen
  * @since  MMBase-1.7
- * @version $Id: LoggerWrapper.java,v 1.6 2006/08/30 17:54:00 michiel Exp $
+ * @version $Id: LoggerWrapper.java,v 1.6.2.1 2008/02/11 16:06:32 michiel Exp $
  **/
 
 public class LoggerWrapper implements Logger {
@@ -31,15 +31,16 @@
     }
 
     private Logger log;
-    private String name;
+    private final String name;
 
     // package
     LoggerWrapper(Logger log, String name) {
-        this.log  = log;
+        setLogger(log);
         this.name = name;
         wrappers.add(this);
     }
 
+
     // package
     String getName() {
         return name;
@@ -47,6 +48,11 @@
 
     // package
     Logger setLogger(Logger log) {
+        if (log == null) {
+            log = SimpleImpl.getLoggerInstance(name);
+            System.err.println("Tried to instantiate logger wrapper with 
null!");
+            log.error("Tried to instantiate logger wrapper with null!", new 
Exception());
+        }
         Logger org = this.log;
         this.log = log;
         return org;
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to