On Fri, 10 Dec 2021 18:18:02 GMT, Andrey Turbanov wrote:
>> This is legacy code with probably poor testing coverage. Usually we don't
>> print anything on the console directly and unconditionally - except in some
>> well identified cases. Ignoring exceptions thrown by `close` is a common
>> id
On Thu, 9 Dec 2021 01:25:20 GMT, Bradford Wetmore wrote:
>> That's one of the places where I personally would always use the var keyword
>> too: it makes the line shorter and the type is already clearly visible in
>> the RHS of the assignment, so repeating it on the left hand side does not
>>
On Tue, 7 Dec 2021 14:48:17 GMT, Daniel Fuchs wrote:
>Remind me to sponsor this after the fork.
fork is done :)
-
PR: https://git.openjdk.java.net/jdk/pull/5818
On Tue, 7 Dec 2021 07:59:54 GMT, Alan Bateman wrote:
>> Andrey Turbanov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8274809: Update java.base classes to use try-with-resources
>> fix review comments
>
> src/java.base/share/classes/
On Tue, 7 Dec 2021 14:24:41 GMT, Andrey Turbanov wrote:
>> src/java.base/share/classes/sun/net/www/MimeTable.java line 385:
>>
>>> 383:
>>> 384: protected boolean saveAsProperties(File file) {
>>> 385: try (FileOutputStream os = new FileOutputStream(file)) {
>>
>> This is not stric
On Wed, 6 Oct 2021 20:37:29 GMT, Andrey Turbanov wrote:
>> 8274809: Update java.base classes to use try-with-resources
>
> Andrey Turbanov has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 8274809: Update java.base classes to use try-with-res
On Tue, 7 Dec 2021 11:41:12 GMT, Daniel Fuchs wrote:
>> Andrey Turbanov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8274809: Update java.base classes to use try-with-resources
>> fix review comments
>
> src/java.base/share/classes/
On Wed, 6 Oct 2021 20:37:29 GMT, Andrey Turbanov wrote:
>> 8274809: Update java.base classes to use try-with-resources
>
> Andrey Turbanov has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 8274809: Update java.base classes to use try-with-res
On Wed, 6 Oct 2021 20:37:29 GMT, Andrey Turbanov wrote:
>> 8274809: Update java.base classes to use try-with-resources
>
> Andrey Turbanov has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 8274809: Update java.base classes to use try-with-res
On Wed, 6 Oct 2021 16:00:26 GMT, Bradford Wetmore wrote:
>> Andrey Turbanov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8274809: Update java.base classes to use try-with-resources
>> fix review comments
>
> src/java.base/share/clas
> 8274809: Update java.base classes to use try-with-resources
Andrey Turbanov has updated the pull request incrementally with one additional
commit since the last revision:
8274809: Update java.base classes to use try-with-resources
fix review comments
-
Changes:
- all: https
11 matches
Mail list logo
