RFR: 8303773: Replace "main.wrapper" with "test.thread.factory" property in test code [v3]
Leonid Mesnik
lmesnik at openjdk.org
Fri Sep 29 18:54:21 UTC 2023
> The main.wrapper was the first name for jtreg test thread factory plugin. However, during integration of this feature in jtreg it was decided to use test.thread.factory name. So this fix just renames "main.wrapper" property to "test.thread.factory" so it is more compliant with jtreg naming. Also, it makes more sense for tests when it is used to create other then main threads in test.
> Testing: tier1-5.
> Verified that "main.wrapper" is not used in test sources anymore.
>
> I haven't rename DebugeeWrapperd and MainWrapper classes in JDI test frameworks because they are actually more main wrappers than thread factories.
Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
spaces added.
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/15950/files
- new: https://git.openjdk.org/jdk/pull/15950/files/896065e5..284a341d
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=15950&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=15950&range=01-02
Stats: 3 lines in 3 files changed: 0 ins; 0 del; 3 mod
Patch: https://git.openjdk.org/jdk/pull/15950.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/15950/head:pull/15950
PR: https://git.openjdk.org/jdk/pull/15950
More information about the serviceability-dev
mailing list