Re: RFR: 8311247: Some cpp files are compiled with -std:c11 flag

2023-07-03 Thread Daniel Jeliński
On Mon, 3 Jul 2023 20:17:23 GMT, Alexey Ivanov wrote: >> Please review this patch that configures C++ arguments on build jobs that >> involve compiling CPP files. As a result of this change, CPP files are >> compiled with `-std:c++14` command line argument instead of `-std:c11`, >> which is us

Re: RFR: JDK-8302618: [MacOS] Problem typing uppercase letters with java.awt.Robot when moving mouse [v2]

2023-07-03 Thread Alexey Ivanov
On Mon, 3 Jul 2023 21:26:13 GMT, Harshitha Onkar wrote: >> I disagree: the common style throughout this file is to put the opening >> brace on the following line. A few lines above, `GetCGKeyCode` follows the >> same style. > > @aivanov-jdk Yeah, I wasn't sure of open brace placement in native

Re: RFR: JDK-8302618: [MacOS] Problem typing uppercase letters with java.awt.Robot when moving mouse [v2]

2023-07-03 Thread Harshitha Onkar
On Mon, 3 Jul 2023 20:28:35 GMT, Alexey Ivanov wrote: >> src/java.desktop/macosx/native/libawt_lwawt/awt/CRobot.m line 426: >> >>> 424: } >>> 425: >>> 426: static inline int GetCGKeyMask(int cgKeyCode) >> >> Suggestion: >> >> static inline int GetCGKeyMask(int cgKeyCode) { > > I disagree: the

Integrated: JDK-8311109: tautological-compare warning in awt_Win32GraphicsDevice.cpp

2023-07-03 Thread Harshitha Onkar
On Mon, 3 Jul 2023 18:28:40 GMT, Harshitha Onkar wrote: > Comparison typo fix. Should be monitor2 (mi2) instead of monitor1 (mi1). This pull request has now been integrated. Changeset: d8a01216 Author:Harshitha Onkar URL: https://git.openjdk.org/jdk/commit/d8a012161ad68d08467a6632c2

Re: RFR: JDK-8302618: [MacOS] Problem typing uppercase letters with java.awt.Robot when moving mouse [v2]

2023-07-03 Thread Alexey Ivanov
On Mon, 3 Jul 2023 04:45:38 GMT, Abhishek Kumar wrote: >> Harshitha Onkar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> kCGEventFlagMaskSecondaryFn added > > src/java.desktop/macosx/native/libawt_lwawt/awt/CRobot.m line 426: > >> 424:

Re: RFR: JDK-8302618: [MacOS] Problem typing uppercase letters with java.awt.Robot when moving mouse [v2]

2023-07-03 Thread Alexey Ivanov
On Sat, 1 Jul 2023 00:29:11 GMT, Harshitha Onkar wrote: >> **Problem:** >> >> Robot erroneously produces lowercase letter when mouse is moved (manually) >> in unison with Robot's keyEvents on MacOS. This issue was originally logged >> by a developer of an on-screen accessibility keyboard - To

Re: RFR: 8311247: Some cpp files are compiled with -std:c11 flag

2023-07-03 Thread Alexey Ivanov
On Mon, 3 Jul 2023 17:15:17 GMT, Daniel Jeliński wrote: > Please review this patch that configures C++ arguments on build jobs that > involve compiling CPP files. As a result of this change, CPP files are > compiled with `-std:c++14` command line argument instead of `-std:c11`, which > is used

Re: RFR: JDK-8311109: tautological-compare warning in awt_Win32GraphicsDevice.cpp

2023-07-03 Thread Alexey Ivanov
On Mon, 3 Jul 2023 18:28:40 GMT, Harshitha Onkar wrote: > Comparison typo fix. Should be monitor2 (mi2) instead of monitor1 (mi1). Marked as reviewed by aivanov (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/14760#pullrequestreview-1511652869

RFR: 8311247: Some cpp files are compiled with -std:c11 flag

2023-07-03 Thread Daniel Jeliński
Please review this patch that configures C++ arguments on build jobs that involve compiling CPP files. As a result of this change, CPP files are compiled with `-std:c++14` command line argument instead of `-std:c11`, which is used when C++ arguments are not configured. While at it, I simplified

RFR: JDK-8311109: tautological-compare warning in awt_Win32GraphicsDevice.cpp

2023-07-03 Thread Harshitha Onkar
Comparison typo fix. Should be monitor2 (mi2) instead of monitor1 (mi1). - Commit messages: - changed to mi2 Changes: https://git.openjdk.org/jdk/pull/14760/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14760&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8311109

Re: RFR: JDK-8302618: Problem typing uppercase letters with java.awt.Robot on Mac when moving mouse [v3]

2023-07-03 Thread Harshitha Onkar
> **Problem:** > > Robot erroneously produces lowercase letter when mouse is moved (manually) in > unison with Robot's keyEvents on MacOS. This issue was originally logged by a > developer of an on-screen accessibility keyboard - TouchBoard. Originally > reported at https://github.com/adoptium

Re: RFR: 8283214: [macos] Screen magnifier does not show the magnified text for JcomboBox [v3]

2023-07-03 Thread Alexey Ivanov
On Mon, 3 Jul 2023 11:11:47 GMT, Alexey Ivanov wrote: >>> What if `JComboBox` uses a non-standard renderer which takes a property >>> from an object? Let's assume, `JComboBox` is populated with `Rectangle` >>> objects, a custom renderer is set which displays the value of the `bottom` >>> field

Re: RFR: 8283214: [macos] Screen magnifier does not show the magnified text for JcomboBox [v3]

2023-07-03 Thread Alexey Ivanov
On Mon, 3 Jul 2023 09:14:23 GMT, Abhishek Kumar wrote: > > What if `JComboBox` uses a non-standard renderer which takes a property > > from an object? Let's assume, `JComboBox` is populated with `Rectangle` > > objects, a custom renderer is set which displays the value of the `bottom` > > fiel

Integrated: 8307934 JRobot.moveMouseTo must access component on EDT

2023-07-03 Thread Renjith Kannath Pariyangad
On Wed, 7 Jun 2023 12:54:54 GMT, Renjith Kannath Pariyangad wrote: > Hi Reviewers, > > I have updated the JRobot.java file for enabling EDT support, along with I > have removed warning BUTTON1_MASK with 'BUTTON1_DOWN_MASK'. > > Regards, > Renjith This pull request has now been integrated. C

Re: RFR: 8283214: [macos] Screen magnifier does not show the magnified text for JcomboBox [v3]

2023-07-03 Thread Abhishek Kumar
On Fri, 30 Jun 2023 19:36:43 GMT, Alexey Ivanov wrote: >> I need to check with custom renderer. >> >> If I understand correctly, with current fix the `getSelectedItem` method >> will return the `rectangle` object and then calling `toString` method will >> return rectangle's dimension instead

Re: RFR: 6928542: Chinese characters in RTF are not decoded

2023-07-03 Thread Ichiroh Takiguchi
On Thu, 20 Apr 2023 05:47:35 GMT, Ichiroh Takiguchi wrote: > "character set of font" (font charset) table was created by "Rich Text Format > Specification 1.9.1" > https://interoperability.blob.core.windows.net/files/Archive_References/[MSFT-RTF].pdf > It refers windgi.h > https://learn.microso