Update of /var/cvs/src/org/mmbase/util/logging
In directory james.mmbase.org:/tmp/cvs-serv10464
Modified Files:
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.8
retrieving revision 1.9
diff -u -b -r1.8 -r1.9
--- LoggerWrapper.java 6 Dec 2007 08:28:07 -0000 1.8
+++ LoggerWrapper.java 11 Feb 2008 16:11:02 -0000 1.9
@@ -17,7 +17,7 @@
*
* @author Michiel Meeuwissen
* @since MMBase-1.7
- * @version $Id: LoggerWrapper.java,v 1.8 2007/12/06 08:28:07 michiel Exp $
+ * @version $Id: LoggerWrapper.java,v 1.9 2008/02/11 16:11:02 michiel Exp $
**/
public class LoggerWrapper implements Logger {
@@ -31,12 +31,12 @@
}
private Logger log;
- private String name;
+ private final String name;
// package
LoggerWrapper(Logger log, String name) {
- this.log = log;
this.name = name;
+ setLogger(log);
wrappers.add(this);
}
@@ -47,6 +47,15 @@
// package
Logger setLogger(Logger log) {
+ if (log == null) {
+ if (this.log == null) {
+ log = SimpleImpl.getLoggerInstance(name);
+ } else {
+ log = this.log;
+ }
+ 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
