On Fri, 9 Oct 2020 09:17:48 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
>> src/java.base/share/classes/sun/net/www/MeteredStream.java line 123: >> >>> 121: lock(); >>> 122: try { >>> 123: if (closed) return -1; >> >> This double check of `closed` is kind of irritating. Is it really need, or >> maybe we just drop it and lock >> unconditionally? > > We could. That's a good suggestion. I agree on that point. ------------- PR: https://git.openjdk.java.net/jdk/pull/558