Re: RFR: 8274465: Fix javax/swing/text/ParagraphView/6364882/bug6364882.java failures

2021-09-29 Thread Jie Fu
On Wed, 29 Sep 2021 19:48:20 GMT, Alexey Ivanov wrote: > I made the test and its assertions more flexible in regards to the font size > and to the number of lines. > > The test still requires there are at least four lines so that the text is > wrapped before the first hard line break; thus at

Re: RFR: 8274505: Too weak variable type leads to unnecessary cast in java.desktop

2021-09-29 Thread Pankaj Bansal
On Tue, 28 Sep 2021 08:02:12 GMT, Andrey Turbanov wrote: > Such casts are actually redundant, but they are inserted, because variable is > declared with too weak type. Marked as reviewed by pbansal (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/5731

Re: RFR: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Jie Fu
On Wed, 29 Sep 2021 23:16:14 GMT, Jie Fu wrote: >> Hi all, >> >> javax/swing/text/ParagraphView/6364882/bug6364882.java was observed failing >> on our non-GUI platforms. >> So I guess it should be marked as headful. >> >> Thanks. >> Best regards, >> Jie > > Will discuss it in https://github.co

Re: RFR: 8274505: Too weak variable type leads to unnecessary cast in java.desktop

2021-09-29 Thread Alexander Zuev
On Tue, 28 Sep 2021 08:02:12 GMT, Andrey Turbanov wrote: > Such casts are actually redundant, but they are inserted, because variable is > declared with too weak type. Marked as reviewed by kizune (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/5731

Re: RFR: 8274505: Too weak variable type leads to unnecessary cast in java.desktop

2021-09-29 Thread Sergey Bylokhov
On Tue, 28 Sep 2021 08:02:12 GMT, Andrey Turbanov wrote: > Such casts are actually redundant, but they are inserted, because variable is > declared with too weak type. Marked as reviewed by serb (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/5731

Re: RFR: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Jie Fu
On Tue, 28 Sep 2021 23:39:45 GMT, Jie Fu wrote: > Hi all, > > javax/swing/text/ParagraphView/6364882/bug6364882.java was observed failing > on our non-GUI platforms. > So I guess it should be marked as headful. > > Thanks. > Best regards, > Jie Will discuss it in https://github.com/openjdk/jd

Withdrawn: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Jie Fu
On Tue, 28 Sep 2021 23:39:45 GMT, Jie Fu wrote: > Hi all, > > javax/swing/text/ParagraphView/6364882/bug6364882.java was observed failing > on our non-GUI platforms. > So I guess it should be marked as headful. > > Thanks. > Best regards, > Jie This pull request has been closed without being

Re: RFR: 8274465: Fix javax/swing/text/ParagraphView/6364882/bug6364882.java failures

2021-09-29 Thread Jie Fu
On Wed, 29 Sep 2021 19:48:20 GMT, Alexey Ivanov wrote: > I made the test and its assertions more flexible in regards to the font size > and to the number of lines. > > The test still requires there are at least four lines so that the text is > wrapped before the first hard line break; thus at

Re: RFR: 8254841: [macos] Enter and Exit events shouldn't be sent whilst resizing [v8]

2021-09-29 Thread Sergey Bylokhov
On Wed, 29 Sep 2021 22:29:19 GMT, Alisen Chung wrote: >> Added a resizing flag when the window is currently being resized to block >> mouseEntered and mouseExited events from being posted to that window. > > Alisen Chung has updated the pull request incrementally with one additional > commit s

Re: RFR: 8254841: [macos] Enter and Exit events shouldn't be sent whilst resizing [v8]

2021-09-29 Thread Alisen Chung
> Added a resizing flag when the window is currently being resized to block > mouseEntered and mouseExited events from being posted to that window. Alisen Chung has updated the pull request incrementally with one additional commit since the last revision: added check for fullscreen mode to sy

Re: RFR: 8254841: [macos] Enter and Exit events shouldn't be sent whilst resizing [v5]

2021-09-29 Thread Alisen Chung
On Fri, 24 Sep 2021 19:49:50 GMT, Sergey Bylokhov wrote: > I guess the only test which makes sense here is the test for JDK-8013468 and > it is "problem listed" due to a reflection issue. I suggest checking how it > works. This test fails both before and after the change. However, before the c

RFR: 8274465: Fix javax/swing/text/ParagraphView/6364882/bug6364882.java failures

2021-09-29 Thread Alexey Ivanov
I made the test and its assertions more flexible in regards to the font size and to the number of lines. The test still requires there are at least four lines so that the text is wrapped before the first hard line break; thus at least one line of text is justified. This pre-requisite gets broke

Re: RFR: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Alexey Ivanov
On Wed, 29 Sep 2021 13:54:25 GMT, Jie Fu wrote: >>> The image `editorPane.png` is automatically saved for analysis if the test >>> fails, you should find in it the test artefacts along with the jtreg test >>> log. >> >> It is not saved on `AssertionError` — my bad. >> I updated the test so t

Re: RFR: 8272229: BasicSplitPaneDivider:oneTouchExpandableChanged() returns leftButton and rightButton as null with GTKLookAndFeel [v3]

2021-09-29 Thread Sergey Bylokhov
On Mon, 27 Sep 2021 13:34:44 GMT, Pankaj Bansal wrote: >> The function documentation for oneTouchExpandableChanged states that the >> function does couple of things, but in reality it only does those things if >> the Look and Feel supports the supportsOneTouchButtons. In case the Look and >>

Re: RFR: 8274496: Use String.contains() instead of String.indexOf() in java.desktop

2021-09-29 Thread Sergey Bylokhov
On Fri, 17 Sep 2021 15:46:51 GMT, Andrey Turbanov wrote: > String.contains was introduced in Java 5. > Some code in java.desktop still uses old approach with String.indexOf to > check if String contains specified substring. > I propose to migrate such usages. Makes code shorter and easier to re

Re: RFR: 8273101: Eliminate the usage of threadgroup sandboxing in the java.util.logging

2021-09-29 Thread Sergey Bylokhov
On Wed, 1 Sep 2021 06:31:16 GMT, Sergey Bylokhov wrote: > At the time Java supported applets and webstart, a special mechanism for > launching various applications in one JVM was used to reduce memory usage and > each application was isolated from each other. > > This isolation was implemented

Re: RFR: 8273634: [TEST_BUG] Improve javax/swing/text/ParagraphView/6364882/bug6364882.java

2021-09-29 Thread Alexey Ivanov
On Wed, 29 Sep 2021 16:09:20 GMT, Phil Race wrote: >> The test is being moved from the closed to open. >> >> I updated the `bug6364882.java` test so that it's headless now. If it fails, >> the image of the `editorPane` is saved for analysis. I also added more >> asserts which verify: >> >> •

Re: RFR: 8273634: [TEST_BUG] Improve javax/swing/text/ParagraphView/6364882/bug6364882.java

2021-09-29 Thread Phil Race
On Thu, 23 Sep 2021 19:19:34 GMT, Alexey Ivanov wrote: > The test is being moved from the closed to open. > > I updated the `bug6364882.java` test so that it's headless now. If it fails, > the image of the `editorPane` is saved for analysis. I also added more > asserts which verify: > > • the

Re: RFR: 8273634: [TEST_BUG] Improve javax/swing/text/ParagraphView/6364882/bug6364882.java

2021-09-29 Thread Phil Race
On Thu, 23 Sep 2021 19:19:34 GMT, Alexey Ivanov wrote: > The test is being moved from the closed to open. > > I updated the `bug6364882.java` test so that it's headless now. If it fails, > the image of the `editorPane` is saved for analysis. I also added more > asserts which verify: > > • the

Re: RFR: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Alexey Ivanov
On Tue, 28 Sep 2021 23:39:45 GMT, Jie Fu wrote: > Hi all, > > javax/swing/text/ParagraphView/6364882/bug6364882.java was observed failing > on our non-GUI platforms. > So I guess it should be marked as headful. > > Thanks. > Best regards, > Jie There are relevant comments raised by @prrace in

Re: RFR: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Jie Fu
On Wed, 29 Sep 2021 12:25:55 GMT, Alexey Ivanov wrote: > > The image `editorPane.png` is automatically saved for analysis if the test > > fails, you should find in it the test artefacts along with the jtreg test > > log. > > It is not saved on `AssertionError` — my bad. I updated the test so t

Re: RFR: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Alexey Ivanov
On Wed, 29 Sep 2021 11:44:49 GMT, Alexey Ivanov wrote: > The image `editorPane.png` is automatically saved for analysis if the test > fails, you should find in it the test artefacts along with the jtreg test log. It is not saved on `AssertionError` — my bad. I updated the test so that it save

HiDPI Fractional Scaling on Linux/GTK

2021-09-29 Thread Matthew Wallace
I'd like to use fractional scaling in my swing application on Linux (GTK). My application already works perfectly with fractional scaling on Windows (Java seems to abide by the system settings) but on Linux the system settings are ignored and setting GDK_SCALE only works for integer values. Could a

Re: RFR: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Alexey Ivanov
On Wed, 29 Sep 2021 11:44:49 GMT, Alexey Ivanov wrote: >>> It is possible that the test is too strict or this is a jdk bug. But I am >>> not sure it is possible to investigate this issue based on the bug >>> description in the JBS. @aivanov-jdk please take a look. >> >> Can I problemlist this

Re: RFR: 8274465: Mark javax/swing/text/ParagraphView/6364882/bug6364882.java as headful

2021-09-29 Thread Alexey Ivanov
On Wed, 29 Sep 2021 01:40:24 GMT, Jie Fu wrote: >> It is possible that the test is too strict or this is a jdk bug. But I am >> not sure it is possible to investigate this issue based on the bug >> description in the JBS. >> @aivanov-jdk please take a look. > >> It is possible that the test is

RFR: 8274505: Too weak variable type leads to unnecessary cast in java.desktop

2021-09-29 Thread Andrey Turbanov
Such casts are actually redundant, but they are inserted, because variable is declared with too weak type. - Commit messages: - [PATCH] Too weak variable type leads to unnecessary cast in java.desktop - [PATCH] Too weak variable type leads to unnecessary cast in java.desktop Chang

Re: RFR: 8274505: Too weak variable type leads to unnecessary cast in java.desktop

2021-09-29 Thread Sergey Bylokhov
On Tue, 28 Sep 2021 08:02:12 GMT, Andrey Turbanov wrote: > Such casts are actually redundant, but they are inserted, because variable is > declared with too weak type. src/java.desktop/share/classes/javax/swing/JTree.java line 2046: > 2044: // expensive! > 2045:

Re: RFR: 8274497: Unnecessary Vector usage in AquaFileSystemModel

2021-09-29 Thread Sergey Bylokhov
On Fri, 17 Sep 2021 20:32:00 GMT, Andrey Turbanov wrote: > Fields AquaFileSystemModel.fileCache and > AquaFileSystemModel.FilesLoader.queuedTasks are accessed only under > synchronized(fileCacheLock). > It means extract synchronization by Vector is not needed. Looks fine, please update the co

RFR: 8274497: Unnecessary Vector usage in AquaFileSystemModel

2021-09-29 Thread Andrey Turbanov
Fields AquaFileSystemModel.fileCache and AquaFileSystemModel.FilesLoader.queuedTasks are accessed only under synchronized(fileCacheLock). It means extract synchronization by Vector is not needed. - Commit messages: - [PATCH] Unnecessary Vector usage in AquaFileSystemModel - [PATCH

Re: RFR: 8274496: Use String.contains() instead of String.indexOf() in java.desktop

2021-09-29 Thread Pankaj Bansal
On Fri, 17 Sep 2021 15:46:51 GMT, Andrey Turbanov wrote: > String.contains was introduced in Java 5. > Some code in java.desktop still uses old approach with String.indexOf to > check if String contains specified substring. > I propose to migrate such usages. Makes code shorter and easier to re

Re: RFR: 8274496: Use String.contains() instead of String.indexOf() in java.desktop

2021-09-29 Thread Sergey Bylokhov
On Fri, 17 Sep 2021 15:46:51 GMT, Andrey Turbanov wrote: > String.contains was introduced in Java 5. > Some code in java.desktop still uses old approach with String.indexOf to > check if String contains specified substring. > I propose to migrate such usages. Makes code shorter and easier to re

RFR: 8274496: Use String.contains() instead of String.indexOf() in java.desktop

2021-09-29 Thread Andrey Turbanov
String.contains was introduced in Java 5. Some code in java.desktop still uses old approach with String.indexOf to check if String contains specified substring. I propose to migrate such usages. Makes code shorter and easier to read. - Commit messages: - [PATCH] Use String.contains(

Re: RFR: 8273617: UninitializedDisplayModeChangeTest.java times out on macOS 12 [v2]

2021-09-29 Thread Sergey Bylokhov
On Wed, 29 Sep 2021 04:13:52 GMT, Prasanta Sadhukhan wrote: > Why it is considered a workaround? There are many instances where we use > [ThreadUtilities performOnMainThreadWaiting:NO block:^() It is a workaround because it does not fix the hang, but just skips part of the functionality. Sinc