jextract: hide memory location wrapping constructor, give it proper name

The reason here is that it's too easy to accidentally autocomplete into
new Thing(bla) which can often be incorrect because we probably intended
to call init on the Thing. This popped up with Data a lot, because it is
so easy to wrongly use new Data rather than Data.init which performs the
proper initialization

Author: ktoso

Sources/JExtractSwiftLib/FFM/FFMSwift2JavaGenerator.swift

@@ -216,9 +216,22 @@ extension FFMSwift2JavaGenerator {
 
       printer.print(
         """
-        public \(decl.swiftNominal.name)(MemorySegment segment, AllocatingSwiftArena arena) {
+        private \(decl.swiftNominal.name)(MemorySegment segment, AllocatingSwiftArena arena) {
           super(segment, arena);
         }
+
+        /** 
+         * Assume that the passed {@code MemorySegment} represents a memory address of a {@link \(decl.swiftNominal.name)}.
+         * <p/>
+         * Warnings:
+         * <ul>
+         *   <li>No checks are performed about the compatibility of the pointed at memory and the actual \(decl.swiftNominal.name) types.</li>
+         *   <li>This operation does not copy, or retain, the pointed at pointer, so its lifetime must be ensured manually to be valid when wrapping.</li>
+         * </ul>
+         */
+        public static \(decl.swiftNominal.name) wrapMemoryAddressUnsafe(MemorySegment selfPointer, AllocatingSwiftArena swiftArena) {
+          return new \(decl.swiftNominal.name)(selfPointer, swiftArena);
+        }
         """
       )
 

Sources/JExtractSwiftLib/JNI/JNISwift2JavaGenerator+JavaBindingsPrinting.swift

@@ -117,9 +117,22 @@ extension JNISwift2JavaGenerator {
 
       printer.print(
         """
-        public \(decl.swiftNominal.name)(long selfPointer, SwiftArena swiftArena) {
+        private \(decl.swiftNominal.name)(long selfPointer, SwiftArena swiftArena) {
           super(selfPointer, swiftArena);
         }
+
+        /** 
+         * Assume that the passed {@code long} represents a memory address of a {@link \(decl.swiftNominal.name)}.
+         * <p/>
+         * Warnings:
+         * <ul>
+         *   <li>No checks are performed about the compatibility of the pointed at memory and the actual \(decl.swiftNominal.name) types.</li>
+         *   <li>This operation does not copy, or retain, the pointed at pointer, so its lifetime must be ensured manually to be valid when wrapping.</li>
+         * </ul>
+         */
+        public static \(decl.swiftNominal.name) wrapMemoryAddressUnsafe(long selfPointer, SwiftArena swiftArena) {
+          return new \(decl.swiftNominal.name)(selfPointer, swiftArena);
+        }
         """
       )
 

Tests/JExtractSwiftTests/JNI/JNIClassTests.swift

@@ -66,11 +66,17 @@ struct JNIClassTests {
             System.loadLibrary(LIB_NAME);
             return true;
           }
-
-          public MyClass(long selfPointer, SwiftArena swiftArena) {
+        """,
+        """
+          private MyClass(long selfPointer, SwiftArena swiftArena) {
             super(selfPointer, swiftArena);
           }
         """,
+        """
+        public static MyClass wrapMemoryAddressUnsafe(long selfPointer, SwiftArena swiftArena) {
+          return new MyClass(selfPointer, swiftArena);
+        }
+        """
       ])
     try assertOutput(
       input: source,

Tests/JExtractSwiftTests/JNI/JNIStructTests.swift

@@ -56,10 +56,16 @@ struct JNIStructTests {
             System.loadLibrary(LIB_NAME);
             return true;
           }
-
-          public MyStruct(long selfPointer, SwiftArena swiftArena) {
+        """,
+        """
+          private MyStruct(long selfPointer, SwiftArena swiftArena) {
             super(selfPointer, swiftArena);
           }
+        """,
+        """
+        public static MyStruct wrapMemoryAddressUnsafe(long selfPointer, SwiftArena swiftArena) {
+          return new MyStruct(selfPointer, swiftArena);
+        }
         """
       ])
     try assertOutput(