> Could I have a review of a PR that fixes several issues with the JFR 
> exception events:
> 
> - Error and its subclasses emitted two JavaExceptionThrow events
> - Error was counted twice in ExceptionStatistics
> - Constructors that only take the cause lost the message
> 
> 
> The testing was inadequate, so I rewrote most of it.
> 
> Testing: jdk_lang, jdk_jfr
> 
> Thanks
> Erik
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Erik Gahlin has updated the pull request incrementally with one additional 
commit since the last revision:

  Fix typos

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/33828/files
  - new: https://git.openjdk.org/jdk/pull/33828/files/a8399a18..811a43d2

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=33828&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=33828&range=00-01

  Stats: 19 lines in 1 file changed: 3 ins; 1 del; 15 mod
  Patch: https://git.openjdk.org/jdk/pull/33828.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/33828/head:pull/33828

PR: https://git.openjdk.org/jdk/pull/33828

Reply via email to