Looks ok
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com Sent from my iPad On Nov 19, 2013, at 6:57 PM, Dan Xu <dan...@oracle.com> wrote: > Hi All, > > Please review the simple fix towards Size.java testcase. It failed once on > windows platform in the recent same binary run, which is mostly due to some > interferences and the special delete handling on windows. > > In the fix, I remove the delete operation in initTestFile() method because > FileOutputStream will truncate the file content and it is not necessary to > delete it first. Thanks! > > Bug:https://bugs.openjdk.java.net/browse/JDK-8028628 > Webrev: http://cr.openjdk.java.net/~dxu/8028628/webrev/ > <http://cr.openjdk.java.net/%7Edxu/8028628/webrev/> > > -Dan