Small fix to make JpegImageUtils backward compatible with java 1.5

Small fix to make JpegImageUtils.java backward compatible with java 1.5

[satya3656, 2010-12-17 10:40:05]

+anupama.dutta

On 17 December 2010 15:22, <satya3656@gmail.com> wrote:

> Reviewers: gagan.goku,
>
> Description:
> Small fix to make JpegImageUtils.java backward compatible with java 1.5
>
> Please review this at http://codereview.appspot.com/3739041/
>
> Affected files:
>
> 
java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/JpegImageUtils.java
>
>
> Index:
>
java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/JpegImageUtils.java
> ===================================================================
> ---
>
java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/JpegImageUtils.java
>     (revision 1050317)
> +++
>
java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/image/JpegImageUtils.java
>     (working copy)
> @@ -168,7 +168,7 @@
>      */
>     public void addQTable(int tableIndex, int[] table) {
>       if (tableIndex == 0 || tableIndex == 1) {
> -        tables[tableIndex] = Arrays.copyOf(table, table.length);
> +        System.arraycopy(table, 0, tables[tableIndex], 0, table.length);
>       }
>     }
>
> @@ -387,4 +387,4 @@
>     new JpegUtils().traverseJFIF(new ByteSourceInputStream(is, filename),
> visitor);
>     return imageParams;
>   }
> -}
> \ No newline at end of file
> +}
>
>
>

[anupama.dutta, 2010-12-17 10:46:13]

LGTM.

[gagan.goku, 2010-12-17 11:42:06]

On 2010/12/17 10:46:13, anupama.dutta wrote:
> LGTM.

LGTM. Submitting as anupama tested it with java 1.5

[gagan.goku, 2010-12-17 11:50:44]

Build looks good.
Submitted as r1050357.