fix issue 636, GuiceFilter should keep a strong ref to its logger.
Revision created by MOE tool push_codebase.
MOE_MIGRATION=3478
git-svn-id: https://google-guice.googlecode.com/svn/trunk@1594 d779f126-a31b-0410-b53b-1d3aecad763e
diff --git a/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java b/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
index 5582174..d069c96 100644
--- a/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
+++ b/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
@@ -77,6 +77,8 @@
+ "in your web application. If this is deliberate, you may safely "
+ "ignore this message. If this is NOT deliberate however, "
+ "your application may not work as expected.";
+
+ private static final Logger LOGGER = Logger.getLogger(GuiceFilter.class.getName());
public GuiceFilter() {
// Use the static FilterPipeline
@@ -94,7 +96,7 @@
// This can happen if you create many injectors and they all have their own
// servlet module. This is legal, caveat a small warning.
if (GuiceFilter.pipeline instanceof ManagedFilterPipeline) {
- Logger.getLogger(GuiceFilter.class.getName()).warning(MULTIPLE_INJECTORS_WARNING);
+ LOGGER.warning(MULTIPLE_INJECTORS_WARNING);
}
// We overwrite the default pipeline