Integrated: 8249867: xml declaration is not followed by a newline

2021-01-29 Thread Joe Wang
On Tue, 12 Jan 2021 01:24:38 GMT, Joe Wang wrote: > Please review a patch to add an explicit control over whether a newline > should be added after the XML header. This is done by adding a DOM > LSSerializer property "jdk-is-standalone" and System property > "jdk.xml.isStandalone". > > This

Re: RFR: 8254702: jpackage app launcher crashes on CentOS [v2]

2021-01-29 Thread Alexander Matveev
On Fri, 29 Jan 2021 23:06:20 GMT, Alexey Semenyuk wrote: >> Fix for https://bugs.openjdk.java.net/browse/JDK-8254702 >> >> The fix splits Linux app launcher in app launcher and launcher shared lib. >> App launcher is pure C and doesn't have C++ code. App launcher lib >> incorporates bulk of

Re: RFR: JDK-8257086: Clarify differences between {Float, Double}.equals and == [v5]

2021-01-29 Thread Joe Darcy
> Updates to the specifications of Double.{equals, compareTo} to explain more > explicitly why the obvious wrappers around "==" and "<"/"==" are not > sufficient for floating-point values. > > Once the wording is worked out, I'll replicate it for the analogous methods > on Float. Joe Darcy

Re: RFR: JDK-8257086: Clarify differences between {Float, Double}.equals and == [v4]

2021-01-29 Thread Joe Darcy
On Tue, 8 Dec 2020 17:10:33 GMT, Brian Burkhalter wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Editing pass on changes in Double; add accommodations in Float. > > src/java.base/share/classes/java/lang/Double.java

Re: bug jpackage in JDK 15

2021-01-29 Thread Michael Hall
> On Jan 26, 2021, at 7:16 PM, Michael Hall wrote: > >> >> >> When I open the built dmg there is an application icon, a drag to indicating >> arrow, but no application folder icon as the drag target. >> >> Possibly related… >> >> Running [osascript, >>

Re: RFR: JDK-8257086: Clarify differences between {Float, Double}.equals and == [v4]

2021-01-29 Thread Joe Darcy
> Updates to the specifications of Double.{equals, compareTo} to explain more > explicitly why the obvious wrappers around "==" and "<"/"==" are not > sufficient for floating-point values. > > Once the wording is worked out, I'll replicate it for the analogous methods > on Float. Joe Darcy

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m [v3]

2021-01-29 Thread Phil Race
On Fri, 29 Jan 2021 22:47:52 GMT, Weijun Wang wrote: >> This fix covers both >> >> - [[macOS]: Remove JNF dependency from >> libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858) >> - [[macOS]: Remove JNF dependency from >>

Re: RFR: 8260617: Merge ZipFile encoding check with the initial hash calculation [v2]

2021-01-29 Thread Claes Redestad
On Fri, 29 Jan 2021 18:43:40 GMT, Lance Andersen wrote: >> Claes Redestad has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Refactor to do most entry validation in one place, out of line from >> initCEN. > >

Re: RFR: 8260617: Merge ZipFile encoding check with the initial hash calculation [v2]

2021-01-29 Thread Claes Redestad
> - Merge checkEncoding into the byte[]-based normalizedHash. The latter is > only used from ZipFile.initCEN right after the checkEncoding today, so > verifying this is equivalent is straightforward. > - Factor out the logic to calculate hash, check encoding etc into the > addEntry method to

Re: RFR: 8258917: NativeMemoryTracking is handled by launcher inconsistenly [v3]

2021-01-29 Thread Alex Menkov
On Thu, 28 Jan 2021 13:45:15 GMT, Zhengyu Gu wrote: >> Alex Menkov has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Updated comment > > Sorry, I overlooked some of details. Final change looks fine to me. @zhengyu123 Thank you for

Re: RFR: 8254702: jpackage app launcher crashes on CentOS [v2]

2021-01-29 Thread Alexey Semenyuk
> Fix for https://bugs.openjdk.java.net/browse/JDK-8254702 > > The fix splits Linux app launcher in app launcher and launcher shared lib. > App launcher is pure C and doesn't have C++ code. App launcher lib > incorporates bulk of C++ code from app launcher. > At startup app launcher loads

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m [v3]

2021-01-29 Thread Weijun Wang
> This fix covers both > > - [[macOS]: Remove JNF dependency from > libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858) > - [[macOS]: Remove JNF dependency from > libosxkrb5/SCDynamicStoreConfig.m](https://bugs.openjdk.java.net/browse/JDK-8257860) Weijun Wang has

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m [v2]

2021-01-29 Thread Weijun Wang
On Fri, 29 Jan 2021 22:05:21 GMT, Phil Race wrote: >> So I should remember how env was retrieved and only detach when it's from >> `AttachCurrentThreadAsDaemon`? In fact, in my test the plain `GetEnv` has >> never succeeded and it was always attached. > > Yes that is the idea. BTW which thread

Re: RFR: 8254702: jpackage app launcher crashes on CentOS

2021-01-29 Thread Alexey Semenyuk
On Fri, 29 Jan 2021 22:00:32 GMT, Phil Race wrote: >> Fix for https://bugs.openjdk.java.net/browse/JDK-8254702 >> >> The fix splits Linux app launcher in app launcher and launcher shared lib. >> App launcher is pure C and doesn't have C++ code. App launcher lib >> incorporates bulk of C++

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m [v2]

2021-01-29 Thread Phil Race
On Fri, 29 Jan 2021 21:47:32 GMT, Weijun Wang wrote: >> src/java.base/macosx/native/libosxsecurity/KeystoreImpl.m line 619: >> >>> 617: (*env)->ReleaseCharArrayElements(env, passwordObj, >>> passwordChars, >>> 618: JNI_ABORT); >>> 619: } >> >> Although you

Re: RFR: 8254702: jpackage app launcher crashes on CentOS

2021-01-29 Thread Phil Race
On Fri, 29 Jan 2021 21:33:40 GMT, Alexey Semenyuk wrote: > Fix for https://bugs.openjdk.java.net/browse/JDK-8254702 > > The fix splits Linux app launcher in app launcher and launcher shared lib. > App launcher is pure C and doesn't have C++ code. App launcher lib > incorporates bulk of C++

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m [v2]

2021-01-29 Thread Weijun Wang
On Fri, 29 Jan 2021 20:51:04 GMT, Phil Race wrote: >> Weijun Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> same behavior as before -- empty realm map > > make/modules/java.security.jgss/Lib.gmk line 84: > >> 82:

Re: RFR: 8260617: Merge ZipFile encoding check with the initial hash calculation

2021-01-29 Thread Lance Andersen
On Fri, 29 Jan 2021 00:54:46 GMT, Claes Redestad wrote: > - Merge checkEncoding into the byte[]-based normalizedHash. The latter is > only used from ZipFile.initCEN right after the checkEncoding today, so > verifying this is equivalent is straightforward. > - Factor out the logic to calculate

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m [v2]

2021-01-29 Thread Phil Race
On Fri, 29 Jan 2021 14:57:56 GMT, Weijun Wang wrote: >> This fix covers both >> >> - [[macOS]: Remove JNF dependency from >> libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858) >> - [[macOS]: Remove JNF dependency from >>

Re: RFR: 8249867: xml declaration is not followed by a newline [v3]

2021-01-29 Thread Joe Wang
On Fri, 29 Jan 2021 18:40:55 GMT, Lance Andersen wrote: >> src/java.xml/share/classes/module-info.java line 78: >> >>> 76: * System Properties >>> 77: * A property may have a corresponding System Property that has the >>> same name >>> 78: * except the prefix as shown above. A System

Re: RFR: 8249867: xml declaration is not followed by a newline [v3]

2021-01-29 Thread Joe Wang
On Fri, 29 Jan 2021 17:57:14 GMT, Daniel Fuchs wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Updated the patch based on review comments. Refer to the previous reviews. > > src/java.xml/share/classes/module-info.java

Re: RFR: 8249867: xml declaration is not followed by a newline [v4]

2021-01-29 Thread Daniel Fuchs
On Fri, 29 Jan 2021 18:24:59 GMT, Joe Wang wrote: >> Please review a patch to add an explicit control over whether a newline >> should be added after the XML header. This is done by adding a DOM >> LSSerializer property "jdk-is-standalone" and System property >> "jdk.xml.isStandalone". >>

Re: RFR: JDK-8257086: Clarify differences between {Float, Double}.equals and == [v3]

2021-01-29 Thread Stuart Marks
On Fri, 29 Jan 2021 17:31:09 GMT, Joe Darcy wrote: >> Updates to the specifications of Double.{equals, compareTo} to explain more >> explicitly why the obvious wrappers around "==" and "<"/"==" are not >> sufficient for floating-point values. >> >> Once the wording is worked out, I'll

Re: RFR: 8249867: xml declaration is not followed by a newline [v3]

2021-01-29 Thread Lance Andersen
On Fri, 29 Jan 2021 17:52:33 GMT, Daniel Fuchs wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Updated the patch based on review comments. Refer to the previous reviews. > > src/java.xml/share/classes/module-info.java

Re: RFR: 8249867: xml declaration is not followed by a newline [v4]

2021-01-29 Thread Joe Wang
> Please review a patch to add an explicit control over whether a newline > should be added after the XML header. This is done by adding a DOM > LSSerializer property "jdk-is-standalone" and System property > "jdk.xml.isStandalone". > > This change addresses an incompatibility introduced

Re: RFR: 8249867: xml declaration is not followed by a newline [v3]

2021-01-29 Thread Daniel Fuchs
On Fri, 29 Jan 2021 00:07:59 GMT, Joe Wang wrote: >> Please review a patch to add an explicit control over whether a newline >> should be added after the XML header. This is done by adding a DOM >> LSSerializer property "jdk-is-standalone" and System property >> "jdk.xml.isStandalone". >>

Re: RFR: 8249867: xml declaration is not followed by a newline [v3]

2021-01-29 Thread Roger Riggs
On Fri, 29 Jan 2021 00:07:59 GMT, Joe Wang wrote: >> Please review a patch to add an explicit control over whether a newline >> should be added after the XML header. This is done by adding a DOM >> LSSerializer property "jdk-is-standalone" and System property >> "jdk.xml.isStandalone". >>

Re: RFR: 8249867: xml declaration is not followed by a newline [v3]

2021-01-29 Thread Naoto Sato
On Fri, 29 Jan 2021 00:07:59 GMT, Joe Wang wrote: >> Please review a patch to add an explicit control over whether a newline >> should be added after the XML header. This is done by adding a DOM >> LSSerializer property "jdk-is-standalone" and System property >> "jdk.xml.isStandalone". >>

Re: RFR: JDK-8257086: Clarify differences between {Float, Double}.equals and == [v3]

2021-01-29 Thread Joe Darcy
> Updates to the specifications of Double.{equals, compareTo} to explain more > explicitly why the obvious wrappers around "==" and "<"/"==" are not > sufficient for floating-point values. > > Once the wording is worked out, I'll replicate it for the analogous methods > on Float. Joe Darcy

Re: RFR: 8249867: xml declaration is not followed by a newline [v3]

2021-01-29 Thread Lance Andersen
On Fri, 29 Jan 2021 00:07:59 GMT, Joe Wang wrote: >> Please review a patch to add an explicit control over whether a newline >> should be added after the XML header. This is done by adding a DOM >> LSSerializer property "jdk-is-standalone" and System property >> "jdk.xml.isStandalone". >>

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m [v2]

2021-01-29 Thread Weijun Wang
> This fix covers both > > - [[macOS]: Remove JNF dependency from > libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858) > - [[macOS]: Remove JNF dependency from > libosxkrb5/SCDynamicStoreConfig.m](https://bugs.openjdk.java.net/browse/JDK-8257860) Weijun Wang has

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m

2021-01-29 Thread Erik Joelsson
On Fri, 18 Dec 2020 19:20:47 GMT, Weijun Wang wrote: > This fix covers both > > - [[macOS]: Remove JNF dependency from > libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858) > - [[macOS]: Remove JNF dependency from >

RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m

2021-01-29 Thread Weijun Wang
This fix covers both - [[macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858) - [[macOS]: Remove JNF dependency from libosxkrb5/SCDynamicStoreConfig.m](https://bugs.openjdk.java.net/browse/JDK-8257860) - Commit

Re: RFR: 8257858: [macOS]: Remove JNF dependency from libosxsecurity/KeystoreImpl.m

2021-01-29 Thread Weijun Wang
On Fri, 18 Dec 2020 19:20:47 GMT, Weijun Wang wrote: > This fix covers both > > - [[macOS]: Remove JNF dependency from > libosxsecurity/KeystoreImpl.m](https://bugs.openjdk.java.net/browse/JDK-8257858) > - [[macOS]: Remove JNF dependency from >

Integrated: 8260592: jpackage tests fail when Desktop is not supported

2021-01-29 Thread Aleksey Shipilev
On Thu, 28 Jan 2021 14:33:35 GMT, Aleksey Shipilev wrote: > If you run x86 configuration (cross-compiled on x86_64), then many tests > would fail with: > > $ CONF=linux-x86-server-fastdebug make images run-test TEST=tools/jpackage > ... > can not load libgnomevfs-2.so > Exception in thread

Re: RFR: 8260592: jpackage tests fail when Desktop is not supported [v2]

2021-01-29 Thread Alexey Semenyuk
On Fri, 29 Jan 2021 07:31:55 GMT, Aleksey Shipilev wrote: >> If you run x86 configuration (cross-compiled on x86_64), then many tests >> would fail with: >> >> $ CONF=linux-x86-server-fastdebug make images run-test TEST=tools/jpackage >> ... >> can not load libgnomevfs-2.so >> Exception in

Re: RFR: JDK-8183372 : Refactor java/lang/Class shell tests to java [v6]

2021-01-29 Thread Mahendra Chhipa
> https://bugs.openjdk.java.net/browse/JDK-8183372 Mahendra Chhipa has updated the pull request incrementally with one additional commit since the last revision: Reset the formatting. - Changes: - all: https://git.openjdk.java.net/jdk/pull/2170/files - new:

Re: RFR: 8260617: Merge ZipFile encoding check with the initial hash calculation

2021-01-29 Thread Claes Redestad
On Fri, 29 Jan 2021 00:54:46 GMT, Claes Redestad wrote: > - Merge checkEncoding into the byte[]-based normalizedHash. The latter is > only used from ZipFile.initCEN right after the checkEncoding today, so > verifying this is equivalent is straightforward. > - Factor out the logic to calculate

RFR: 8260617: Merge ZipFile encoding check with the initial hash calculation

2021-01-29 Thread Claes Redestad
- Merge checkEncoding into the byte[]-based normalizedHash. The latter is only used from ZipFile.initCEN right after the checkEncoding today, so verifying this is equivalent is straightforward. - Factor out the logic to calculate hash, check encoding etc into the addEntry method to allow JITs