Re: RFR: 8326090: Rename jvmti_aod.h [v2]

2024-02-20 Thread Kim Barrett
> Please review this trivial change that renames the file > test/hotspot/jtreg/vmTestbase/nsk/share/jvmti/aod/jvmti_aod.h to > jvmti_aod.hpp, > and replace uses of NULL in the file. > > The #include updates were performed mechanically, and build would fail if > there were mistakes. All of the inc

Re: RFR: 8326090: Rename jvmti_aod.h [v2]

2024-02-20 Thread Kim Barrett
On Tue, 20 Feb 2024 15:15:35 GMT, Coleen Phillimore wrote: >> Kim Barrett has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains three additional >> co

Re: RFR: 8326090: Rename jvmti_aod.h

2024-02-20 Thread Serguei Spitsyn
On Fri, 16 Feb 2024 21:42:18 GMT, Kim Barrett wrote: > Please review this trivial change that renames the file > test/hotspot/jtreg/vmTestbase/nsk/share/jvmti/aod/jvmti_aod.h to > jvmti_aod.hpp, > and replace uses of NULL in the file. > > The #include updates were performed mechanically, and bu

Re: RFR: 8326090: Rename jvmti_aod.h

2024-02-20 Thread Coleen Phillimore
On Fri, 16 Feb 2024 21:42:18 GMT, Kim Barrett wrote: > Please review this trivial change that renames the file > test/hotspot/jtreg/vmTestbase/nsk/share/jvmti/aod/jvmti_aod.h to > jvmti_aod.hpp, > and replace uses of NULL in the file. > > The #include updates were performed mechanically, and bu

RFR: 8326090: Rename jvmti_aod.h

2024-02-16 Thread Kim Barrett
Please review this trivial change that renames the file test/hotspot/jtreg/vmTestbase/nsk/share/jvmti/aod/jvmti_aod.h to jvmti_aod.hpp, and replace uses of NULL in the file. The #include updates were performed mechanically, and build would fail if there were mistakes. All of the including files ha