On Wed, 9 Jul 2025 05:49:30 GMT, Erik Gahlin <egah...@openjdk.org> wrote:
> Could I have a review of the change that ensures that the top frame for I/O > events is not an internal JDK method, such as FileReadEvent::offer? This > change is dependent on > [JDK-8361640](https://bugs.openjdk.org/browse/JDK-8361640). > > Testing: tier1, tier2 + jdk/jdk/jfr > > Thanks > Erik test/jdk/jdk/jfr/event/io/TestIOTopFrame.java line 227: > 225: sc.read(buffers[0]); // 1 > 226: sc.read(buffers); // 2 > 227: try (InputStream is = sc.socket().getInputStream();) { Suggestion: try (InputStream is = sc.socket().getInputStream()) { ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26211#discussion_r2210640867
