Author: johnh
Date: Mon Aug  2 23:36:21 2010
New Revision: 981705

URL: http://svn.apache.org/viewvc?rev=981705&view=rev
Log:
Better logging in MutableContent, for when GadgetException is caught during 
Document parsing.

Patch provided by Gagan Singh.


Modified:
    
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/MutableContent.java

Modified: 
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/MutableContent.java
URL: 
http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/MutableContent.java?rev=981705&r1=981704&r2=981705&view=diff
==============================================================================
--- 
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/MutableContent.java
 (original)
+++ 
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/MutableContent.java
 Mon Aug  2 23:36:21 2010
@@ -35,6 +35,8 @@ import java.nio.ByteBuffer;
 import java.nio.charset.Charset;
 import java.util.Arrays;
 import java.util.Map;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  * Object that maintains a String representation of arbitrary contents
@@ -59,6 +61,7 @@ public class MutableContent {
   private Map<String, Object> pipelinedData;
 
   private static final String MUTABLE_CONTENT_LISTENER = 
"MutableContentListener";
+  private static final Logger logger = 
Logger.getLogger(MutableContent.class.getName());
 
   public static void notifyEdit(Document doc) {
     MutableContent mc = (MutableContent) 
doc.getUserData(MUTABLE_CONTENT_LISTENER);
@@ -236,7 +239,7 @@ public class MutableContent {
       document = contentParser.parseDom(getContent());
       document.setUserData(MUTABLE_CONTENT_LISTENER, this, null);
     } catch (GadgetException e) {
-      // TODO: emit info message
+      logger.log(Level.WARNING, "Got GadgetException when parsing content", e);
       return null;
     }
     return document;


Reply via email to