Re: RFR: JDK-8303071: Memory leaks in libjdwp [v2]

2023-02-24 Thread Chris Plummer
On Wed, 22 Feb 2023 15:19:56 GMT, Justin King wrote: >> Fix minor memory leaks in libjdwp. > > Justin King has updated the pull request incrementally with one additional > commit since the last revision: > > Use jvmtiDeallocate wrapper > > Signed-off-by: Justin King Approved. @jcking

Re: RFR: JDK-8303071: Memory leaks in libjdwp [v2]

2023-02-23 Thread David Holmes
On Wed, 22 Feb 2023 15:19:56 GMT, Justin King wrote: >> Fix minor memory leaks in libjdwp. > > Justin King has updated the pull request incrementally with one additional > commit since the last revision: > > Use jvmtiDeallocate wrapper > > Signed-off-by: Justin King Thanks! Now I can j

Re: RFR: JDK-8303071: Memory leaks in libjdwp [v2]

2023-02-23 Thread Justin King
On Wed, 22 Feb 2023 15:19:56 GMT, Justin King wrote: >> Fix minor memory leaks in libjdwp. > > Justin King has updated the pull request incrementally with one additional > commit since the last revision: > > Use jvmtiDeallocate wrapper > > Signed-off-by: Justin King callback->alloc is

Re: RFR: JDK-8303071: Memory leaks in libjdwp [v2]

2023-02-23 Thread David Holmes
On Wed, 22 Feb 2023 15:19:56 GMT, Justin King wrote: >> Fix minor memory leaks in libjdwp. > > Justin King has updated the pull request incrementally with one additional > commit since the last revision: > > Use jvmtiDeallocate wrapper > > Signed-off-by: Justin King Just a sanity check

Re: RFR: JDK-8303071: Memory leaks in libjdwp [v2]

2023-02-23 Thread Justin King
On Wed, 22 Feb 2023 15:19:56 GMT, Justin King wrote: >> Fix minor memory leaks in libjdwp. > > Justin King has updated the pull request incrementally with one additional > commit since the last revision: > > Use jvmtiDeallocate wrapper > > Signed-off-by: Justin King Tests look unrelate

Re: RFR: JDK-8303071: Memory leaks in libjdwp [v2]

2023-02-22 Thread Serguei Spitsyn
On Wed, 22 Feb 2023 15:19:56 GMT, Justin King wrote: >> Fix minor memory leaks in libjdwp. > > Justin King has updated the pull request incrementally with one additional > commit since the last revision: > > Use jvmtiDeallocate wrapper > > Signed-off-by: Justin King It looks good. Than

Re: RFR: JDK-8303071: Memory leaks in libjdwp [v2]

2023-02-22 Thread Justin King
> Fix minor memory leaks in libjdwp. Justin King has updated the pull request incrementally with one additional commit since the last revision: Use jvmtiDeallocate wrapper Signed-off-by: Justin King - Changes: - all: https://git.openjdk.org/jdk/pull/12716/files - new: h

RFR: JDK-8303071: Memory leaks in libjdwp

2023-02-22 Thread Justin King
Fix minor memory leaks in libjdwp. - Commit messages: - Memory leaks in libjdwp Changes: https://git.openjdk.org/jdk/pull/12716/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=12716&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8303071 Stats: 6 lines in 2 files cha