Re: RFR: 8267706: bin/idea.sh tries to use cygpath on WSL [v2]

2021-06-01 Thread Jonathan Gibbons
On Thu, 27 May 2021 17:45:40 GMT, Nikita Gubarkov wrote: >> 8267706: bin/idea.sh tries to use cygpath on WSL > > Nikita Gubarkov has updated the pull request incrementally with one > additional commit since the last revision: > > 8267706: Break long line in make/ide/idea/jdk/idea.gmk The fi

RFR: 8265440: IGV: make node selection more visible

2021-06-01 Thread Koichi Sakata
This pull request makes node selection more visible. At present when selecting node, node name is bolder. In addition to that, thickness of the border is bolder after applying this code. I tested manually. Here are the images that I took before and after applying this patch as follows. ### Aft

Re: RFR: 8267706: bin/idea.sh tries to use cygpath on WSL [v2]

2021-06-01 Thread Nikita Gubarkov
On Wed, 2 Jun 2021 00:19:56 GMT, Jonathan Gibbons wrote: >> @jonathan-gibbons this can be fixed with `brew install coreutils`. We >> probably need to check `realpath` availability in idea.sh and suggest >> installing `coreutils` if it's not available > > @YaaZ I'm aware of the workaround, but t

Re: RFR: 8267706: bin/idea.sh tries to use cygpath on WSL [v2]

2021-06-01 Thread Jonathan Gibbons
On Tue, 1 Jun 2021 22:20:25 GMT, Nikita Gubarkov wrote: >> The fix fails on a Mac, where `realpath` is not available by default. > > @jonathan-gibbons this can be fixed with `brew install coreutils`. We > probably need to check `realpath` availability in idea.sh and suggest > installing `coreu

Re: RFR: 8267706: bin/idea.sh tries to use cygpath on WSL [v2]

2021-06-01 Thread Nikita Gubarkov
On Tue, 1 Jun 2021 22:10:41 GMT, Jonathan Gibbons wrote: >> Nikita Gubarkov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8267706: Break long line in make/ide/idea/jdk/idea.gmk > > The fix fails on a Mac, where `realpath` is not availa

Re: RFR: 8267706: bin/idea.sh tries to use cygpath on WSL [v2]

2021-06-01 Thread Jonathan Gibbons
On Thu, 27 May 2021 17:45:40 GMT, Nikita Gubarkov wrote: >> 8267706: bin/idea.sh tries to use cygpath on WSL > > Nikita Gubarkov has updated the pull request incrementally with one > additional commit since the last revision: > > 8267706: Break long line in make/ide/idea/jdk/idea.gmk The fi

Re: RFR: 8262891: Compiler implementation for Pattern Matching for switch (Preview) [v8]

2021-06-01 Thread Maurizio Cimadamore
On Tue, 1 Jun 2021 07:50:49 GMT, Jan Lahoda wrote: >> This is a preview of a patch implementing JEP 406: Pattern Matching for >> switch (Preview): >> https://bugs.openjdk.java.net/browse/JDK-8213076 >> >> The current draft of the specification is here: >> http://cr.openjdk.java.net/~gbierman/je

Integrated: 8265431: Add -fno-delete-null-pointer-checks to clang builds

2021-06-01 Thread Maxim Kartashev
On Mon, 24 May 2021 14:03:45 GMT, Maxim Kartashev wrote: > This commit adds the `-fno-delete-null-pointer-checks` compiler option to > clang builds, which is going to become necessary starting from clang version > 12.0.0 (see the [bug > report](https://bugs.openjdk.java.net/browse/JDK-8265431

Re: RFR: 8265431: Add -fno-delete-null-pointer-checks to clang builds

2021-06-01 Thread Maxim Kartashev
On Tue, 1 Jun 2021 13:23:49 GMT, Erik Joelsson wrote: >> This commit adds the `-fno-delete-null-pointer-checks` compiler option to >> clang builds, which is going to become necessary starting from clang version >> 12.0.0 (see the [bug >> report](https://bugs.openjdk.java.net/browse/JDK-8265431

Re: RFR: 8265431: Add -fdelete-null-pointer-checks to clang builds

2021-06-01 Thread Erik Joelsson
On Mon, 24 May 2021 14:03:45 GMT, Maxim Kartashev wrote: > This commit adds the `-fno-delete-null-pointer-checks` compiler option to > clang builds, which is going to become necessary starting from clang version > 12.0.0 (see the [bug > report](https://bugs.openjdk.java.net/browse/JDK-8265431

Re: RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v8]

2021-06-01 Thread Joe Wang
On Tue, 1 Jun 2021 15:21:33 GMT, Weijun Wang wrote: >> Please review this implementation of [JEP >> 411](https://openjdk.java.net/jeps/411). >> >> The code change is divided into 3 commits. Please review them one by one. >> >> 1. >> https://github.com/openjdk/jdk/commit/576161d15423f58281e384

Re: RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v8]

2021-06-01 Thread Alan Bateman
On Tue, 1 Jun 2021 15:21:33 GMT, Weijun Wang wrote: >> Please review this implementation of [JEP >> 411](https://openjdk.java.net/jeps/411). >> >> The code change is divided into 3 commits. Please review them one by one. >> >> 1. >> https://github.com/openjdk/jdk/commit/576161d15423f58281e384

Re: RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v8]

2021-06-01 Thread Weijun Wang
> Please review this implementation of [JEP > 411](https://openjdk.java.net/jeps/411). > > The code change is divided into 3 commits. Please review them one by one. > > 1. > https://github.com/openjdk/jdk/commit/576161d15423f58281e384174d28c9f9be7941a1 > The essential change for this JEP, incl

Re: RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v7]

2021-06-01 Thread Weijun Wang
> Please review this implementation of [JEP > 411](https://openjdk.java.net/jeps/411). > > The code change is divided into 3 commits. Please review them one by one. > > 1. > https://github.com/openjdk/jdk/commit/576161d15423f58281e384174d28c9f9be7941a1 > The essential change for this JEP, incl

Re: RFR: 8265431: Add -fdelete-null-pointer-checks to clang builds

2021-06-01 Thread Maxim Kartashev
On Tue, 1 Jun 2021 13:23:49 GMT, Erik Joelsson wrote: > I left a comment in the bug that you haven't answered. Sorry, don't have write access to the bug. To answer your comment > You seem to be mixing up -fno-delete-null-pointer-checks and > -fdelete-null-pointer-checks between the descriptio

Re: RFR: 8265431: Add -fdelete-null-pointer-checks to clang builds

2021-06-01 Thread Erik Joelsson
On Mon, 24 May 2021 14:03:45 GMT, Maxim Kartashev wrote: > This commit adds the `-fno-delete-null-pointer-checks` compiler option to > clang builds, which is going to become necessary starting from clang version > 12.0.0 (see the [bug > report](https://bugs.openjdk.java.net/browse/JDK-8265431

Re: RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v6]

2021-06-01 Thread Alan Bateman
On Mon, 31 May 2021 15:02:57 GMT, Weijun Wang wrote: >> Please review this implementation of [JEP >> 411](https://openjdk.java.net/jeps/411). >> >> The code change is divided into 3 commits. Please review them one by one. >> >> 1. >> https://github.com/openjdk/jdk/commit/576161d15423f58281e38

Re: RFR: 8262891: Compiler implementation for Pattern Matching for switch (Preview) [v8]

2021-06-01 Thread Jan Lahoda
> This is a preview of a patch implementing JEP 406: Pattern Matching for > switch (Preview): > https://bugs.openjdk.java.net/browse/JDK-8213076 > > The current draft of the specification is here: > http://cr.openjdk.java.net/~gbierman/jep406/jep406-20210430/specs/patterns-switch-jls.html > > A