one very quick q.

http://codereview.appspot.com/816045/diff/1/3
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/http/HttpResponseBuilder.java
(right):

http://codereview.appspot.com/816045/diff/1/3#newcode79
java/gadgets/src/main/java/org/apache/shindig/gadgets/http/HttpResponseBuilder.java:79:
public HttpResponseBuilder setEncoding(Charset charset) {
looks like we intend to make this an implementation detail.
package-private for testing purposes?

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

Reply via email to