On Mon, 2 Dec 2024 21:18:10 GMT, Weijun Wang <wei...@openjdk.org> wrote:
>> Francisco Ferrari Bihurriet has updated the pull request with a new target
>> base due to a merge or a rebase. The pull request now contains three commits:
>>
>> - Merge openjdk/master into JDK-8345221
>> - Method rename.
>>
>> Co-authored-by: Martin Balao Alonso <mba...@redhat.com>
>> Co-authored-by: Francisco Ferrari Bihurriet <fferr...@redhat.com>
>> - 8345221: Replace legacy with new Provider APIs in SunNativeGSS
>>
>> Co-authored-by: Francisco Ferrari Bihurriet <fferr...@redhat.com>
>> Co-authored-by: Martin Balao <mba...@redhat.com>
>
> src/java.security.jgss/share/classes/sun/security/jgss/wrapper/SunNativeProvider.java
> line 69:
>
>> 67:
>> 68: @SuppressWarnings("restricted")
>> 69: private static Oid[] constructMechMap() {
>
> Maybe change the method name? The return value is not a map now.
Sounds good. Changed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22461#discussion_r1866701239
