> 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 <[email protected]> > Sent: 23 November 2018 19:42 > To: Nick Gasson <[email protected]>; Magnus Ihse Bursie > <[email protected]>; build-dev <[email protected]>; > [email protected] > Cc: nd <[email protected]> > 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
