Very simple code review: 7196677: diff compares same file to itself in PaddingTest regression test.

[Xuelei Fan]

Looks fine to me.

Xuelei

On 9/7/2012 9:46 AM, Weijun Wang wrote:
> http://cr.openjdk.java.net/~weijun/7196677/webrev.00/
> 
> Thanks
> Max
> 
> 
> -------- Original Message --------
> 7196677: diff compares same file to itself in PaddingTest regression test.
> 
> 
> === *Description*
> ============================================================
> Introduced due to 6330275
> Changeset http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c8dbb9e19355
> 
> + private static void diff(String fname1, String fname2) throws Exception {
> + if (!Arrays.equals(Files.readAllBytes(Paths.get(fname1)),
> + Files.readAllBytes(Paths.get(fname1)))) {
> + throw new Exception(
> + "files " + fname1 + " and " + fname2 + " differ");
> + }
> + }
> 
> 
> There is a error in the diff, reads n compares fname1 with itself,
> instead of fname2
>