On Thu, 31 Aug 2023 01:29:28 GMT, David Holmes <[email protected]> wrote:
>> Leonid Mesnik has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> removed unused code.
>
> test/hotspot/jtreg/serviceability/jdwp/DebuggeeLauncher.java line 103:
>
>> 101:
>> 102: @Override
>> 103: public void onStringRead(StreamHandler handler, String line) {
>
> `handler` seems unused now as well. There was obviously some kind of grander
> design for this API that we now seem to be cutting off.
Submitted https://github.com/openjdk/jdk/pull/15502
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15499#discussion_r1311009141
