RFR(s): 8204958: Minor cleanups for the diagnostic framework
thomas.stuefe at gmail.com
Wed Jun 13 18:15:04 UTC 2018
(jdk-submit tests came back clean)
On Wed, Jun 13, 2018 at 3:41 PM, Thomas Stüfe <thomas.stuefe at gmail.com> wrote:
> Hi all,
> while working on jcmd I saw some minor cleanup possibilities.
> Webrev: http://cr.openjdk.java.net/~stuefe/webrevs/8204958-jcmd-cleanups/webrev.00/webrev/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8204958
> Most of them are const correctness and using initializer lists instead
> of java-style member initialization in these C++ classes.
> Other changes:
> - Added a ResourceMark into the DCmd::parse_and_execute() loop
> - removed DCmdFactory::create_global_DCmd() (and following that,
> DCmdFactory::create_Cheap_instance()) since I did not find anyone
> using that function.
> - Also in DCmdFactory, I removed a number of setters which were never
> called and for attributes which should be immutable: hidden, flags,
> This is really a rather light cleanup. It did not touch any larger issues.
> Thank you,
More information about the hotspot-runtime-dev