RFR: 8133686: HttpURLConnection.getHeaderFields and URLConnection.getRequestProperties methods return field values in reverse order [v2]
Evan Whelan
ewhelan at openjdk.java.net
Fri Jan 29 11:03:09 UTC 2021
On Thu, 28 Jan 2021 17:02:31 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Evan Whelan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> URLConnection doc fixes
>
> src/java.base/share/classes/java/net/URLConnection.java line 596:
>
>> 594: * corresponding values, they must be returned in the order they were added,
>> 595: * preserving the insertion-order.
>> 596: *
>
> Here the `@implSpec` is right because it describes how subclasses should implement the method.
> Maybe there should be an additional `@implNote` as well to describe what the default implementation does.
>
> * @implNote The default implementation of this method returns an empty map always.
>
> The `@implNote` is useful for subclasses to decide whether they should override the default implementation.
Added, thanks Daniel
> src/java.base/share/classes/java/net/URLConnection.java line 1201:
>
>> 1199: * @implSpec The default implementation of this method should preserve insertion order when
>> 1200: * multiple values are added for a given key. They must be
>> 1201: * returned in the order they were added.
>
> If we are speaking about what the default implementation actually does, then the conditional tense should be removed. And this should probably be an @implNote.
>
> * @implNote The default implementation of this method preserves the insertion order when
> * multiple values are added for a given key. The values are returned in the order they
> * were added.
This has been changed, thanks
-------------
PR: https://git.openjdk.java.net/jdk/pull/2294
More information about the net-dev
mailing list