LGTM

Thanks Satya!

On 2010/10/18 07:51:04, satya3656 wrote:

http://codereview.appspot.com/2322042/diff/41001/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/BaseOptimizer.java
File

java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/BaseOptimizer.java
(right):


http://codereview.appspot.com/2322042/diff/41001/java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/BaseOptimizer.java#newcode185

java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/BaseOptimizer.java:185:
writeParam);
On 2010/10/14 23:20:37, johnfargo wrote:
> Could you use this same method call signature at line 190? If so,
I'd
recommend
> doing:
> ImageWriteParam metadataWriteParam = writeParam;
> if (writer instanceof JPEGImageWriter) {
>   // Comment about workaround
>   metadataWriteParam = null;
> }
> writer.write(...);

Done.



http://codereview.appspot.com/2322042/

Reply via email to