Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v7]

2022-02-01 Thread Alexey Ivanov
On Tue, 1 Feb 2022 09:40:10 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has update

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v7]

2022-02-01 Thread Prasanta Sadhukhan
> The changes done under JDK-8278175 suppress on a per-file basis various > missing comments that would otherwise trigger doclint warnings. Fixed them so > as to remove the doclint:missing warnings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since t

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v5]

2022-01-30 Thread Prasanta Sadhukhan
On Fri, 28 Jan 2022 19:29:14 GMT, Alexey Ivanov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Address review comment > > src/java.desktop/share/classes/java/awt/Component.java line 4701: > >> 4699: *

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v6]

2022-01-30 Thread Prasanta Sadhukhan
> The changes done under JDK-8278175 suppress on a per-file basis various > missing comments that would otherwise trigger doclint warnings. Fixed them so > as to remove the doclint:missing warnings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since t

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v5]

2022-01-28 Thread Alexey Ivanov
On Mon, 24 Jan 2022 06:03:42 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has updat

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v5]

2022-01-26 Thread Prasanta Sadhukhan
On Mon, 24 Jan 2022 06:03:42 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has updat

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v5]

2022-01-25 Thread Sergey Bylokhov
On Mon, 24 Jan 2022 06:03:42 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has updat

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v5]

2022-01-24 Thread Sergey Bylokhov
On Mon, 24 Jan 2022 06:03:42 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has updat

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v5]

2022-01-24 Thread Alexey Ivanov
On Mon, 24 Jan 2022 06:03:42 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has updat

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-24 Thread Alexey Ivanov
On Mon, 24 Jan 2022 11:34:04 GMT, Alexey Ivanov wrote: >> Since it is mentioned in other places, I would rather not scrap it. Also, >> since in most places it is mentioned as a non-link value, I chose to modify >> it. > > The `` element does not create a link, it uses monospace font to render

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-24 Thread Alexey Ivanov
On Mon, 24 Jan 2022 11:28:06 GMT, Prasanta Sadhukhan wrote: >> If `ShellFolder` remains mentioned, I'd rather use >> `ShellFolder` as it was done originally. >> >> I guess Sergey challenged the fact of referencing `ShellFolder`. The fact >> that it could be `ShellFolder` rather than `File` is

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-24 Thread Prasanta Sadhukhan
On Mon, 24 Jan 2022 10:51:00 GMT, Alexey Ivanov wrote: >> It is already mentioned in getSystemDisplayName, getSystemTypeDescription, >> getSystemIcon, isRoot. Anyways, I have removed the linkage for the new >> addition as it will not be able to resolve. > > If `ShellFolder` remains mentioned, I

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-24 Thread Alexey Ivanov
On Mon, 24 Jan 2022 06:00:02 GMT, Prasanta Sadhukhan wrote: >> src/java.desktop/share/classes/javax/swing/filechooser/FileSystemView.java >> line 353: >> >>> 351: * File(parent, fileName) except when the parent and child >>> are both >>> 352: * special folders, in which case the Fil

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v5]

2022-01-23 Thread Prasanta Sadhukhan
> The changes done under JDK-8278175 suppress on a per-file basis various > missing comments that would otherwise trigger doclint warnings. Fixed them so > as to remove the doclint:missing warnings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since t

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-23 Thread Prasanta Sadhukhan
On Fri, 21 Jan 2022 07:43:17 GMT, Sergey Bylokhov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix > > src/java.desktop/share/classes/javax/swing/filechooser/FileSystemView.java > line 353: > >> 351:

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-20 Thread Sergey Bylokhov
On Thu, 20 Jan 2022 06:12:29 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has updat

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-20 Thread Alexey Ivanov
On Thu, 20 Jan 2022 06:12:29 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has updat

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-20 Thread Alexey Ivanov
On Thu, 20 Jan 2022 15:11:37 GMT, Alexey Ivanov wrote: > Shall we indent `serialVersionUID`? It's for `BCSChild` but its indentation > suggests it's for the surrounding class. It's right below the class declaration. - PR: https://git.openjdk.java.net/jdk/pull/7004

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v4]

2022-01-19 Thread Prasanta Sadhukhan
> The changes done under JDK-8278175 suppress on a per-file basis various > missing comments that would otherwise trigger doclint warnings. Fixed them so > as to remove the doclint:missing warnings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since t

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v3]

2022-01-19 Thread Prasanta Sadhukhan
> The changes done under JDK-8278175 suppress on a per-file basis various > missing comments that would otherwise trigger doclint warnings. Fixed them so > as to remove the doclint:missing warnings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since t

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v2]

2022-01-19 Thread Alexey Ivanov
On Tue, 18 Jan 2022 16:30:07 GMT, Alexey Ivanov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review fix > > src/java.desktop/share/classes/javax/swing/JScrollBar.java line 120: > >> 118: /** >> 119:

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v2]

2022-01-18 Thread Alexey Ivanov
On Mon, 17 Jan 2022 05:26:47 GMT, Prasanta Sadhukhan wrote: >> src/java.desktop/share/classes/javax/swing/undo/UndoableEditSupport.java >> line 150: >> >>> 148: >>> 149: /** >>> 150: * Start UndoableEdit. >> >> What does it mean? Is it clear enough for the class which "managing >>

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v2]

2022-01-18 Thread Alexey Ivanov
On Mon, 17 Jan 2022 05:31:00 GMT, Prasanta Sadhukhan wrote: >> The changes done under JDK-8278175 suppress on a per-file basis various >> missing comments that would otherwise trigger doclint warnings. Fixed them >> so as to remove the doclint:missing warnings. > > Prasanta Sadhukhan has updat

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v2]

2022-01-16 Thread Prasanta Sadhukhan
On Tue, 11 Jan 2022 05:19:56 GMT, Sergey Bylokhov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review fix > > src/java.desktop/share/classes/java/awt/BufferCapabilities.java line 65: > >> 63: >> 64:

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module [v2]

2022-01-16 Thread Prasanta Sadhukhan
> The changes done under JDK-8278175 suppress on a per-file basis various > missing comments that would otherwise trigger doclint warnings. Fixed them so > as to remove the doclint:missing warnings. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since t

Re: RFR: 8278254: Cleanup doclint warnings in java.desktop module

2022-01-10 Thread Sergey Bylokhov
On Mon, 10 Jan 2022 05:25:51 GMT, Prasanta Sadhukhan wrote: > The changes done under JDK-8278175 suppress on a per-file basis various > missing comments that would otherwise trigger doclint warnings. Fixed them so > as to remove the doclint:missing warnings. src/java.desktop/share/classes/jav

RFR: 8278254: Cleanup doclint warnings in java.desktop module

2022-01-09 Thread Prasanta Sadhukhan
The changes done under JDK-8278175 suppress on a per-file basis various missing comments that would otherwise trigger doclint warnings. Fixed them so as to remove the doclint:missing warnings. - Commit messages: - Fix - 8278254: Cleanup doclint warnings in java.desktop module Cha