Looks great. I had 2 comments but cleaned it up myself and committed.
Thanks!


http://codereview.appspot.com/186233/diff/18/22
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderingResults.java
(right):

http://codereview.appspot.com/186233/diff/18/22#newcode38
java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderingResults.java:38:
private RenderingResults(Status status, String content, String
errorMessage, int httpStatusCode, Uri redirect) {
100char

http://codereview.appspot.com/186233/diff/18/22#newcode57
java/gadgets/src/main/java/org/apache/shindig/gadgets/render/RenderingResults.java:57:
return new RenderingResults(Status.MUST_REDIRECT, null, null,
HttpServletResponse.SC_FOUND, redirect);
same

http://codereview.appspot.com/186233/show

Reply via email to