Hi,
Could someone please review my changes (and CSR) to drop support for pre JDK
1.4 `DatagramSocketImpl` implementations?
These changes propose to drop support for `DatagramSocketImpls` that were
compiled with JDK 1.3 or older, which do not have support for connected
sockets, for peeking at r
On Mon, 11 Oct 2021 09:42:12 GMT, Patrick Concannon
wrote:
> Hi,
>
> Could someone please review my changes (and CSR) to drop support for pre JDK
> 1.4 `DatagramSocketImpl` implementations?
>
> These changes propose to drop support for `DatagramSocketImpls` that were
> compiled with JDK 1.3
Cleanup unnecessary String.valueOf calls (and similar) when conversion will
happen implicitly anyway
-
Commit messages:
- 8275079: Remove unnecessary conversion to String in java.net.http
- [PATCH] Remove unnecessary conversion to String
Changes: https://git.openjdk.java.net/jdk/p
On Sat, 2 Oct 2021 20:05:37 GMT, Andrey Turbanov wrote:
> Cleanup unnecessary String.valueOf calls (and similar) when conversion will
> happen implicitly anyway
Some small comments.
src/java.net.http/share/classes/jdk/internal/net/http/Http1AsyncReceiver.java
line 738:
> 736: if
On Mon, 4 Oct 2021 13:39:00 GMT, Weijun Wang wrote:
>> Cleanup unnecessary String.valueOf calls (and similar) when conversion will
>> happen implicitly anyway
>
> src/java.net.http/share/classes/jdk/internal/net/http/Http1AsyncReceiver.java
> line 738:
>
>> 736: if (flowTag != null
On Thu, 7 Oct 2021 16:48:06 GMT, Naoto Sato wrote:
>> StringBuffer is a legacy synchronized class. There are more modern
>> alternatives which perform better:
>> 1. Plain String concatenation should be preferred
>> 2. StringBuilder is a direct replacement to StringBuffer which generally
>> have