> If you can then it would be great, if only to save others from looking > at it and wondering if it should also be changed. Maybe for another > issue but there are several other usages in the java launcher that > should be looked at.
Hi Alan, I've done this here: http://cr.openjdk.java.net/~njian/8214077/webrev.3/ Thanks, Nick > -----Original Message----- > From: Alan Bateman <alan.bate...@oracle.com> > Sent: 23 November 2018 19:42 > To: Nick Gasson <nick.gas...@arm.com>; Magnus Ihse Bursie > <magnus.ihse.bur...@oracle.com>; build-dev <build-...@openjdk.java.net>; > core-libs-dev@openjdk.java.net > Cc: nd <n...@arm.com> > Subject: Re: RFR: 8214077: test java/io/File/SetLastModified.java fails on > ARM32 > > On 23/11/2018 11:13, Nick Gasson wrote: > >> This looks right and reduces the stat usages in libjava down to one > >> remaining case (ProcessHandlerImpl_linux.c). > > Hi Alan, > > > > Do you want me to do this one too for completeness? Although as > > it's calling stat() on the /proc/<pid> directory I don't think it's > > possible to > > hit the EOVERFLOW case. > If you can then it would be great, if only to save others from looking > at it and wondering if it should also be changed. Maybe for another > issue but there are several other usages in the java launcher that > should be looked at. > > -Alan
