You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by ga...@apache.org on 2010/11/17 21:46:43 UTC

svn commit: r1036219 - in /shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite: RewriteModule.java RewritePath.java

Author: gagan
Date: Wed Nov 17 20:46:43 2010
New Revision: 1036219

URL: http://svn.apache.org/viewvc?rev=1036219&view=rev
Log:
patch by gagan.goku | Issue 3175041: Small enhancements to RewriteModule to make it reusable | http://codereview.appspot.com/3175041/

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

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java?rev=1036219&r1=1036218&r2=1036219&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java Wed Nov 17 20:46:43 2010
@@ -86,16 +86,17 @@ public class RewriteModule extends Abstr
     mapbinder.addBinding(rewritePath).toProvider(list);
   }
 
-  void addBindingForRewritePath(String container, RewriteFlow rewriteFlow) {
+  protected void addBindingForRewritePath(String container, RewriteFlow rewriteFlow) {
     addBindingForRewritePath(container, rewriteFlow, binder().getProvider(
         getKey(container, rewriteFlow)));
   }
 
-  Provider<List<ResponseRewriter>> getResponseRewriters(String container, RewriteFlow flow) {
+  protected Provider<List<ResponseRewriter>> getResponseRewriters(String container,
+                                                                  RewriteFlow flow) {
     return binder().getProvider(getKey(container, flow));
   }
 
-  Key<List<ResponseRewriter>> getKey(String container, RewriteFlow flow) {
+  protected Key<List<ResponseRewriter>> getKey(String container, RewriteFlow flow) {
     return Key.get(new TypeLiteral<List<ResponseRewriter>>() {},
                    new RewritePath(container, flow));
   }

Modified: shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java
URL: http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java?rev=1036219&r1=1036218&r2=1036219&view=diff
==============================================================================
--- shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java (original)
+++ shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewritePath.java Wed Nov 17 20:46:43 2010
@@ -23,7 +23,7 @@ import java.lang.annotation.Annotation;
 /**
  * Implementation of ResponseRewriterList annotation interface.
  */
-class RewritePath implements ResponseRewriterList {
+public class RewritePath implements ResponseRewriterList {
   protected final String container;
   protected final RewriteFlow rewriteFlow;
 
@@ -63,4 +63,4 @@ class RewritePath implements ResponseRew
   public Class<? extends Annotation> annotationType() {
     return ResponseRewriterList.class;
   }
-}
\ No newline at end of file
+}