RFR : 8217093: Support extended-length paths in parse_manifest.c on windows

Alan Bateman Alan.Bateman at oracle.com
Sun Jan 27 15:57:09 UTC 2019


On 24/01/2019 16:19, Baesken, Matthias wrote:
>> Did you consider adding a Unix version of open_jarfile to avoid #ifdef
>> WIN32 ?
> Hi Alan, do you mean , adding a separate c-file for UNIX  containing the function  open_jarfile ?
> I considered this, but did not like the idea to add a separate file for UNIX just for the small function .
>
> Your  suggestion to use  FILE_SHARE_READ  is probably the right way to do it , I  changed this, second webrev :
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8217093.2/
>
Christoph Langer's suggestion to use jli_util seems worth exploring to 
keep the platform specific code together.

I still find the call to _open and checking for ENOENT a bit icky. If 
this change gets pushed then I think we need a follow-up issue created 
to replace this to use CreateFile consistently.

-Alan


More information about the core-libs-dev mailing list