Fix visibility of abstract methods (java-native-access#697)

Motivation: These methods should be marked as private-package as its arguments are also package-private Modifications: Fix visibility Result: Cleanup
mstyura · Mar 14, 2024 · acfbcfe · acfbcfe
1 parent 7ad7a82
commit acfbcfe
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/codec-classes-quic/src/main/java/io/netty/incubator/codec/quic/QuicClientCodecBuilder.java b/codec-classes-quic/src/main/java/io/netty/incubator/codec/quic/QuicClientCodecBuilder.java
@@ -43,7 +43,7 @@ public QuicClientCodecBuilder clone() {
     }
 
     @Override
-    protected ChannelHandler build(QuicheConfig config,
+    ChannelHandler build(QuicheConfig config,
                                    Function<QuicChannel, ? extends QuicSslEngine> sslEngineProvider,
                                    Executor sslTaskExecutor,
                                    int localConnIdLength, FlushStrategy flushStrategy) {

diff --git a/codec-classes-quic/src/main/java/io/netty/incubator/codec/quic/QuicCodecBuilder.java b/codec-classes-quic/src/main/java/io/netty/incubator/codec/quic/QuicCodecBuilder.java
@@ -501,7 +501,7 @@ public final ChannelHandler build() {
      * @param flushStrategy         the {@link FlushStrategy}  that should be used.
      * @return                      the {@link ChannelHandler} which acts as codec.
      */
-    protected abstract ChannelHandler build(QuicheConfig config,
+    abstract ChannelHandler build(QuicheConfig config,
                                             Function<QuicChannel, ? extends QuicSslEngine> sslContextProvider,
                                             Executor sslTaskExecutor,
                                             int localConnIdLength, FlushStrategy flushStrategy);

diff --git a/codec-classes-quic/src/main/java/io/netty/incubator/codec/quic/QuicServerCodecBuilder.java b/codec-classes-quic/src/main/java/io/netty/incubator/codec/quic/QuicServerCodecBuilder.java
@@ -196,7 +196,7 @@ protected void validate() {
     }
 
     @Override
-    protected ChannelHandler build(QuicheConfig config,
+    ChannelHandler build(QuicheConfig config,
                                    Function<QuicChannel, ? extends QuicSslEngine> sslEngineProvider,
                                    Executor sslTaskExecutor,
                                    int localConnIdLength, FlushStrategy flushStrategy) {