On Mon, 4 Aug 2025 21:11:42 GMT, Phil Race <p...@openjdk.org> wrote:
>> Convert StringContent's Position usage tracking to use a WeakReference
>>
>> The test that is added only implicitly tests this but I added it because of
>> a complete lack of any test for this code.
>
> Phil Race has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8364230
test/jdk/javax/swing/text/AbstractDocument/StringContentPositionTest.java line
33:
> 31: */
> 32:
> 33: public class StringContentPositionTest {
I believe @bug is not added because this is not strictly a regression test for
this issue as I see it does not fail even without the fix?
Since `System.gc` is used, can we try restricting the heap size in commandline
to see if it will fail without the fix and pass with it?
test/jdk/javax/swing/text/AbstractDocument/StringContentPositionTest.java line
54:
> 52: for (int i = 0; i < SIZE; i++) {
> 53: SC.insertString(0, TEXT);
> 54: positions[i] = SC.createPosition(5);
should it be LEN instead of 5?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26519#discussion_r2255919971
PR Review Comment: https://git.openjdk.org/jdk/pull/26519#discussion_r2255917014
