Re: RFR: 8332403: Anachronistic reference to Netscape Communicator in Swing API docs [v2]

2024-05-17 Thread Phil Race
On Fri, 17 May 2024 14:04:28 GMT, Alexey Ivanov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - doc clarity >> - doc clarity > > src/java.desktop/share/classes/javax/swing/CellEditor.java line 91: > >>

Re: RFR: 8332403: Anachronistic reference to Netscape Communicator in Swing API docs [v2]

2024-05-17 Thread Phil Race
On Fri, 17 May 2024 12:16:15 GMT, Prasanta Sadhukhan wrote: >> Inadvertent mention of Netscape in Javadoc is removed.. > > Prasanta Sadhukhan has updated the pull request incrementally with two > additional commits since the last revision: > > - doc clarity > - doc clarity Marked as

Re: RFR: 8332403: Anachronistic reference to Netscape Communicator in Swing API docs [v2]

2024-05-17 Thread Alexey Ivanov
On Fri, 17 May 2024 12:16:15 GMT, Prasanta Sadhukhan wrote: >> Inadvertent mention of Netscape in Javadoc is removed.. > > Prasanta Sadhukhan has updated the pull request incrementally with two > additional commits since the last revision: > > - doc clarity > - doc clarity Marked as

Re: RFR: 8331671: Implement JEP 472: Prepare to Restrict the Use of JNI [v8]

2024-05-17 Thread Maurizio Cimadamore
> This PR implements [JEP 472](https://openjdk.org/jeps/472), by restricting > the use of JNI in the following ways: > > * `System::load` and `System::loadLibrary` are now restricted methods > * `Runtime::load` and `Runtime::loadLibrary` are now restricted methods > * binding a JNI `native`

Re: RFR: 8331671: Implement JEP 472: Prepare to Restrict the Use of JNI [v7]

2024-05-17 Thread Maurizio Cimadamore
On Thu, 16 May 2024 18:39:57 GMT, Alan Bateman wrote: >> Maurizio Cimadamore has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Add note on --illegal-native-access default value in the launcher help > >

Re: RFR: 8321428: Deprecate for removal the package java.beans.beancontext [v5]

2024-05-17 Thread Alexey Ivanov
On Wed, 15 May 2024 17:39:16 GMT, Larry Cable wrote: >> the beancontext package was added (by me) in JDK 1.2 to provide >> JavaBeans(tm) with a containment and services hierarchy. >> >> based upon concepts from OpenDoc, which was a popular component model at the >> time, the API pre-dated the

Re: RFR: 8332403: Anachronistic reference to Netscape Communicator in Swing API docs [v2]

2024-05-17 Thread Prasanta Sadhukhan
> Inadvertent mention of Netscape in Javadoc is removed.. Prasanta Sadhukhan has updated the pull request incrementally with two additional commits since the last revision: - doc clarity - doc clarity - Changes: - all: https://git.openjdk.org/jdk/pull/19276/files - new:

Integrated: 8321428: Deprecate for removal the package java.beans.beancontext

2024-05-17 Thread Larry Cable
On Mon, 1 Apr 2024 22:53:12 GMT, Larry Cable wrote: > the beancontext package was added (by me) in JDK 1.2 to provide JavaBeans(tm) > with a containment and services hierarchy. > > based upon concepts from OpenDoc, which was a popular component model at the > time, the API pre-dated the

Re: RFR: 8332403: Anachronistic reference to Netscape Communicator in Swing API docs

2024-05-17 Thread Alexey Ivanov
On Fri, 17 May 2024 03:37:21 GMT, Prasanta Sadhukhan wrote: > Inadvertent mention of Netscape in Javadoc is removed.. Changes requested by aivanov (Reviewer). src/java.desktop/share/classes/javax/swing/CellEditor.java line 91: > 89: * cell should be selected. However, it is useful to

Re: RFR: 8331671: Implement JEP 472: Prepare to Restrict the Use of JNI [v7]

2024-05-17 Thread Alan Bateman
On Thu, 16 May 2024 12:23:44 GMT, Maurizio Cimadamore wrote: >> This PR implements [JEP 472](https://openjdk.org/jeps/472), by restricting >> the use of JNI in the following ways: >> >> * `System::load` and `System::loadLibrary` are now restricted methods >> * `Runtime::load` and

Re: RFR: 8331671: Implement JEP 472: Prepare to Restrict the Use of JNI [v7]

2024-05-17 Thread Alan Bateman
On Thu, 16 May 2024 12:23:44 GMT, Maurizio Cimadamore wrote: >> This PR implements [JEP 472](https://openjdk.org/jeps/472), by restricting >> the use of JNI in the following ways: >> >> * `System::load` and `System::loadLibrary` are now restricted methods >> * `Runtime::load` and

Re: RFR: 8331671: Implement JEP 472: Prepare to Restrict the Use of JNI [v7]

2024-05-17 Thread Alan Bateman
On Thu, 16 May 2024 12:23:44 GMT, Maurizio Cimadamore wrote: >> This PR implements [JEP 472](https://openjdk.org/jeps/472), by restricting >> the use of JNI in the following ways: >> >> * `System::load` and `System::loadLibrary` are now restricted methods >> * `Runtime::load` and

Re: RFR: 8331671: Implement JEP 472: Prepare to Restrict the Use of JNI [v7]

2024-05-17 Thread Alan Bateman
On Thu, 16 May 2024 12:23:44 GMT, Maurizio Cimadamore wrote: >> This PR implements [JEP 472](https://openjdk.org/jeps/472), by restricting >> the use of JNI in the following ways: >> >> * `System::load` and `System::loadLibrary` are now restricted methods >> * `Runtime::load` and

Re: RFR: 8075917: The regression-swing case failed as the text on label is not painted red with the GTK L [v8]

2024-05-17 Thread Abhishek Kumar
> JLabel text is not painted with the LAF defined foreground color in GTK LAF. > In GTK LAF the foreground color is retrieved by using native system APIs. Fix > is to return the foreground color if it is set by LAF defined property > otherwise return the default color by calling native APIs. >

Re: RFR: 8075917: The regression-swing case failed as the text on label is not painted red with the GTK L [v7]

2024-05-17 Thread Abhishek Kumar
> JLabel text is not painted with the LAF defined foreground color in GTK LAF. > In GTK LAF the foreground color is retrieved by using native system APIs. Fix > is to return the foreground color if it is set by LAF defined property > otherwise return the default color by calling native APIs. >

Re: RFR: 8075917: The regression-swing case failed as the text on label is not painted red with the GTK L [v7]

2024-05-17 Thread Abhishek Kumar
On Fri, 17 May 2024 07:26:27 GMT, Prasanta Sadhukhan wrote: >> Abhishek Kumar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> bug id added > > test/jdk/ProblemList.txt line 673: > >> 671:

Re: RFR: 8075917: The regression-swing case failed as the text on label is not painted red with the GTK L [v6]

2024-05-17 Thread Prasanta Sadhukhan
On Thu, 16 May 2024 08:48:30 GMT, Abhishek Kumar wrote: >> JLabel text is not painted with the LAF defined foreground color in GTK LAF. >> In GTK LAF the foreground color is retrieved by using native system APIs. >> Fix is to return the foreground color if it is set by LAF defined property >>