Apply SuppressWarnings and use private constructors

Author: mikomikotaishi

README.md

@@ -80,18 +80,36 @@ WARNING: Use --enable-native-access=ALL-UNNAMED to avoid a warning for callers i
 WARNING: Restricted methods will be blocked in a future release unless native access is enabled
 ```
 
-To remove these warnings, you need to either allow these methods through the command line arguments:
+To remove these warnings, you need to allow native access for the module that calls the restricted methods.
+
+### Unnamed modules (classpath)
+
+If you are running jdave on the classpath (no `module-info.java`), use:
 
 ```shell
 java --enable-native-access=ALL-UNNAMED ...
 ```
 
-Or enabling them in your JAR-file manifest:
+Or in your JAR-file manifest:
 
-```shell
+```
 Enable-Native-Access: ALL-UNNAMED
 ```
 
+### Named modules (module path)
+
+If your application uses JPMS modules, specify the jdave module name instead:
+
+```shell
+java --enable-native-access=club.minnced.discord.jdave ...
+```
+
+Or in your JAR-file manifest:
+
+```
+Enable-Native-Access: club.minnced.discord.jdave
+```
+
 ## Why Java 25?
 
 This library uses the [Foreign Function & Memory (FFM) API](https://docs.oracle.com/en/java/javase/22/core/foreign-function-and-memory-api.html) which has been stabilized in Java 22.

api/src/main/java/club/minnced/discord/jdave/ffi/LibDave.java

@@ -62,6 +62,7 @@ public static short getMaxSupportedProtocolVersion() {
         }
     }
 
+    @SuppressWarnings("restricted")
     public static void setLogSinkCallback(@NonNull Arena arena, @NonNull LogSinkCallback logSinkCallback) {
         LogSinkCallbackMapper upcallMapper = new LogSinkCallbackMapper(logSinkCallback);
 

api/src/main/java/club/minnced/discord/jdave/ffi/LibDaveDecryptorBinding.java

@@ -9,7 +9,9 @@
 import java.lang.invoke.MethodHandle;
 import org.jspecify.annotations.NonNull;
 
+@SuppressWarnings("restricted")
 public class LibDaveDecryptorBinding {
+    private LibDaveDecryptorBinding() {}
 
     static final MethodHandle daveDecryptorCreate;
     static final MethodHandle daveDecryptorDestroy;

api/src/main/java/club/minnced/discord/jdave/ffi/LibDaveEncryptorBinding.java

@@ -8,7 +8,10 @@
 import java.lang.invoke.MethodHandle;
 import org.jspecify.annotations.NonNull;
 
+@SuppressWarnings("restricted")
 public class LibDaveEncryptorBinding {
+    private LibDaveEncryptorBinding() {}
+
     static final MethodHandle daveEncryptorCreate;
     static final MethodHandle daveEncryptorDestroy;
     static final MethodHandle daveEncryptorSetKeyRatchet;

api/src/main/java/club/minnced/discord/jdave/ffi/LibDaveKeyRatchetBinding.java

@@ -8,6 +8,8 @@
 import org.jspecify.annotations.NonNull;
 
 public class LibDaveKeyRatchetBinding {
+    private LibDaveKeyRatchetBinding() {}
+
     private static final MethodHandle destroyKeyRatchet;
 
     static {

api/src/main/java/club/minnced/discord/jdave/ffi/LibDaveLookup.java

@@ -7,7 +7,10 @@
 import java.lang.foreign.SymbolLookup;
 import java.lang.invoke.MethodHandle;
 
+@SuppressWarnings("restricted")
 public class LibDaveLookup {
+    private LibDaveLookup() {}
+
     static final Linker LINKER = Linker.nativeLinker();
     static final SymbolLookup SYMBOL_LOOKUP;
     public static final MemoryLayout C_SIZE;

api/src/main/java/club/minnced/discord/jdave/ffi/LibDaveSessionBinding.java

@@ -11,7 +11,10 @@
 import java.util.List;
 import org.jspecify.annotations.NonNull;
 
+@SuppressWarnings("restricted")
 public class LibDaveSessionBinding {
+    private LibDaveSessionBinding() {}
+
     static final MethodHandle daveSessionCreate;
     static final MethodHandle daveSessionDestroy;
     static final MethodHandle daveSessionInit;

api/src/main/java/club/minnced/discord/jdave/ffi/NativeUtils.java

@@ -7,7 +7,10 @@
 import org.jspecify.annotations.Nullable;
 
 public class NativeUtils {
+    private NativeUtils() {}
+
     @NonNull
+    @SuppressWarnings("restricted")
     public static String asJavaString(@NonNull MemorySegment nullTerminatedString) {
         return nullTerminatedString.reinterpret(1024 * 64).getString(0);
     }

api/src/main/java/club/minnced/discord/jdave/interop/JDaveSessionFactory.java

@@ -6,6 +6,8 @@
 import org.jspecify.annotations.NonNull;
 
 public class JDaveSessionFactory implements DaveSessionFactory {
+    public JDaveSessionFactory() {}
+
     @NonNull
     @Override
     public DaveSession createDaveSession(@NonNull DaveProtocolCallbacks callbacks, long userId, long channelId) {

api/src/main/java/club/minnced/discord/jdave/utils/DaveLogger.java

@@ -9,6 +9,8 @@
 import org.slf4j.event.Level;
 
 public class DaveLogger {
+    private DaveLogger() {}
+
     @SuppressWarnings("LoggerInitializedWithForeignClass")
     public static final Logger log = LoggerFactory.getLogger(LibDave.class);