RFR: 8375480: Remove usage of AppContext from javax/swing/text [v2]
Phil Race
prr at openjdk.org
Wed Jan 21 18:48:03 UTC 2026
On Wed, 21 Jan 2026 09:29:15 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8375350
>
> src/java.desktop/share/classes/javax/swing/text/html/parser/Element.java line 109:
>
>> 107: this.index = index;
>> 108: if (index > getMaxIndex()) {
>> 109: maxIndex = index;
>
> guess before AppContext was added the code was
> `maxIndex = Math.max(maxIndex, index);`
>
> https://hg.openjdk.org/jdk8/jdk8/jdk/rev/0967103c1b65
I prefer it as I have it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29259#discussion_r2713879933
More information about the client-libs-dev
mailing list