On Wed, 30 Aug 2023 23:28:23 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Leonid Mesnik has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> restore errorhandler
>
> test/hotspot/jtreg/serviceability/jdwp/DebuggeeLauncher.java line 110:
>
>> 108: @Override
>> 109: public void onStringRead(StreamHandler handler, String line) {
>> 110: processDebuggeeOutput(line);
>
> You might want to look at #4560, which added the onDebuggeeError() code,
> which is no longer being called from anywhere after this change. @alexmenkov
This method existed from beginning. I'll remove it.
See:
https://github.com/openjdk/jdk/commit/4f55b6c7e03687e9b799456d6e4d0b82027566ec
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15499#discussion_r1310959495
