On Fri, 26 Apr 2024 15:37:13 GMT, robert engels <d...@openjdk.org> wrote:
>> improve the HttpExchange api with documented constants and convenience >> methods to avoid common bugs > > robert engels has updated the pull request incrementally with one additional > commit since the last revision: > > Update > src/jdk.httpserver/share/classes/com/sun/net/httpserver/HttpExchange.java > > Co-authored-by: Michael McMahon > <70538289+michael-mc-ma...@users.noreply.github.com> src/jdk.httpserver/share/classes/com/sun/net/httpserver/HttpExchange.java line 237: > 235: */ > 236: public final OutputStream sendResponseHeadersChunked(int code) > throws IOException { > 237: sendResponseHeaders(code,CHUNKED_CONTENT); I don't like the fact that this methor returns an `OutputStream` when other `sendResponseXxx` methods return `void`. IMO it should either be renamed or the return type changed to `void`, or maybe it should simply be dropped. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/18955#discussion_r1585074446