Re: RFR: 8325055: Rename Injector.h [v2]

2024-02-02 Thread Serguei Spitsyn
On Wed, 31 Jan 2024 15:15:16 GMT, Kim Barrett wrote: >> Please review this trivial change that renames the file >> test/hotspot/jtreg/vmTestbase/nsk/share/jvmti/Injector.h to Injector.hpp. >> >> Testing: mach5 tier1 > > Kim Barrett has updated the pull request incrementally with one additional

Re: RFR: 8325055: Rename Injector.h [v2]

2024-02-01 Thread Alex Menkov
On Wed, 31 Jan 2024 15:15:16 GMT, Kim Barrett wrote: >> Please review this trivial change that renames the file >> test/hotspot/jtreg/vmTestbase/nsk/share/jvmti/Injector.h to Injector.hpp. >> >> Testing: mach5 tier1 > > Kim Barrett has updated the pull request incrementally with one additional

Re: RFR: 8325055: Rename Injector.h [v2]

2024-02-01 Thread Kim Barrett
On Thu, 1 Feb 2024 07:12:08 GMT, David Holmes wrote: > Seems fine. Thanks Trivial? - PR Comment: https://git.openjdk.org/jdk/pull/17656#issuecomment-1922036034

Re: RFR: 8325055: Rename Injector.h [v2]

2024-01-31 Thread David Holmes
On Wed, 31 Jan 2024 15:15:16 GMT, Kim Barrett wrote: >> Please review this trivial change that renames the file >> test/hotspot/jtreg/vmTestbase/nsk/share/jvmti/Injector.h to Injector.hpp. >> >> Testing: mach5 tier1 > > Kim Barrett has updated the pull request incrementally with one additional

Re: RFR: 8325055: Rename Injector.h [v2]

2024-01-31 Thread Kim Barrett
> Please review this trivial change that renames the file > test/hotspot/jtreg/vmTestbase/nsk/share/jvmti/Injector.h to Injector.hpp. > > Testing: mach5 tier1 Kim Barrett has updated the pull request incrementally with one additional commit since the last revision: fix name in README ---