svn commit: r1390234 - /shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/ApiServlet.java

[ss...@apache.org]

Author: ssievers
Date: Wed Sep 26 00:20:06 2012
New Revision: 1390234

URL: http://svn.apache.org/viewvc?rev=1390234&view=rev
Log:
SHINDIG-1874 | Leaky abstraction for BeanJsonConverter | Patch from Jan Willem Janssen.  Thanks!

Modified:
    shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/ApiServlet.java

Modified: shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/ApiServlet.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/ApiServlet.java?rev=1390234&r1=1390233&r2=1390234&view=diff
==============================================================================
--- shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/ApiServlet.java (original)
+++ shindig/trunk/java/common/src/main/java/org/apache/shindig/protocol/ApiServlet.java Wed Sep 26 00:20:06 2012
@@ -67,7 +67,7 @@ public abstract class ApiServlet extends
        Key.get(new TypeLiteral<Set<Object>>(){}, Names.named("org.apache.shindig.handlers"));
 
   protected HandlerRegistry dispatcher;
-  protected BeanJsonConverter jsonConverter;
+  protected BeanConverter jsonConverter;
   protected BeanConverter xmlConverter;
   protected BeanConverter atomConverter;
   protected ContainerConfig containerConfig;
@@ -105,8 +105,7 @@ public abstract class ApiServlet extends
       @Named("shindig.bean.converter.json") BeanConverter jsonConverter,
       @Named("shindig.bean.converter.xml") BeanConverter xmlConverter,
       @Named("shindig.bean.converter.atom") BeanConverter atomConverter) {
-    // fix this
-    this.jsonConverter = (BeanJsonConverter) jsonConverter;
+    this.jsonConverter = jsonConverter;
     this.xmlConverter = xmlConverter;
     this.atomConverter = atomConverter;
   }