RFR: 8264309: JFR: Improve .jfc parser

Erik Gahlin egahlin at openjdk.java.net
Fri Mar 26 18:46:37 UTC 2021


Hi,

Could I have a review of a fix that removes unnecessary number of allocations of StringBuilders. Events are now referred to by name, not path. The code should reflect that. The method getSettings() is not used and can be removed.

Testing: jdk/jdk/jfr

Thanks
Erik

-------------

Commit messages:
 - Remove getSettings()
 - Initial

Changes: https://git.openjdk.java.net/jdk/pull/3222/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3222&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8264309
  Stats: 15 lines in 1 file changed: 1 ins; 6 del; 8 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3222.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3222/head:pull/3222

PR: https://git.openjdk.java.net/jdk/pull/3222


More information about the hotspot-jfr-dev mailing list