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 >
