RFR: 7903519 : jtreg/jtharness is missing features for basic crash testing [v4]

andrlos duke at openjdk.org
Thu Dec 14 13:58:15 UTC 2023


On Thu, 14 Dec 2023 09:17:19 GMT, Vladimir Sitnikov <vsitnikov at openjdk.org> wrote:

>> @vlsi it just occured to me that I can initialize the `statusModifier` during the initialization of the `Script` instance, therefore there should be no problem reading it from multiple threads.. WDYT? Sorry for throwing half witted ideas out here.. and I read the blog you shared I can see why you dont like the current approach and I agree it must be altered.
>
> Initializing it along with `Script` initialization sounds reasonable to me, and it might indeed simplify the code.
> For instance, you could treat `null` implementation as a missing one, so you don not need to have "default implementation" in the code.

okay, should be done and yeah you are right, in this simplified code there is no need for default implementation and it seems a lot nicer this way, thanks!

-------------

PR Review Comment: https://git.openjdk.org/jtharness/pull/57#discussion_r1426705119


More information about the jtharness-dev mailing list