On Fri, 5 Nov 2021 14:14:43 GMT, Leo Korinth <[email protected]> wrote:
>> Use override specifier in HeapDumper. It is safer.
>
> Leo Korinth has updated the pull request incrementally with one additional
> commit since the last revision:
>
> restart failed github tests
Hi Leo, seems fine.
..Thomas
src/hotspot/share/services/heapDumper.cpp line 626:
> 624: void deactivate() override { flush();
> _backend.deactivate(); }
> 625: // Get the backend pointer, used by parallel dump writer.
> 626: CompressionBackend* backend_ptr() { return &_backend; }
Unrelated, but could be a const method, no?
-------------
Marked as reviewed by stuefe (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/6274
