Hi Thorsten, I saw your other mail already but didn't find time to reply.
I'm actually not convinced that it is a good idea to add ERROR_NO_MORE_FILES to lastErrorReportable. The error codes listed there are conditions on which canonicalization of a path is stopped but the result is deemed correct. E.g. if the path only exists up to a certain directory, one can assume the rest of the path is canonic. Or if there are conditions like network errors or access denied, then further canonicalization isn't possible, too. However, your case, the sporadic ERROR_NO_MORE_FILES, needs to be understood first. I rather think if this happens, there's a real condition for an IOException. It should definitely be analyzed and understood what the reason is for ERROR_NO_MORE_FILES. Are you aware of other reports of this issue? Was this already analyzed by some Windows experts, e.g. Microsoft support? Best regards Christoph > -----Original Message----- > From: Thorsten Schöning <tschoen...@am-soft.de> > Sent: Montag, 18. November 2019 14:09 > To: core-libs-dev@openjdk.java.net > Cc: Langer, Christoph <christoph.lan...@sap.com>; hotspot-runtime- > d...@openjdk.java.net; gerard ziemski <gerard.ziem...@oracle.com> > Subject: Re: RFR: 8234185: Cleanup usage of canonicalize function between > libjava, hotspot and libinstrument > > Guten Tag Langer, Christoph, > am Donnerstag, 14. November 2019 um 16:37 schrieben Sie: > > > please review this cleanup change regarding function "canonicalize" of > libjava. > [...] > > The goal is to cleanup how this function is defined and used.[...] > > If you are already changing "lastErrorReportable" for Windows, how > about adding ERROR_NO_MORE_FILES there as well to not run into > unnecessary exceptions under some circumstances? > > https://mail.openjdk.java.net/pipermail/core-libs-dev/2019- > November/063437.html > https://stackoverflow.com/questions/58825588/does-java-need-to- > support-error-no-more-files-when-canonicalizing-paths-on-windo > https://stackoverflow.com/questions/58825963/when-does-findfirstfilew- > set-last-error-to-be-error-no-more-files-instead-of-err?noredirect=1&lq=1 > > Mit freundlichen Grüßen, > > Thorsten Schöning > > -- > Thorsten Schöning E-Mail: thorsten.schoen...@am-soft.de > AM-SoFT IT-Systeme http://www.AM-SoFT.de/ > > Telefon...........05151- 9468- 55 > Fax...............05151- 9468- 88 > Mobil..............0178-8 9468- 04 > > AM-SoFT GmbH IT-Systeme, Brandenburger Str. 7c, 31789 Hameln > AG Hannover HRB 207 694 - Geschäftsführer: Andreas Muchow
