Revision: 1587
Author: sberlin
Date: Sun Oct 16 08:55:38 2011
Log: fix issue 636, GuiceFilter should keep a strong ref to its logger.
http://code.google.com/p/google-guice/source/detail?r=1587
Modified:
/trunk/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
=======================================
---
/trunk/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
Tue Sep 27 08:36:19 2011
+++
/trunk/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
Sun Oct 16 08:55:38 2011
@@ -78,6 +78,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());
//VisibleForTesting
@Inject
@@ -86,7 +88,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
--
You received this message because you are subscribed to the Google Groups
"google-guice-dev" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/google-guice-dev?hl=en.
