RFR: 8144100: Incorrect case-sensitive equality in com.sun.net.httpserver.BasicAuthenticator [v2]
Nizar Benalla
duke at openjdk.org
Fri May 10 13:29:24 UTC 2024
On Fri, 10 May 2024 09:46:44 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Nizar Benalla has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Declare `ServerAuthenticator.invoked` as volatile
>
> test/jdk/com/sun/net/httpserver/BasicAuthToken.java line 24:
>
>> 22: */
>> 23:
>> 24: /**
>
> Suggestion:
>
> /*
>
> It was recently suggested that test comments are not API documentation comments, and that we should avoid `/**` in that case. Maybe we will do a global pass on the test base at some point (or not) but in the mean time let's avoid propagating this pattern in new tests.
Maybe we can use a script to fix them all in one a large change, similar to #18268
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19133#discussion_r1596751704
More information about the net-dev
mailing list