Re: RFR: 8257707: Fix incorrect format string in Http1HeaderParser

2020-12-03 Thread Aleksey Shipilev
On Sat, 28 Nov 2020 19:42:01 GMT, Turbanov Andrey wrote: > Incorrect format string was found by IntelliJ IDEA inspection `Java | > Probable bugs | Malformed format string` Submitted: https://bugs.openjdk.java.net/browse/JDK-8257707 -- rename this PR to "8257707: Fix incorrect format string in

RFR: 8257707: Fix incorrect format string in Http1HeaderParser

2020-12-03 Thread Turbanov Andrey
Incorrect format string was found by IntelliJ IDEA inspection `Java | Probable bugs | Malformed format string` - Commit messages: - [PATCH] Fix incorrect format string in Http1HeaderParser Changes: https://git.openjdk.java.net/jdk/pull/1495/files Webrev: https://webrevs.openjdk.ja

Re: RFR: 8257707: Fix incorrect format string in Http1HeaderParser

2020-12-03 Thread Daniel Fuchs
On Thu, 3 Dec 2020 17:42:00 GMT, Aleksey Shipilev wrote: >> Incorrect format string was found by IntelliJ IDEA inspection `Java | >> Probable bugs | Malformed format string` > > Submitted: https://bugs.openjdk.java.net/browse/JDK-8257707 -- rename this PR > to "8257707: Fix incorrect format str

Re: RFR: 8257707: Fix incorrect format string in Http1HeaderParser

2020-12-03 Thread Andrey Turbanov
On Thu, 3 Dec 2020 19:01:12 GMT, Daniel Fuchs wrote: >> Submitted: https://bugs.openjdk.java.net/browse/JDK-8257707 -- rename this >> PR to "8257707: Fix incorrect format string in Http1HeaderParser" to get it >> hooked here. Also, merge from master to get Windows builds fixed and >> re-tested

RFR: 8257736: InputStream from BodyPublishers.ofInputStream() leaks when IOE happens

2020-12-03 Thread Yasumasa Suenaga
`InputStream` from `BodyPublishers.ofInputStream()` is usually closed when the stream reaches EOF. However IOE handler returns without closing. I confirmed this problem in `BodyPublishers.ofInputStream()`, but I think `BodyPublishers.ofFile()`has same problem because it also use `StreamIterator

Re: RFR: 8257736: InputStream from BodyPublishers.ofInputStream() leaks when IOE happens

2020-12-03 Thread Yasumasa Suenaga
On Fri, 4 Dec 2020 00:47:51 GMT, Yasumasa Suenaga wrote: > `InputStream` from `BodyPublishers.ofInputStream()` is usually closed when > the stream reaches EOF. However IOE handler returns without closing. > > I confirmed this problem in `BodyPublishers.ofInputStream()`, but I think > `BodyPubl

Re: RFR: 8257707: Fix incorrect format string in Http1HeaderParser

2020-12-03 Thread Aleksey Shipilev
On Thu, 3 Dec 2020 20:05:36 GMT, Andrey Turbanov wrote: > === Output from failing command(s) repeated here === > * For target > hotspot_variant-server_libjvm_objs_bytecodeInterpreterWithChecks.obj: > bytecodeInterpreterWithChecks.cpp > ./build/windows-x86_64-server-release/hotspot/variant-serve

Re: RFR: 8257707: Fix incorrect format string in Http1HeaderParser

2020-12-03 Thread Aleksey Shipilev
On Fri, 4 Dec 2020 06:33:35 GMT, Aleksey Shipilev wrote: >> make test-tier2 fails with error on my machine: :( >> user@WORK-PC /cygdrive/f/Projects/official_openjdk >> $ make test-tier2 >> Building target 'test-tier2' in configuration 'windows-x86_64-server-release' >> Compiling 127 files for jav