> 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:

  fix

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/15950/files
  - new: https://git.openjdk.org/jdk/pull/15950/files/97e4ae30..896065e5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=15950&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=15950&range=00-01

  Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 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

Reply via email to