James,
On 11/12/17 00:47, James Roper wrote:
> Hi Chris,
>
> This looks like a straight forward way to solve the problem with minimal
> disruption from the existing API. Can I make a few suggestions though?
Of course, your input here is much appreciated.
> We could add a contentLength parameter
There's now a Martin-style benchmark at
http://cr.openjdk.java.net/~martin/webrevs/jdk/URLMicroBenchmark/
that suggests the code is ~ 25% faster with default JVM flags (C2) but ~
25% slower with C1, as you might expect with multiple String
concatenation. I think we want to optimize for the default