Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v7]

2025-06-25 Thread Christian Stein
On Wed, 25 Jun 2025 21:28:16 GMT, Alisen Chung wrote: >> This issue is responsible for updating the translations of all the >> localize(able) resources in the JDK since the previous L10n drop. > > Alisen Chung has updated the pull request incrementally with one additional > commit since the las

Re: RFR: 8360462: [macosx] row selection not working with Ctrl+Shift+Down/Up in AquaL&F [v4]

2025-06-25 Thread Prasanta Sadhukhan
> Ctrl+Shift+Down/Up does extend row selection down or up in Aqua L&F as it > does in native tree view in Finder. > Fixed the keybindings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Test modification - Change

Re: RFR: 8360462: [macosx] row selection not working with Ctrl+Shift+Down/Up in AquaL&F [v3]

2025-06-25 Thread Prasanta Sadhukhan
On Thu, 26 Jun 2025 04:55:01 GMT, Abhishek Kumar wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix > > test/jdk/javax/swing/JTree/TestTreeRowSelection.java line 62: > >> 60: frame.pack(); >

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v3]

2025-06-25 Thread Tejesh R
On Wed, 25 Jun 2025 12:15:54 GMT, Prasanta Sadhukhan wrote: >> BasicTextUI.getVisibleEditorRect wording is rephrased to remove the wording >> "Due to an unfortunate set of historical events this method is >> inappropriately named". > > Prasanta Sadhukhan has updated the pull request incrementa

Re: RFR: 8360462: [macosx] row selection not working with Ctrl+Shift+Down/Up in AquaL&F [v3]

2025-06-25 Thread Abhishek Kumar
On Thu, 26 Jun 2025 02:31:17 GMT, Prasanta Sadhukhan wrote: >> Ctrl+Shift+Down/Up does extend row selection down or up in Aqua L&F as it >> does in native tree view in Finder. >> Fixed the keybindings. > > Prasanta Sadhukhan has updated the pull request incrementally with one > additional comm

Re: RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v29]

2025-06-25 Thread Prasanta Sadhukhan
On Thu, 26 Jun 2025 03:09:29 GMT, Prasanta Sadhukhan wrote: >> When JRadioButtonMenuItem is called with imageIcon, then only imageIcon is >> shown without radiobutton in WIndowsLookAndFeel as there was no provision of >> drawing the radiobutton alongside icon. >> If icon is not there, the radi

Re: RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v29]

2025-06-25 Thread Prasanta Sadhukhan
> When JRadioButtonMenuItem is called with imageIcon, then only imageIcon is > shown without radiobutton in WIndowsLookAndFeel as there was no provision of > drawing the radiobutton alongside icon. > If icon is not there, the radiobutton is drawn. Added provision of drawing > the radiobutton win

Re: RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v28]

2025-06-25 Thread Prasanta Sadhukhan
> When JRadioButtonMenuItem is called with imageIcon, then only imageIcon is > shown without radiobutton in WIndowsLookAndFeel as there was no provision of > drawing the radiobutton alongside icon. > If icon is not there, the radiobutton is drawn. Added provision of drawing > the radiobutton win

Re: RFR: 8360462: [macosx] row selection not working with Ctrl+Shift+Down/Up in AquaL&F [v3]

2025-06-25 Thread Prasanta Sadhukhan
On Thu, 26 Jun 2025 02:31:17 GMT, Prasanta Sadhukhan wrote: >> Ctrl+Shift+Down/Up does extend row selection down or up in Aqua L&F as it >> does in native tree view in Finder. >> Fixed the keybindings. > > Prasanta Sadhukhan has updated the pull request incrementally with one > additional comm

Re: RFR: 8360462: [macosx] row selection not working with Ctrl+Shift+Down/Up in AquaL&F [v3]

2025-06-25 Thread Prasanta Sadhukhan
> Ctrl+Shift+Down/Up does extend row selection down or up in Aqua L&F as it > does in native tree view in Finder. > Fixed the keybindings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Fix - Changes: - all: ht

Re: RFR: 8359266: Delete the usage of AppContext in the GraphicsDevice

2025-06-25 Thread Sergey Bylokhov
On Wed, 25 Jun 2025 14:59:53 GMT, Alexey Ivanov wrote: >> This PR removes the usage of AppContext from the GraphicsDevice class. The >> original use case was to store the full-screen window in some AppContext, >> which was necessary in the context of plugin/appletviewer environments. >> Howeve

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v7]

2025-06-25 Thread Damon Nguyen
On Wed, 25 Jun 2025 21:28:16 GMT, Alisen Chung wrote: >> This issue is responsible for updating the translations of all the >> localize(able) resources in the JDK since the previous L10n drop. > > Alisen Chung has updated the pull request incrementally with one additional > commit since the las

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v2]

2025-06-25 Thread Justin Lu
On Tue, 24 Jun 2025 14:58:38 GMT, Weijun Wang wrote: >> However, there is no space required between two Chinese sentences. In >> English, we usually write "I am here, and you are there." But in Chinese, >> with the full-width punctuation, it's always "我在这里,你在那里。". > > Some people like to insert

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v6]

2025-06-25 Thread Justin Lu
On Wed, 25 Jun 2025 17:08:20 GMT, Alisen Chung wrote: >> This issue is responsible for updating the translations of all the >> localize(able) resources in the JDK since the previous L10n drop. > > Alisen Chung has updated the pull request incrementally with three additional > commits since the

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v7]

2025-06-25 Thread Justin Lu
On Wed, 25 Jun 2025 21:28:16 GMT, Alisen Chung wrote: >> This issue is responsible for updating the translations of all the >> localize(able) resources in the JDK since the previous L10n drop. > > Alisen Chung has updated the pull request incrementally with one additional > commit since the las

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v7]

2025-06-25 Thread Alisen Chung
> This issue is responsible for updating the translations of all the > localize(able) resources in the JDK since the previous L10n drop. Alisen Chung has updated the pull request incrementally with one additional commit since the last revision: revert german translation, revert english wxl fo

"Smudge" factor on Mac OSX mouse events?

2025-06-25 Thread Laird Nelson
(Disclaimer: I happen to work for Oracle but this has nothing to do with my job.) I have been having trouble double-clicking things with my Apple Magic Mouse on my M1 Mac when running (Swing-based) Netbeans (see https://github.com/apache/netbeans/issues/8609 and the inciting discussion at https://

Re: RFR: 8359266: Delete the usage of AppContext in the GraphicsDevice

2025-06-25 Thread Sergey Bylokhov
On Tue, 24 Jun 2025 12:50:14 GMT, Alexey Ivanov wrote: >>>What I mean is two threads that called GraphicsDevice.getFullScreenWindow >>>still established a happens-before relation for each call, which will be >>>gone with the volatile modifier. >>>It may be fine… or may be not… this is why I'm a

Integrated: 8131136: java/awt/font/JNICheck/JNICheck.sh issue warning on core-libs code

2025-06-25 Thread Daniel Jeliński
On Wed, 11 Jun 2025 13:18:47 GMT, Daniel Jeliński wrote: > Add an exception check after a JNI static method call, and re-enable checks > for CallStatic in JNICheck.sh test. > > I verified that the JNICheck test is passing on all headless and headful > platforms I had access to (Windows, Linux

[jdk25] RFR: 8358697: TextLayout/MyanmarTextTest.java passes if no Myanmar font is found

2025-06-25 Thread SendaoYan
Hi all, This pull request contains a backport of commit [bcad87ea](https://github.com/openjdk/jdk/commit/bcad87eacbd7fbfd3254479b7e061bab34e64aec) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. The commit being backported was authored by Manukumar V S on 24 Jun 2025 and was re

Re: RFR: 8360487: Remove unnecessary List.indexOf key from AbstractMidiDevice.TransmitterList.remove

2025-06-25 Thread Alexey Ivanov
On Mon, 23 Jun 2025 08:53:16 GMT, Andrey Turbanov wrote: > No need to call `List.indexOf(Object)` before `List.remove(int)`. Instead we > can call `List.remove(Object)` directly. It's faster and cleaner. Marked as reviewed by aivanov (Reviewer). - PR Review: https://git.openjdk.or

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v6]

2025-06-25 Thread Alisen Chung
On Wed, 25 Jun 2025 17:08:20 GMT, Alisen Chung wrote: >> This issue is responsible for updating the translations of all the >> localize(able) resources in the JDK since the previous L10n drop. > > Alisen Chung has updated the pull request incrementally with three additional > commits since the

Re: RFR: 8346753: Test javax/swing/JMenuItem/RightLeftOrientation/RightLeftOrientation.java fails on Windows Server 2025 x64 because the icons of RBMenuItem and CBMenuItem are not visible in Nimbus Lo

2025-06-25 Thread Alexey Ivanov
On Wed, 25 Jun 2025 09:37:10 GMT, Prasanta Sadhukhan wrote: > I dont want to do it as I already told before..Its the same strategy used in > other recently opensourced tests I mentioned before where > `SwingUtilities.updateComponentTreeUI(frame)` is used ... But missing `SwingUtilities.update

Re: RFR: 8360487: Remove unnecessary List.indexOf key from AbstractMidiDevice.TransmitterList.remove

2025-06-25 Thread Sergey Bylokhov
On Mon, 23 Jun 2025 08:53:16 GMT, Andrey Turbanov wrote: > No need to call `List.indexOf(Object)` before `List.remove(int)`. Instead we > can call `List.remove(Object)` directly. It's faster and cleaner. Marked as reviewed by serb (Reviewer). please rerun the Pre-submit tests in the GA --

Re: RFR: 8359266: Delete the usage of AppContext in the GraphicsDevice

2025-06-25 Thread Alexey Ivanov
On Mon, 16 Jun 2025 06:17:32 GMT, Sergey Bylokhov wrote: > This PR removes the usage of AppContext from the GraphicsDevice class. The > original use case was to store the full-screen window in some AppContext, > which was necessary in the context of plugin/appletviewer environments. > However,

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v3]

2025-06-25 Thread Alexey Ivanov
On Wed, 25 Jun 2025 12:15:54 GMT, Prasanta Sadhukhan wrote: >> BasicTextUI.getVisibleEditorRect wording is rephrased to remove the wording >> "Due to an unfortunate set of historical events this method is >> inappropriately named". > > Prasanta Sadhukhan has updated the pull request incrementa

Re: RFR: 8360070: AccessibleText.getBeforeIndex returns null for last character [v3]

2025-06-25 Thread Alexey Ivanov
On Tue, 24 Jun 2025 11:44:13 GMT, Abhishek Kumar wrote: >> `AccessibleText.getBeforeIndex` method returns `null for last characte`r due >> to the **wrong boundary value condition check**. >> This method returns `null` when the `passed index parameter` is equal to >> `text's length` which is inc

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v2]

2025-06-25 Thread Weijun Wang
On Tue, 24 Jun 2025 14:54:17 GMT, Weijun Wang wrote: >> Thank you for the clarification. > > However, there is no space required between two Chinese sentences. In > English, we usually write "I am here, and you are there." But in Chinese, > with the full-width punctuation, it's always "我在这里,你在那

Re: RFR: 8359266: Delete the usage of AppContext in the GraphicsDevice

2025-06-25 Thread Alexey Ivanov
On Sat, 21 Jun 2025 00:17:26 GMT, Sergey Bylokhov wrote: > That only affects the case where the second thread tries to read shared data > (unrelated to GraphicsDevice) without synchronization, which was written > before the first thread called getFullScreenWindow(). If visibility is > required

Re: RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v26]

2025-06-25 Thread Alexey Ivanov
On Thu, 19 Jun 2025 06:06:23 GMT, Prasanta Sadhukhan wrote: > > > In SwingSet2, now there will be gap between bullet and text to accomodate > > > non-existing imageicon > > > > > > That gap should only be there if _at least one_ menu item has an image > > icon. It shoudn't be there if none h

Re: RFR: 8131136: java/awt/font/JNICheck/JNICheck.sh issue warning on core-libs code [v2]

2025-06-25 Thread Daniel Jeliński
On Mon, 23 Jun 2025 13:41:38 GMT, Alexey Ivanov wrote: >> Daniel Jeliński has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Update copyright > > Marked as reviewed by aivanov (Reviewer). Thanks @aivanov-jdk for the re-review. ---

Re: RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v26]

2025-06-25 Thread Prasanta Sadhukhan
On Thu, 19 Jun 2025 09:45:14 GMT, Alexey Ivanov wrote: > > In SwingSet2, now there will be gap between bullet and text to accomodate > > non-existing imageicon > > That gap should only be there if _at least one_ menu item has an image icon. > It shoudn't be there if none have an icon. @prrace

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v3]

2025-06-25 Thread Alexey Ivanov
On Wed, 25 Jun 2025 07:06:13 GMT, Prasanta Sadhukhan wrote: >>> It is passed to `View.setSize` also so just mentioning one may not be right… >> >> Right. Both uses cases fall into the same category: the allocation controls >> the size of the root view. >> >> The returned allocation rectangle

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v6]

2025-06-25 Thread Alisen Chung
> This issue is responsible for updating the translations of all the > localize(able) resources in the JDK since the previous L10n drop. Alisen Chung has updated the pull request incrementally with three additional commits since the last revision: - update majorticks translation - Merge branc

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v4]

2025-06-25 Thread Justin Lu
On Wed, 25 Jun 2025 00:27:48 GMT, Damon Nguyen wrote: >> src/java.base/share/classes/sun/security/util/resources/security_zh_CN.properties >> line 49: >> >>> 47: .Principal.=\t主用户:\u0020 >>> 48: .Public.Credential.=\t公共身份证明:\u0020 >>> 49: .Private.Credential.=\t专用身份证明:\u0020 >> >> Why aren't t

Re: RFR: 8360070: AccessibleText.getBeforeIndex returns null for last character

2025-06-25 Thread Sergey Bylokhov
On Wed, 25 Jun 2025 04:43:14 GMT, Abhishek Kumar wrote: > So, if someone wants to retrieve the first or last character, they shouldn't > rely on `getAfterIndex and getBeforeIndex` method respectively, rather use > `getAtIndex` method to get the first and last character with index passed as > 0

Re: RFR: 8359761: JDK 25 RDP1 L10n resource files update [v4]

2025-06-25 Thread Alisen Chung
On Wed, 25 Jun 2025 16:59:09 GMT, Justin Lu wrote: >> I checked the ASCII codes here and on the source file. The updated colons >> are localized to full-width colons as @justin-curtis-lu mentioned in another >> thread here. >> >> I agree. I also believe lines 48 & 49 need to be updated to the

Re: RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v27]

2025-06-25 Thread Alexey Ivanov
On Tue, 24 Jun 2025 05:42:04 GMT, Prasanta Sadhukhan wrote: >> When JRadioButtonMenuItem is called with imageIcon, then only imageIcon is >> shown without radiobutton in WIndowsLookAndFeel as there was no provision of >> drawing the radiobutton alongside icon. >> If icon is not there, the radi

Re: RFR: 8360462: [macosx] row selection not working with Ctrl+Shift+Down/Up in AquaL&F [v2]

2025-06-25 Thread Damon Nguyen
On Wed, 25 Jun 2025 06:52:15 GMT, Prasanta Sadhukhan wrote: >> Ctrl+Shift+Down/Up does extend row selection down or up in Aqua L&F as it >> does in native tree view in Finder. >> Fixed the keybindings. > > Prasanta Sadhukhan has updated the pull request incrementally with one > additional comm

Re: RFR: 8360070: AccessibleText.getBeforeIndex returns null for last character

2025-06-25 Thread Abhishek Kumar
On Wed, 25 Jun 2025 15:05:27 GMT, Alexey Ivanov wrote: > > The spec says > > What spec in particular? https://docs.oracle.com/en/java/javase/24/docs/api/java.desktop/javax/accessibility/AccessibleText.html#getBeforeIndex(int,int) It says, `index - an index within the text` for the `getBeforeIn

Re: RFR: 8360070: AccessibleText.getBeforeIndex returns null for last character

2025-06-25 Thread Alexey Ivanov
On Wed, 25 Jun 2025 04:43:14 GMT, Abhishek Kumar wrote: > The spec says What spec in particular? - PR Comment: https://git.openjdk.org/jdk/pull/25941#issuecomment-3005132532

Re: RFR: 8359266: Delete the usage of AppContext in the GraphicsDevice

2025-06-25 Thread Alexey Ivanov
On Tue, 24 Jun 2025 17:34:54 GMT, Sergey Bylokhov wrote: >>> That only affects the case where the second thread tries to read shared >>> data (unrelated to GraphicsDevice) without synchronization, which was >>> written before the first thread called getFullScreenWindow(). If visibility >>> is

Re: RFR: 8360487: Remove unnecessary List.indexOf key from AbstractMidiDevice.TransmitterList.remove

2025-06-25 Thread Alexander Zvegintsev
On Mon, 23 Jun 2025 08:53:16 GMT, Andrey Turbanov wrote: > No need to call `List.indexOf(Object)` before `List.remove(int)`. Instead we > can call `List.remove(Object)` directly. It's faster and cleaner. Marked as reviewed by azvegint (Reviewer). - PR Review: https://git.openjdk.o

Re: RFR: 8346753: Test javax/swing/JMenuItem/RightLeftOrientation/RightLeftOrientation.java fails on Windows Server 2025 x64 because the icons of RBMenuItem and CBMenuItem are not visible in Nimbus Lo

2025-06-25 Thread Prasanta Sadhukhan
On Wed, 25 Jun 2025 10:01:52 GMT, Alexey Ivanov wrote: > > I dont want to do it as I already told before..Its the same strategy used > > in other recently opensourced tests I mentioned before where > > `SwingUtilities.updateComponentTreeUI(frame)` is used ... > > But missing `SwingUtilities.up

Re: RFR: 8346753: Test javax/swing/JMenuItem/RightLeftOrientation/RightLeftOrientation.java fails on Windows Server 2025 x64 because the icons of RBMenuItem and CBMenuItem are not visible in Nimbus Lo

2025-06-25 Thread Alexey Ivanov
On Wed, 25 Jun 2025 03:14:33 GMT, Prasanta Sadhukhan wrote: > I dont think its a product bug since the L&F was not propagated to all nodes > of the Swing components after L&F change so it should be a test issue which > was solved. I didn't say it was a product bug, it is rather a product *fea

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v3]

2025-06-25 Thread Prasanta Sadhukhan
> BasicTextUI.getVisibleEditorRect wording is rephrased to remove the wording > "Due to an unfortunate set of historical events this method is > inappropriately named". Prasanta Sadhukhan has updated the pull request incrementally with two additional commits since the last revision: - Update

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v2]

2025-06-25 Thread Prasanta Sadhukhan
On Wed, 25 Jun 2025 12:12:54 GMT, Prasanta Sadhukhan wrote: >> If linked with ‘and’, there should be no comma. > > ok..modified.. ok..modified.. - PR Review Comment: https://git.openjdk.org/jdk/pull/25850#discussion_r2166567168

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v2]

2025-06-25 Thread Prasanta Sadhukhan
On Wed, 25 Jun 2025 10:47:27 GMT, Alexey Ivanov wrote: >> We may break the description into paragraphs like this: >> >> >> /** >> * Gets the allocation for the root view. >> * >> * The returned rectangle is unrelated to visibility. >> * It is used to set the size of the

Re: RFR: 8158801: [TEST_BUG] Mixing tests fail because of focus workaround trick [v2]

2025-06-25 Thread Khalid Boulanouare
> Many Mixing tests failed because the work around click lands on the > minimizing area in the window control and causes the tests to fail. > > This fix changes the width of base frames which allows most of tests to pass. Khalid Boulanouare has updated the pull request incrementally with one ad

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v2]

2025-06-25 Thread Alexey Ivanov
On Wed, 25 Jun 2025 10:36:46 GMT, Alexey Ivanov wrote: >> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTextUI.java line >> 1034: >> >>> 1032: * Gets the allocation for the root view. >>> 1033: * The returned rectangle is unrelated to visibility, >>> 1034: * and is u

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v2]

2025-06-25 Thread Alexey Ivanov
On Wed, 25 Jun 2025 07:09:10 GMT, Prasanta Sadhukhan wrote: >> BasicTextUI.getVisibleEditorRect wording is rephrased to remove the wording >> "Due to an unfortunate set of historical events this method is >> inappropriately named". > > Prasanta Sadhukhan has updated the pull request incrementa

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v2]

2025-06-25 Thread Alexey Ivanov
On Wed, 25 Jun 2025 10:33:48 GMT, Alexey Ivanov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with three >> additional commits since the last revision: >> >> - Update javadoc >> - Update javadoc >> - Update javadoc > > src/java.desktop/share/classes/javax/swing/plaf

Re: RFR: 8346753: Test javax/swing/JMenuItem/RightLeftOrientation/RightLeftOrientation.java fails on Windows Server 2025 x64 because the icons of RBMenuItem and CBMenuItem are not visible in Nimbus Lo

2025-06-25 Thread Prasanta Sadhukhan
On Wed, 25 Jun 2025 09:24:21 GMT, Alexey Ivanov wrote: > > Rewriting the test, if needed, could be done as part of new issue but this > > one fixes the issue and let manual testing team run the test in all L&F > > Why not now? We have a bug in the test, your fix doesn't fix the fundamental > p

RFR: 8360487: Remove unnecessary List.indexOf key from AbstractMidiDevice.TransmitterList.remove

2025-06-25 Thread Andrey Turbanov
No need to call `List.indexOf(Object)` before `List.remove(int)`. Instead we can call `List.remove(Object)` directly. It's faster and cleaner. - Commit messages: - [PATCH] Remove unnecessary List.indexOf key from AbstractMidiDevice.TransmitterList.remove Changes: https://git.openj

Re: RFR: 8360487: Remove unnecessary List.indexOf key from AbstractMidiDevice.TransmitterList.remove

2025-06-25 Thread Sergey Bylokhov
On Mon, 23 Jun 2025 08:53:16 GMT, Andrey Turbanov wrote: > No need to call `List.indexOf(Object)` before `List.remove(int)`. Instead we > can call `List.remove(Object)` directly. It's faster and cleaner. looks fine - PR Comment: https://git.openjdk.org/jdk/pull/25931#issuecomment-

Re: RFR: 8352407: PixelInterleavedSampleModel with unused components throws RasterFormatException: Incorrect pixel stride [v2]

2025-06-25 Thread Nikita Gubarkov
On Wed, 19 Mar 2025 13:06:56 GMT, Nikita Gubarkov wrote: >> 8352407: PixelInterleavedSampleModel with unused components throws >> RasterFormatException: Incorrect pixel stride > > Nikita Gubarkov has updated the pull request incrementally with one > additional commit since the last revision: >

RFR: 8158801: [TEST_BUG] Mixing tests fail because of focus workaround trick

2025-06-25 Thread Khalid Boulanouare
Many Mixing tests failed because the work around click lands on the minimizing area in the window control and causes the tests to fail. This fix changes the width of base frames which allows most of tests to pass. - Commit messages: - Problem list files for Mac OS only - Removes p

Re: RFR: 8360462: [macosx] row selection not working with Ctrl+Shift+Down/Up in AquaL&F [v2]

2025-06-25 Thread Prasanta Sadhukhan
> Ctrl+Shift+Down/Up does extend row selection down or up in Aqua L&F as it > does in native tree view in Finder. > Fixed the keybindings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Correct bugid - Changes:

Re: RFR: 8360120: Bundled macOS applications not receiving OpenURL events when launched as subprocess [v2]

2025-06-25 Thread Dmitry Kulikov
> Added an on-demand installation of the native OpenURL event handler to the > `Application.setOpenURIHandler()`. > > This does not break the specification for the affected API, since the > requirement of the application being bundled and containing > `CFBundleURLTypes` in the `Info.plist` is s

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v2]

2025-06-25 Thread Prasanta Sadhukhan
On Fri, 20 Jun 2025 11:26:09 GMT, Alexey Ivanov wrote: >> It is passed to `View.setSize` also so just mentioning one may not be >> right..and I guess {@code allocation} is a no-op as it is not a class.. >> >> Maybe we can mention "The returned rectangle is unrelated to visibility, and >> is us

Re: RFR: 6955128: Spec for javax.swing.plaf.basic.BasicTextUI.getVisibleEditorRect contains inappropriate wording [v2]

2025-06-25 Thread Prasanta Sadhukhan
> BasicTextUI.getVisibleEditorRect wording is rephrased to remove the wording > "Due to an unfortunate set of historical events this method is > inappropriately named". Prasanta Sadhukhan has updated the pull request incrementally with three additional commits since the last revision: - Updat