Hello Alan,  here is a second webrev :

http://cr.openjdk.java.net/~mbaesken/webrevs/8217093.1/

I moved the  Windows-only code into a new file 

src/java.base/windows/native/libjli/parse_manifest_md.c


Best regards, Matthias


> -----Original Message-----
> From: Alan Bateman <[email protected]>
> Sent: Mittwoch, 16. Januar 2019 16:43
> To: Baesken, Matthias <[email protected]>; core-libs-
> [email protected]
> Subject: Re: RFR : 8217093: Support extended-length paths in
> parse_manifest.c on windows
> 
> On 16/01/2019 14:43, Baesken, Matthias wrote:
> > Hi Alan , do you think it is good to start  a UNIX   parse_manifest_md.c too
> with just a one-liner in it ?
> > Or do you prefer  only having the windows - file with the Windows coding
> and  keeping the UNIX one-liner    for   open_jarfile   in   parse_manifest.c 
> ?
> >
> I think this makes sense.
> 
> -Alan

Reply via email to