LGTM.

http://codereview.appspot.com/33061/diff/1/7
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/FetchResponseUtils.java
(right):

http://codereview.appspot.com/33061/diff/1/7#newcode72
Line 72: Collection<String> values = response.getHeaders(name);
Looks like response.getHeaders is guaranteed immutable by the
HttpResponse contract.  Might be nice to add a comment to that effect.

http://codereview.appspot.com/33061

Reply via email to