Re: RFR: 8285306: Fix typos in java.desktop [v6]

2022-06-11 Thread Magnus Ihse Bursie
On Fri, 13 May 2022 16:53:00 GMT, Magnus Ihse Bursie wrote: >> I ran `codespell` on the `src/java.desktop` directory, and accepted those >> changes where it indeed discovered real typos. >> >> I ignored typos in public methods and variables. Maybe they can be fixed >> later on without much fan

Re: RFR: 8285306: Fix typos in java.desktop

2022-05-13 Thread Alexey Ivanov
On Wed, 27 Apr 2022 03:38:09 GMT, Phil Race wrote: > Please exclude ALL 3rd party libraries from this PR. Are these the only files to be excluded? src/java.desktop/share/native/libfreetype/src/autofit/afcjk.c src/java.desktop/share/native/libfreetype/src/autofit/aflatin.c src/java.desktop/shar

Re: RFR: 8285306: Fix typos in java.desktop [v6]

2022-05-13 Thread Alexey Ivanov
On Fri, 13 May 2022 16:48:35 GMT, Magnus Ihse Bursie wrote: >> src/java.desktop/share/classes/sun/swing/text/TextComponentPrintable.java >> line 772: >> >>> 770: /* >>> 771: * we do not store the same value as previous. >>> in our >>> 772:

Re: RFR: 8285306: Fix typos in java.desktop [v6]

2022-05-13 Thread Alexey Ivanov
On Fri, 13 May 2022 16:53:31 GMT, Magnus Ihse Bursie wrote: >> Nearly 500 files are too many. Smaller chunks would be easier to review. >> >> Some of the native code files could come from upstream libraries. > > @aivanov-jdk While I approve of finding and fixing spelling mistakes, > grammatical

Re: RFR: 8285306: Fix typos in java.desktop [v6]

2022-05-13 Thread Magnus Ihse Bursie
On Thu, 28 Apr 2022 21:25:02 GMT, Alexey Ivanov wrote: >> Magnus Ihse Bursie has updated the pull request incrementally with 17 >> additional commits since the last revision: >> >> - Update src/java.desktop/windows/classes/sun/awt/windows/WPrinterJob.java >> >>Co-authored-by: Alexey Iv

Re: RFR: 8285306: Fix typos in java.desktop [v6]

2022-05-13 Thread Magnus Ihse Bursie
On Thu, 28 Apr 2022 20:37:22 GMT, Alexey Ivanov wrote: >> Magnus Ihse Bursie has updated the pull request incrementally with 17 >> additional commits since the last revision: >> >> - Update src/java.desktop/windows/classes/sun/awt/windows/WPrinterJob.java >> >>Co-authored-by: Alexey Iv

Re: RFR: 8285306: Fix typos in java.desktop [v6]

2022-05-13 Thread Magnus Ihse Bursie
> I ran `codespell` on the `src/java.desktop` directory, and accepted those > changes where it indeed discovered real typos. > > I ignored typos in public methods and variables. Maybe they can be fixed > later on without much fanfare, if they are in internal classes. Typos in > exposed APIs are

Re: RFR: 8285306: Fix typos in java.desktop [v5]

2022-05-13 Thread Magnus Ihse Bursie
> I ran `codespell` on the `src/java.desktop` directory, and accepted those > changes where it indeed discovered real typos. > > I ignored typos in public methods and variables. Maybe they can be fixed > later on without much fanfare, if they are in internal classes. Typos in > exposed APIs are

Re: RFR: 8285306: Fix typos in java.desktop [v4]

2022-05-13 Thread Magnus Ihse Bursie
> I ran `codespell` on the `src/java.desktop` directory, and accepted those > changes where it indeed discovered real typos. > > I ignored typos in public methods and variables. Maybe they can be fixed > later on without much fanfare, if they are in internal classes. Typos in > exposed APIs are

Re: RFR: 8285306: Fix typos in java.desktop [v3]

2022-05-13 Thread Magnus Ihse Bursie
On Thu, 28 Apr 2022 16:48:04 GMT, Alexey Ivanov wrote: >> Magnus Ihse Bursie has updated the pull request incrementally with four >> additional commits since the last revision: >> >> - Update src/java.desktop/share/classes/javax/swing/JLayeredPane.java >> >>Co-authored-by: Alexey Ivano

Re: RFR: 8285306: Fix typos in java.desktop [v3]

2022-05-13 Thread Magnus Ihse Bursie
> I ran `codespell` on the `src/java.desktop` directory, and accepted those > changes where it indeed discovered real typos. > > I ignored typos in public methods and variables. Maybe they can be fixed > later on without much fanfare, if they are in internal classes. Typos in > exposed APIs are

Re: RFR: 8285306: Fix typos in java.desktop [v2]

2022-05-13 Thread Magnus Ihse Bursie
> I ran `codespell` on the `src/java.desktop` directory, and accepted those > changes where it indeed discovered real typos. > > I ignored typos in public methods and variables. Maybe they can be fixed > later on without much fanfare, if they are in internal classes. Typos in > exposed APIs are

Re: RFR: 8285306: Fix typos in java.desktop

2022-04-28 Thread Alexey Ivanov
On Thu, 21 Apr 2022 08:35:36 GMT, Magnus Ihse Bursie wrote: > I ran `codespell` on the `src/java.desktop` directory, and accepted those > changes where it indeed discovered real typos. > > I ignored typos in public methods and variables. Maybe they can be fixed > later on without much fanfare,

Re: RFR: 8285306: Fix typos in java.desktop

2022-04-26 Thread Phil Race
On Thu, 21 Apr 2022 08:35:36 GMT, Magnus Ihse Bursie wrote: > I ran `codespell` on the `src/java.desktop` directory, and accepted those > changes where it indeed discovered real typos. > > I ignored typos in public methods and variables. Maybe they can be fixed > later on without much fanfare,

RFR: 8285306: Fix typos in java.desktop

2022-04-21 Thread Magnus Ihse Bursie
I ran `codespell` on the `src/java.desktop` directory, and accepted those changes where it indeed discovered real typos. I ignored typos in public methods and variables. Maybe they can be fixed later on without much fanfare, if they are in internal classes. Typos in exposed APIs are likely here