http://codereview.appspot.com/28042/diff/1/7
File
java/common/src/main/java/org/apache/shindig/protocol/ApiServlet.java
(right):

http://codereview.appspot.com/28042/diff/1/7#newcode56
Line 56:
typo?  application/javascript?

http://codereview.appspot.com/28042/diff/1/7#newcode63
Line 63:
FORMAT_TO_PREFERRED_CONTENT_TYPE maybe better

http://codereview.appspot.com/28042/diff/1/7#newcode67
Line 67:
some Javadoc for what this constant means?

http://codereview.appspot.com/28042/diff/1/7#newcode170
Line 170:
Some javadoc, e.g.
/**
  * Checks if the request uses a valid content types for POST and PUT.
  */

http://codereview.appspot.com/28042

Reply via email to