Re: RFR: 8327056: Remove unused static char array in JvmtiAgentList::lookup

2024-02-29 Thread Denghui Dong
On Thu, 29 Feb 2024 15:02:02 GMT, Denghui Dong wrote: > Hi, > > Please help review this trivial change that removed an unused static char > array in JvmtiAgentList::lookup. > > Thanks Thanks for the review. The GHA failure ([linux-cross-compile / build (riscv64)](https://github.com/D-D-H/jdk

Re: RFR: 8327056: Remove unused static char array in JvmtiAgentList::lookup

2024-02-29 Thread Serguei Spitsyn
On Thu, 29 Feb 2024 15:02:02 GMT, Denghui Dong wrote: > Hi, > > Please help review this trivial change that removed an unused static char > array in JvmtiAgentList::lookup. > > Thanks Looks good and trivial. - Marked as reviewed by sspitsyn (Reviewer). PR Review: https://git.op

Re: RFR: 8327056: Remove unused static char array in JvmtiAgentList::lookup

2024-02-29 Thread Alex Menkov
On Thu, 29 Feb 2024 15:02:02 GMT, Denghui Dong wrote: > Hi, > > Please help review this trivial change that removed an unused static char > array in JvmtiAgentList::lookup. > > Thanks Marked as reviewed by amenkov (Reviewer). Looks good and trivial - PR Review: https://git.open

RFR: 8327056: Remove unused static char array in JvmtiAgentList::lookup

2024-02-29 Thread Denghui Dong
Hi, Please help review this trivial change that removed an unused static char array in JvmtiAgentList::lookup. Thanks - Commit messages: - 8327056: Remove unused static char array in JvmtiAgentList::lookup Changes: https://git.openjdk.org/jdk/pull/18066/files Webrev: https://web