RFR: 8276337: Use override specifier in HeapDumper [v2]

[Thomas Stuefe]

On Fri, 5 Nov 2021 14:14:43 GMT, Leo Korinth <lkorinth at openjdk.org> 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