RFR: 8218547: Simplify JLI_Open on Windows in native code (libjli) - was : RE: RFR : 8217093: Support extended-length paths in parse_manifest.c on windows
Langer, Christoph
christoph.langer at sap.com
Thu Mar 28 10:58:35 UTC 2019
Hi Matthias,
the change to src/java.base/windows/native/libjli/java_md.c looks good to me now.
As for the test, I think you could also check a jar in a short path to exercise both cases in JLI_Open.
And a few nits:
Line 506: better do:
Path pelp = pcreated.resolve(elp.jar);
Line 507:
2 spaces are one too much after elp
Line 508:
Insert 2 additional spaces in the source code string, a space between "){" and another in the end after the ; of the System.out.println statement, like this:
createJar(elp, new File("Foo"), "public static void main(String[] args) { System.out.println(\"Hello from ELP\"); }");
You could also do a little cleanup: Remove method private static String removeExtraQuotes(String in) { from line 64. It seems, it is not needed.
Best regards
Christoph
> -----Original Message-----
> From: Baesken, Matthias
> Sent: Mittwoch, 27. März 2019 18:05
> To: Langer, Christoph <christoph.langer at sap.com>
> Cc: core-libs-dev at openjdk.java.net; Alan Bateman
> <Alan.Bateman at oracle.com>
> Subject: RE: RFR: 8218547: Simplify JLI_Open on Windows in native code
> (libjli) - was : RE: RFR : 8217093: Support extended-length paths in
> parse_manifest.c on windows
>
> Hello could you please look into it ?
>
> Best regards, Matthias
>
>
>
> > -----Original Message-----
> > From: Baesken, Matthias
> > Sent: Montag, 25. März 2019 18:05
> > To: Langer, Christoph <christoph.langer at sap.com>
> > Cc: 'core-libs-dev at openjdk.java.net' <core-libs-dev at openjdk.java.net>;
> > 'Alan Bateman' <Alan.Bateman at oracle.com>
> > Subject: RE: RFR: 8218547: Simplify JLI_Open on Windows in native code
> > (libjli) - was : RE: RFR : 8217093: Support extended-length paths in
> > parse_manifest.c on windows
> >
> > Hello here is an updated webrev :
> >
> >
> > http://cr.openjdk.java.net/~mbaesken/webrevs/8218547.1/webrev/
> >
> >
> >
> > > > In JLI_Open(const char* name, int flags), you should remove ret and
> only
> > > > use fd, I think.
> >
> >
> > I removed ret and adjusted some comments.
> >
> > Additionally I added a test that uses (on Windows) JLI_Open on a jar
> file
> > with a "long" path (> 260 chars).
> > [ JLI_Open is currently called from parse_manifest.c with jarfile as
> > parameter ]
> >
More information about the core-libs-dev
mailing list