On Wed, 2 Sep 2026 21:08:36 GMT, Andy Goryachev <[email protected]> wrote:
>> Well, what does this test test then? Your points 1 and 5 are about lack of >> thread safety. A test that they are resolved should fail before the fix and >> not fail after, unless you think that it's trivial, and then there's no need >> for a test. > > well, it did fail twice in a row with threadCount = 49, but would pass > afterwards. > > we **know** the `HashMap` will fail if it's not synchronized, what is the > point of testing it? > unless you think that it's trivial that's why I did not include the test initially. I can either keep the test (esp. since it did fail for me twice w/o syn'd keyword), or remove it. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2197#discussion_r3918740100
