Re: RFR: 8297310: Remove unimplemented HeapInspection::iterate_over_heap [v2]

2022-11-22 Thread Kim Barrett
> Please review this trivial change to remove a function declaration that is > never defined. 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 contai

Re: RFR: 8297310: Remove unimplemented HeapInspection::iterate_over_heap [v2]

2022-11-22 Thread Kim Barrett
On Tue, 22 Nov 2022 12:36:07 GMT, Aleksey Shipilev 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 two additional >> commi

Re: RFR: 8297310: Remove unimplemented HeapInspection::iterate_over_heap

2022-11-22 Thread Aleksey Shipilev
On Tue, 22 Nov 2022 00:40:09 GMT, Kim Barrett wrote: > Please review this trivial change to remove a function declaration that is > never defined. Looks fine and trivial. - Marked as reviewed by shade (Reviewer). PR: https://git.openjdk.org/jdk/pull/11280

Re: RFR: 8297310: Remove unimplemented HeapInspection::iterate_over_heap

2022-11-21 Thread Chris Plummer
On Tue, 22 Nov 2022 00:40:09 GMT, Kim Barrett wrote: > Please review this trivial change to remove a function declaration that is > never defined. Marked as reviewed by cjplummer (Reviewer). - PR: https://git.openjdk.org/jdk/pull/11280

Re: RFR: 8297310: Remove unimplemented HeapInspection::iterate_over_heap

2022-11-21 Thread Alex Menkov
On Tue, 22 Nov 2022 00:40:09 GMT, Kim Barrett wrote: > Please review this trivial change to remove a function declaration that is > never defined. Marked as reviewed by amenkov (Reviewer). - PR: https://git.openjdk.org/jdk/pull/11280

RFR: 8297310: Remove unimplemented HeapInspection::iterate_over_heap

2022-11-21 Thread Kim Barrett
Please review this trivial change to remove a function declaration that is never defined. - Commit messages: - remove decl Changes: https://git.openjdk.org/jdk/pull/11280/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=11280&range=00 Issue: https://bugs.openjdk.org/browse