Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v4]

2021-03-10 Thread Alexey Ivanov
On Wed, 10 Mar 2021 12:28:18 GMT, Prasanta Sadhukhan wrote: >> BasicTextUI: installDefaults javadoc specifies only font, foreground and >> background properties are set if their current value is either null or a >> UIResource >> and other properties are set if the current value is null >>

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v3]

2021-03-10 Thread Prasanta Sadhukhan
On Wed, 10 Mar 2021 12:19:45 GMT, Alexey Ivanov wrote: >> I think these are nouns and as per I know, we shouldn't place comma before >> "and" when we are simply listing the items. > >> I think these are nouns and as per I know, we shouldn't place comma before >> "and" when we are simply

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v4]

2021-03-10 Thread Prasanta Sadhukhan
> BasicTextUI: installDefaults javadoc specifies only font, foreground and > background properties are set if their current value is either null or a > UIResource > and other properties are set if the current value is null > but in reality all properties such as font, foreground, background,

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v3]

2021-03-10 Thread Alexey Ivanov
On Wed, 10 Mar 2021 12:13:43 GMT, Prasanta Sadhukhan wrote: > I think these are nouns and as per I know, we shouldn't place comma before > "and" when we are simply listing the items. This is called [Oxford

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v3]

2021-03-10 Thread Prasanta Sadhukhan
On Wed, 10 Mar 2021 11:49:40 GMT, Alexey Ivanov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> copyright change > > src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTextUI.java line > 306: > >>

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v3]

2021-03-10 Thread Alexey Ivanov
On Wed, 10 Mar 2021 04:38:19 GMT, Prasanta Sadhukhan wrote: >> BasicTextUI: installDefaults javadoc specifies only font, foreground and >> background properties are set if their current value is either null or a >> UIResource >> and other properties are set if the current value is null >>

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v3]

2021-03-10 Thread Alexander Zvegintsev
On Wed, 10 Mar 2021 04:38:19 GMT, Prasanta Sadhukhan wrote: >> BasicTextUI: installDefaults javadoc specifies only font, foreground and >> background properties are set if their current value is either null or a >> UIResource >> and other properties are set if the current value is null >>

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v3]

2021-03-09 Thread Prasanta Sadhukhan
On Tue, 9 Mar 2021 21:17:13 GMT, Alexey Ivanov wrote: >> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTextUI.java line >> 306: >> >>> 304: * Initializes component properties, such as font, foreground, >>> 305: * background, caret color, selection color, selected text

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v3]

2021-03-09 Thread Prasanta Sadhukhan
> BasicTextUI: installDefaults javadoc specifies only font, foreground and > background properties are set if their current value is either null or a > UIResource > and other properties are set if the current value is null > but in reality all properties such as font, foreground, background,

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation [v2]

2021-03-09 Thread Prasanta Sadhukhan
> BasicTextUI: installDefaults javadoc specifies only font, foreground and > background properties are set if their current value is either null or a > UIResource > and other properties are set if the current value is null > but in reality all properties such as font, foreground, background,

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation

2021-03-09 Thread Alexey Ivanov
On Tue, 9 Mar 2021 21:31:03 GMT, Alexander Zuev wrote: > I'm not an expert in English but i think comma after set is not needed. You're right, comma is not needed in this case. I updated my comment and removed the comma. - PR: https://git.openjdk.java.net/jdk/pull/2888

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation

2021-03-09 Thread Alexander Zuev
On Tue, 9 Mar 2021 21:12:08 GMT, Alexey Ivanov wrote: >> BasicTextUI: installDefaults javadoc specifies only font, foreground and >> background properties are set if their current value is either null or a >> UIResource >> and other properties are set if the current value is null >> but in

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation

2021-03-09 Thread Alexey Ivanov
On Tue, 9 Mar 2021 13:34:20 GMT, Alexander Zvegintsev wrote: >> BasicTextUI: installDefaults javadoc specifies only font, foreground and >> background properties are set if their current value is either null or a >> UIResource >> and other properties are set if the current value is null >>

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation

2021-03-09 Thread Alexey Ivanov
On Tue, 9 Mar 2021 08:09:04 GMT, Prasanta Sadhukhan wrote: > BasicTextUI: installDefaults javadoc specifies only font, foreground and > background properties are set if their current value is either null or a > UIResource > and other properties are set if the current value is null > but in

Re: RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation

2021-03-09 Thread Alexander Zvegintsev
On Tue, 9 Mar 2021 08:09:04 GMT, Prasanta Sadhukhan wrote: > BasicTextUI: installDefaults javadoc specifies only font, foreground and > background properties are set if their current value is either null or a > UIResource > and other properties are set if the current value is null > but in

RFR: 6251901: BasicTextUI: installDefaults method are contrary to the documentation

2021-03-09 Thread Prasanta Sadhukhan
BasicTextUI: installDefaults javadoc specifies only font, foreground and background properties are set if their current value is either null or a UIResource and other properties are set if the current value is null but in reality all properties such as font, foreground, background, caret