Author: seb
Date: Mon Mar 5 17:00:41 2007
New Revision: 1406
Modified:
logback/trunk/logback-core/src/main/java/ch/qos/logback/core/boolex/EventEvaluator.java
logback/trunk/logback-core/src/main/java/ch/qos/logback/core/filter/EvaluatorFilter.java
Log:
EventEvaluator now extends LifeCycle
Better fail check in EvaluatorFilter
Modified:
logback/trunk/logback-core/src/main/java/ch/qos/logback/core/boolex/EventEvaluator.java
==============================================================================
---
logback/trunk/logback-core/src/main/java/ch/qos/logback/core/boolex/EventEvaluator.java
(original)
+++
logback/trunk/logback-core/src/main/java/ch/qos/logback/core/boolex/EventEvaluator.java
Mon Mar 5 17:00:41 2007
@@ -12,6 +12,7 @@
package ch.qos.logback.core.boolex;
import ch.qos.logback.core.spi.ContextAware;
+import ch.qos.logback.core.spi.LifeCycle;
/**
* An EventEvaluator has the responsibility to evaluate whether a given an
event
@@ -23,7 +24,7 @@
* @author Ceki Gülcü
*/
-public interface EventEvaluator extends ContextAware {
+public interface EventEvaluator extends ContextAware, LifeCycle {
/**
Modified:
logback/trunk/logback-core/src/main/java/ch/qos/logback/core/filter/EvaluatorFilter.java
==============================================================================
---
logback/trunk/logback-core/src/main/java/ch/qos/logback/core/filter/EvaluatorFilter.java
(original)
+++
logback/trunk/logback-core/src/main/java/ch/qos/logback/core/filter/EvaluatorFilter.java
Mon Mar 5 17:00:41 2007
@@ -47,7 +47,7 @@
public FilterReply decide(Object event) {
// let us not throw an exception
// see also bug #17.
- if(!isStarted()) {
+ if(!isStarted() || !evaluator.isStarted()) {
return FilterReply.NEUTRAL;
}
try {
_______________________________________________
logback-dev mailing list
[email protected]
http://qos.ch/mailman/listinfo/logback-dev
