Merge pull request #37 from DougGregor/javakit-static-methods

Introduce the @JavaStaticMethod and use it for calling static Java methods

Author: DougGregor

Sources/Java2Swift/JavaTranslator.swift

@@ -416,8 +416,9 @@ extension JavaTranslator {
 
     let throwsStr = javaMethod.throwsCheckedException ? "throws" : ""
 
+    let methodAttribute: AttributeSyntax = javaMethod.isStatic ? "@JavaStaticMethod" : "@JavaMethod";
     return """
-      @JavaMethod
+      \(methodAttribute)
       public func \(raw: javaMethod.getName())\(raw: genericParameterClause)(\(raw: parametersStr))\(raw: throwsStr)\(raw: resultTypeStr)\(raw: whereClause)
       """
   }

Sources/JavaKit/JavaObject+MethodCalls.swift

@@ -301,9 +301,10 @@ extension AnyJavaObject {
 extension JavaClass {
   /// Call a Java static method with the given name and arguments, which must be
   /// of the correct type, that produces the given result type.
-  public func dynamicJavaMethodCall<each Param: JavaValue, Result: JavaValue>(
+  public func dynamicJavaStaticMethodCall<each Param: JavaValue, Result: JavaValue>(
     methodName: String,
-    args: repeat each Param
+    arguments: repeat each Param,
+    resultType: Result.Type
   ) throws -> Result {
     let thisClass = javaThis
     let environment = javaEnvironment
@@ -325,7 +326,7 @@ extension JavaClass {
 
     // Retrieve the method that performs this call, then
     let jniMethod = Result.jniStaticMethodCall(in: environment)
-    let jniArgs = getJValues(repeat each args, in: environment)
+    let jniArgs = getJValues(repeat each arguments, in: environment)
     let jniResult = try environment.translatingJNIExceptions {
       jniMethod(environment, thisClass, methodID, jniArgs)
     }
@@ -335,9 +336,9 @@ extension JavaClass {
 
   /// Call a Java static method with the given name and arguments, which must be
   /// of the correct type, that produces the given result type.
-  public func dynamicJavaMethodCall<each Param: JavaValue>(
+  public func dynamicJavaStaticMethodCall<each Param: JavaValue>(
     methodName: String,
-    args: repeat each Param
+    arguments: repeat each Param
   ) throws {
     let thisClass = javaThis
     let environment = javaEnvironment
@@ -359,7 +360,7 @@ extension JavaClass {
 
     // Retrieve the method that performs this call, then
     let jniMethod = environment.interface.CallStaticVoidMethodA
-    let jniArgs = getJValues(repeat each args, in: environment)
+    let jniArgs = getJValues(repeat each arguments, in: environment)
     try environment.translatingJNIExceptions {
       jniMethod!(environment, thisClass, methodID, jniArgs)
     }

Sources/JavaKit/Macros.swift

@@ -95,7 +95,7 @@ public macro JavaField(_ javaFieldName: String? = nil) = #externalMacro(module:
 
 /// Attached macro that turns a Swift method into one that wraps a Java method on the underlying Java object.
 ///
-/// The macro must be used within either a AnyJavaObject-conforming type or a specific JavaClass instance.
+/// The macro must be used in an AnyJavaObject-conforming type.
 ///
 /// ```swift
 /// @JavaMethod
@@ -115,6 +115,18 @@ public macro JavaField(_ javaFieldName: String? = nil) = #externalMacro(module:
 @attached(body)
 public macro JavaMethod() = #externalMacro(module: "JavaKitMacros", type: "JavaMethodMacro")
 
+/// Attached macro that turns a Swift method on JavaClass into one that wraps
+/// a Java static method on the underlying Java class object.
+///
+/// The macro must be used within a specific JavaClass instance.
+///
+/// ```swift
+/// @JavaMethod
+/// func sayHelloBack(_ i: Int32) -> Double
+/// ```
+@attached(body)
+public macro JavaStaticMethod() = #externalMacro(module: "JavaKitMacros", type: "JavaMethodMacro")
+
 /// Macro that exposes the given Swift method as a native method in Java.
 ///
 /// The macro must be used within a struct type marked with `@JavaClass`, and there

Sources/JavaKitMacros/JavaMethodMacro.swift

@@ -34,6 +34,7 @@ extension JavaMethodMacro: BodyMacro {
       fatalError("not a function")
     }
 
+    let isStatic = node.attributeName.trimmedDescription == "JavaStaticMethod"
     let funcName = funcDecl.name.text
     let params = funcDecl.signature.parameterClause.parameters
     let resultType: String =
@@ -54,7 +55,7 @@ extension JavaMethodMacro: BodyMacro {
       ? "try" : "try!"
 
     return [
-      "return \(raw: tryKeyword) dynamicJavaMethodCall(methodName: \(literal: funcName)\(raw: parametersAsArgs)\(raw: resultType))"
+      "return \(raw: tryKeyword) dynamicJava\(raw: isStatic ? "Static" : "")MethodCall(methodName: \(literal: funcName)\(raw: parametersAsArgs)\(raw: resultType))"
     ]
   }
 

Sources/JavaKitNetwork/generated/URI.swift

@@ -125,6 +125,6 @@ public struct URI {
   public func wait() throws
 }
 extension JavaClass<URI> {
-  @JavaMethod
+  @JavaStaticMethod
   public func create(_ arg0: String) -> URI?
 }

Sources/JavaKitNetwork/generated/URLClassLoader.swift

@@ -74,12 +74,12 @@ public struct URLClassLoader {
   public func wait() throws
 }
 extension JavaClass<URLClassLoader> {
-  @JavaMethod
+  @JavaStaticMethod
   public func newInstance(_ arg0: [URL?]) -> URLClassLoader?
 
-  @JavaMethod
+  @JavaStaticMethod
   public func getSystemResource(_ arg0: String) -> URL?
 
-  @JavaMethod
+  @JavaStaticMethod
   public func getSystemResources(_ arg0: String) throws -> Enumeration<URL>?
 }

Sources/JavaKitNetwork/generated/URLConnection.swift

@@ -146,7 +146,7 @@ extension JavaClass<URLConnection> {
   @JavaMethod
   public func setDefaultAllowUserInteraction(_ arg0: Bool)
 
-  @JavaMethod
+  @JavaStaticMethod
   public func getDefaultAllowUserInteraction() -> Bool
 
   @JavaMethod

Sources/JavaKitReflection/generated/AccessibleObject.swift

@@ -65,6 +65,6 @@ public struct AccessibleObject {
   public func wait() throws
 }
 extension JavaClass<AccessibleObject> {
-  @JavaMethod
+  @JavaStaticMethod
   public func setAccessible(_ arg0: [AccessibleObject?], _ arg1: Bool)
 }

Sources/JavaKitReflection/generated/Constructor.swift

@@ -122,6 +122,6 @@ public struct Constructor<T: AnyJavaObject> {
   public func wait() throws
 }
 extension JavaClass {
-  @JavaMethod
+  @JavaStaticMethod
   public func setAccessible<T: AnyJavaObject>(_ arg0: [AccessibleObject?], _ arg1: Bool) where ObjectType == Constructor<T>
 }

Sources/JavaKitReflection/generated/Executable.swift

@@ -119,6 +119,6 @@ public struct Executable {
   public func wait() throws
 }
 extension JavaClass<Executable> {
-  @JavaMethod
+  @JavaStaticMethod
   public func setAccessible(_ arg0: [AccessibleObject?], _ arg1: Bool)
 }