Revision: 1594
Author: [email protected]
Date: Sun Oct 16 16:33:52 2011
Log:
fix issue 636, GuiceFilter should keep a strong ref to its logger.
Revision created by MOE tool push_codebase.
MOE_MIGRATION=3478
http://code.google.com/p/google-guice/source/detail?r=1594
Modified:
/trunk/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
=======================================
---
/trunk/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
Sun Oct 16 15:35:11 2011
+++
/trunk/extensions/servlet/src/com/google/inject/servlet/GuiceFilter.java
Sun Oct 16 16:33:52 2011
@@ -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
--
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.
