Re: RFR: 8295774 : Write a test to verify that List Item selection events. [v2]

2022-11-01 Thread ravi gupta
On Sun, 30 Oct 2022 12:57:38 GMT, Alexey Ivanov wrote: >> ravi gupta has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8295774 : Write a test to verify that List Item selection events. > > test/jdk/java/awt/event/ComponentEvent/ListItemEve

Re: RFR: 8295774 : Write a test to verify that List Item selection events. [v2]

2022-11-01 Thread ravi gupta
> This testcase verify that List Item selection via mouse/keys generates > ItemEvent/ActionEvent appropriately. > > a. Single click on the list generate ItemEvent and double click on item > generates ActionEvent. > b. UP/DOWN keys on the list generate ItemEvent and enter key on item > generates

Re: RFR: 8294488: Delete KCMS transforms wrappers

2022-11-01 Thread Sergey Bylokhov
On Wed, 28 Sep 2022 03:12:55 GMT, Sergey Bylokhov wrote: > This is a request to cleanup the shared code for the colors conversions. That > code still uses some wrappers which can be simplified and/or deleted after > KCMS removal. For example, each conversion from one color space to another > c

Re: RFR: 8294488: Delete KCMS transforms wrappers

2022-11-01 Thread Phil Race
On Wed, 28 Sep 2022 03:12:55 GMT, Sergey Bylokhov wrote: > This is a request to cleanup the shared code for the colors conversions. That > code still uses some wrappers which can be simplified and/or deleted after > KCMS removal. For example, each conversion from one color space to another > c

Re: RFR: JDK-8293776 : Adds CSS 4 and 8 digits hex coded Color [v10]

2022-11-01 Thread ScientificWare
On Tue, 1 Nov 2022 16:38:34 GMT, Alexey Ivanov wrote: >> ScientificWare has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Corrects a value in a message. >> >> A message is added to the result in case of failure only. The updated code

Integrated: 8282958: Rendering issues of borders, TextFields on Windows High-DPI systems

2022-11-01 Thread Rajat Mahajan
On Wed, 12 Oct 2022 16:41:52 GMT, Rajat Mahajan wrote: > This fix is based on a similar approach as described here > https://github.com/openjdk/jdk/pull/7449#issuecomment-1068218648. > We rescale the line border to render the Textfield border correctly, so that > all sides of the Textfield bor

Re: RFR: JDK-8015739: Background of JInternalFrame is located out of JInternalFrame [v11]

2022-11-01 Thread Alexander Zuev
On Fri, 21 Oct 2022 22:21:09 GMT, Harshitha Onkar wrote: >> JInternalFrame background color seems to overflow into the border region. >> This issue is more prominently seen on Windows - Metal LAF (with fractional >> scaling, as shown below). The primary reason is border scaling issue as >> obs

Re: RFR: 8295737: macOS: Print content cut off when width > height with portrait orientation

2022-11-01 Thread Phil Race
On Fri, 21 Oct 2022 08:42:12 GMT, Alexander Scherbatiy wrote: > A printed content is truncated on macOS if the content paper size width > larger than height with portrait orientation or width is less than height > with landscape orientation. > > To reproduce the issue run the > [CutOffImage]

Re: RFR: 8282958: Rendering issues of borders, TextFields on Windows High-DPI systems [v3]

2022-11-01 Thread Alexey Ivanov
On Tue, 1 Nov 2022 17:38:57 GMT, Rajat Mahajan wrote: >> This fix is based on a similar approach as described here >> https://github.com/openjdk/jdk/pull/7449#issuecomment-1068218648. >> We rescale the line border to render the Textfield border correctly, so that >> all sides of the Textfield

Re: RFR: 8282958: Rendering issues of borders, TextFields on Windows High-DPI systems [v3]

2022-11-01 Thread Alexander Zuev
On Tue, 1 Nov 2022 17:38:57 GMT, Rajat Mahajan wrote: >> This fix is based on a similar approach as described here >> https://github.com/openjdk/jdk/pull/7449#issuecomment-1068218648. >> We rescale the line border to render the Textfield border correctly, so that >> all sides of the Textfield

Re: RFR: 8282958: Rendering issues of borders, TextFields on Windows High-DPI systems [v3]

2022-11-01 Thread Rajat Mahajan
> This fix is based on a similar approach as described here > https://github.com/openjdk/jdk/pull/7449#issuecomment-1068218648. > We rescale the line border to render the Textfield border correctly, so that > all sides of the Textfield border are of same thickness. Rajat Mahajan has updated the

Re: RFR: 8282958: Rendering issues of borders, TextFields on Windows High-DPI systems [v2]

2022-11-01 Thread Harshitha Onkar
On Fri, 28 Oct 2022 16:00:10 GMT, Rajat Mahajan wrote: >> This fix is based on a similar approach as described here >> https://github.com/openjdk/jdk/pull/7449#issuecomment-1068218648. >> We rescale the line border to render the Textfield border correctly, so that >> all sides of the Textfield

Re: RFR: 8282958: Rendering issues of borders, TextFields on Windows High-DPI systems [v2]

2022-11-01 Thread Harshitha Onkar
On Fri, 28 Oct 2022 16:00:10 GMT, Rajat Mahajan wrote: >> This fix is based on a similar approach as described here >> https://github.com/openjdk/jdk/pull/7449#issuecomment-1068218648. >> We rescale the line border to render the Textfield border correctly, so that >> all sides of the Textfield

Re: RFR: 6257207: JTable.getDefaultEditor throws NullPointerException [v3]

2022-11-01 Thread Alexey Ivanov
On Mon, 31 Oct 2022 09:14:23 GMT, Prasanta Sadhukhan wrote: >> If getDefaultEditor() is called before the JTable model is setup, it results >> in NPE. >> >> This is because when JTable sets its model, which ends up firing a table >> changed event. The testcase is listening for tableChanged ev

Re: RFR: 6257207: JTable.getDefaultEditor throws NullPointerException [v2]

2022-11-01 Thread Alexey Ivanov
On Mon, 31 Oct 2022 09:11:21 GMT, Prasanta Sadhukhan wrote: >> If getDefaultEditor() is called before the JTable model is setup, it results >> in NPE. >> >> This is because when JTable sets its model, which ends up firing a table >> changed event. The testcase is listening for tableChanged ev

Re: RFR: JDK-8293776 : Adds CSS 4 and 8 digits hex coded Color [v10]

2022-11-01 Thread Alexey Ivanov
On Tue, 27 Sep 2022 03:17:26 GMT, ScientificWare wrote: >> This is referenced in Java Bug Database as >> - [JDK-8293776 : Adds CSS 4 and 8 digits hex coded >> Color](https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8293776) >> >> This is tracked in JBS as >> - [JDK-8293776 : Adds CSS 4 and

Re: RFR: 6201035: Document NPE for passing null insets to constructors and methods of several javax.swing.border.* classes [v5]

2022-11-01 Thread Alexey Ivanov
On Mon, 31 Oct 2022 07:48:38 GMT, Prasanta Sadhukhan wrote: >> The behavior of MatteBorder constructors taking Insets object as a parameter >> is undocumented. It would throw NPE if null object is passed to it, which >> should be documented in the spec. > > Prasanta Sadhukhan has updated the p

Re: RFR: 8294241: Deprecate URL public constructors [v2]

2022-11-01 Thread Daniel Fuchs
On Tue, 1 Nov 2022 14:47:49 GMT, Alan Bateman wrote: >> Actually... Maybe I could move up the paragraph that says that URL >> constructors are deprecated up here, just after the

title? Would >> that be better? > > Try it, it might be better. I think the main thing is that link brings the


Re: RFR: 8294241: Deprecate URL public constructors [v3]

2022-11-01 Thread Phil Race
On Tue, 1 Nov 2022 16:10:47 GMT, Daniel Fuchs wrote: >> Deprecate URL constructors. Developers are encouraged to use `java.net.URI` >> to parse or construct any URL. >> >> The `java.net.URL` class does not itself encode or decode any URL components >> according to the escaping mechanism define

Re: RFR: 8294241: Deprecate URL public constructors [v3]

2022-11-01 Thread Daniel Fuchs
> Deprecate URL constructors. Developers are encouraged to use `java.net.URI` > to parse or construct any URL. > > The `java.net.URL` class does not itself encode or decode any URL components > according to the escaping mechanism defined in RFC2396. It is the > responsibility of the caller to e

Re: RFR: 8252075: Documentation error in LayoutManager2 interface [v5]

2022-11-01 Thread Alexey Ivanov
On Mon, 31 Oct 2022 13:24:30 GMT, Renjith Kannath Pariyangad wrote: >> Updated the documentation with proper meaningful message for better >> understanding. > > Renjith Kannath Pariyangad has updated the pull request incrementally with > one additional commit since the last revision: > > Up

Re: RFR: 6187113: DefaultListSelectionModel.removeIndexInterval(0, Integer.MAX_VALUE) fails [v10]

2022-11-01 Thread Alexey Ivanov
On Mon, 31 Oct 2022 08:59:31 GMT, Prasanta Sadhukhan wrote: >> DefaultListSelectionModel.removeIndexInterva accepts `int` value which >> allows it to take in Integer.MAX_VALUE theoratically but it does calculation >> with that value which can results in IOOBE. >> Fix is to make sure the calcul

Re: RFR: 8295737: macOS: Print content cut off when width > height with portrait orientation

2022-11-01 Thread Alexander Scherbatiy
On Tue, 1 Nov 2022 03:06:16 GMT, Phil Race wrote: > The analysis of the problem sounds fine but then it all gets confusing to me > when I come to the fix. > > So first you capture the intended orientation in the switch but do not set it > to the printer context. The captured orientation is s

Re: RFR: 8294241: Deprecate URL public constructors [v2]

2022-11-01 Thread Alan Bateman
On Tue, 1 Nov 2022 14:22:18 GMT, Daniel Fuchs wrote: >> To be discussed: I actually wanted the deprecation link ( the link from >> `@deprecated` ) to lead here because I find that the whole section is >> relevant for developers who might want to decide whether to actually move >> away from usi

Re: RFR: 8294241: Deprecate URL public constructors [v2]

2022-11-01 Thread Daniel Fuchs
On Tue, 1 Nov 2022 14:10:01 GMT, Daniel Fuchs wrote: >> src/java.base/share/classes/java/net/URL.java line 133: >> >>> 131: * specified. The optional fragment is not inherited. >>> 132: * >>> 133: * Constructing instances of >>> {@code URL} >> >> Would it be better to move the anchor to lin

Re: RFR: 8294241: Deprecate URL public constructors [v2]

2022-11-01 Thread Daniel Fuchs
On Sat, 29 Oct 2022 14:24:09 GMT, Alan Bateman wrote: >> Daniel Fuchs has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains four additional >> commits

Re: RFR: 8294241: Deprecate URL public constructors [v2]

2022-11-01 Thread Daniel Fuchs
On Sat, 29 Oct 2022 14:17:12 GMT, Alan Bateman wrote: >> Daniel Fuchs has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains four additional >> commits

Re: RFR: 8294241: Deprecate URL public constructors [v2]

2022-11-01 Thread Daniel Fuchs
On Sat, 29 Oct 2022 14:16:24 GMT, Alan Bateman wrote: >> Daniel Fuchs has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains four additional >> commits

Re: RFR: 8294241: Deprecate URL public constructors [v2]

2022-11-01 Thread Daniel Fuchs
On Sat, 29 Oct 2022 14:14:22 GMT, Alan Bateman wrote: >> Daniel Fuchs has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains four additional >> commits

Re: RFR: 8294241: Deprecate URL public constructors [v2]

2022-11-01 Thread Daniel Fuchs
On Mon, 31 Oct 2022 22:00:01 GMT, Phil Race wrote: > Deprecate URL constructors. Developers are encouraged to use java.net.URI to > parse or construct any URL. ... To construct a URL, using URI::toURL should > be preferred. > > You have jumped through some refactoring hoops to be able to apply

Re: RFR: 8295729: Add jcheck whitespace checking for properties files [v3]

2022-11-01 Thread Daniel Fuchs
On Mon, 24 Oct 2022 19:21:07 GMT, Magnus Ihse Bursie wrote: >> Properties files is essentially source code. It should have the same >> whitespace checks as all other source code, so we don't get spurious >> trailing whitespace changes. >> >> With the new Skara jcheck, it is possible to increas