On 05/05/2025 21:05, Steffen Nießing wrote:
Hello,
recently I've discovered that the JavaDoc of the instance methods of
java.net.http.HttpHeaders lacks the throws declarations for
NullPointerExceptions. The static creator method `of` contains such a
declaration. So the snippet `HttpHeaders.of(null, null)` is documented
to throw an NPE, while the snippet `HttpHeaders.of(Map.of(), (k, v) ->
true).firstValue(null)` currently isn't.
I'd propose to add the missing JavaDoc throws declarations to the
#allValues(String), #firstValueAsLong(String) and #firstValue(String)
methods as they all can throw an NPE through `requireNonNull(name);`
in the allValues method.
If there's interest in this change, I'd appreciate if anyone would
like to sponsor me on this one.
Can you bring this to net-dev to discuss? net-dev is where the
networking API and the HTTP client are maintained.
-Alan
