RFR: 8332314: Add window size configuration option to JavaShellToolBuilder interface [v2]

Archie Cobbs acobbs at openjdk.org
Tue Jun 18 16:24:05 UTC 2024


On Tue, 18 Jun 2024 12:00:18 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

>> Archie Cobbs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>> 
>>  - Bump @since tag 23 -> 24.
>>  - Merge branch 'master' into JavaShellToolBuilder.windowSize
>>  - Update copyright years.
>>  - Fix typo in exception message.
>>  - Add new method JavaShellToolBuilder.windowSize().
>
> src/jdk.jshell/share/classes/jdk/internal/jshell/tool/JShellToolBuilder.java line 2:
> 
>> 1: /*
>> 2:  * Copyright (c) 2016, 2024 Oracle and/or its affiliates. All rights reserved.
> 
> (Assuming the header is intended:) there must be a comma after the year (i.e. `2024,` instead of `2024`).

Thanks - fixed in 1769df6.

> src/jdk.jshell/share/classes/jdk/jshell/tool/JavaShellToolBuilder.java line 2:
> 
>> 1: /*
>> 2:  * Copyright (c) 2016, 2024 Oracle and/or its affiliates. All rights reserved.
> 
> (Assuming the header is intended:) there must be a comma after the year (i.e. `2024,` instead of `2024`).

Thanks - fixed in 1769df6.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19226#discussion_r1644746518
PR Review Comment: https://git.openjdk.org/jdk/pull/19226#discussion_r1644746629


More information about the kulla-dev mailing list