format all swift sources, enable format checks

Author: ktoso

.github/workflows/pull_request.yml

@@ -9,8 +9,6 @@ jobs:
     uses: swiftlang/github-workflows/.github/workflows/soundness.yml@main
     with:
       api_breakage_check_enabled: false
-      # FIXME: Something is off with the format task and it gets "stuck", need to investigate
-      format_check_enabled: false
       license_header_check_project_name: Swift.org
 
   test-java:

Samples/JavaKitSampleApp/Package.swift

@@ -55,7 +55,7 @@ let package = Package(
       name: "JavaKitExample",
       type: .dynamic,
       targets: ["JavaKitExample"]
-    ),
+    )
   ],
 
   dependencies: [
@@ -71,12 +71,12 @@ let package = Package(
       ],
       swiftSettings: [
         .swiftLanguageMode(.v5),
-        .unsafeFlags(["-I\(javaIncludePath)", "-I\(javaPlatformIncludePath)"])
+        .unsafeFlags(["-I\(javaIncludePath)", "-I\(javaPlatformIncludePath)"]),
       ],
       plugins: [
         .plugin(name: "JavaCompilerPlugin", package: "swift-java"),
         .plugin(name: "Java2SwiftPlugin", package: "swift-java"),
       ]
-    ),
+    )
   ]
 )

Sources/ExampleSwiftLibrary/MySwiftLibrary.swift

@@ -18,9 +18,9 @@
 // No annotations are necessary on the Swift side to perform the export.
 
 #if os(Linux)
-import Glibc
+  import Glibc
 #else
-import Darwin.C
+  import Darwin.C
 #endif
 
 public func helloWorld() {
@@ -35,7 +35,7 @@ public func globalTakeIntInt(i: Int, j: Int) {
   p("i:\(i), j:\(j)")
 }
 
-public func globalCallMeRunnable(run: () -> ()) {
+public func globalCallMeRunnable(run: () -> Void) {
   run()
 }
 
@@ -85,13 +85,15 @@ public func _getTypeByMangledNameInEnvironment(
   _ name: UnsafePointer<UInt8>,
   _ nameLength: UInt,
   genericEnvironment: UnsafeRawPointer?,
-  genericArguments: UnsafeRawPointer?)
+  genericArguments: UnsafeRawPointer?
+)
   -> Any.Type?
 
-
 // ==== Internal helpers
 
-private func p(_ msg: String, file: String = #fileID, line: UInt = #line, function: String = #function) {
+private func p(
+  _ msg: String, file: String = #fileID, line: UInt = #line, function: String = #function
+) {
   print("[swift][\(file):\(line)](\(function)) \(msg)")
   fflush(stdout)
 }

Sources/JExtractSwift/CodePrinter.swift

@@ -34,7 +34,7 @@ public struct CodePrinter {
   }
   public var indentationText: String = ""
 
-  public static func toString(_ block: (inout CodePrinter) -> ()) -> String {
+  public static func toString(_ block: (inout CodePrinter) -> Void) -> String {
     var printer = CodePrinter()
     block(&printer)
     return printer.finalize()
@@ -66,7 +66,7 @@ public struct CodePrinter {
     function: String = #function,
     file: String = #fileID,
     line: UInt = #line,
-    body: (inout CodePrinter) -> ()
+    body: (inout CodePrinter) -> Void
   ) {
     indent()
     print("\(text) {")
@@ -145,12 +145,16 @@ public struct CodePrinter {
     return contents
   }
 
-  public mutating func indent(file: String = #fileID, line: UInt = #line, function: String = #function) {
+  public mutating func indent(
+    file: String = #fileID, line: UInt = #line, function: String = #function
+  ) {
     indentationDepth += 1
     log.trace("Indent => \(indentationDepth)", file: file, line: line, function: function)
   }
 
-  public mutating func outdent(file: String = #fileID, line: UInt = #line, function: String = #function) {
+  public mutating func outdent(
+    file: String = #fileID, line: UInt = #line, function: String = #function
+  ) {
     indentationDepth -= 1
     log.trace("Outdent => \(indentationDepth)", file: file, line: line, function: function)
     assert(indentationDepth >= 0, "Outdent beyond zero at [\(file):\(line)](\(function))")

Sources/JExtractSwift/Convenience/Collection+Extensions.swift

@@ -12,9 +12,9 @@
 //
 //===----------------------------------------------------------------------===//
 
-public extension Dictionary {
+extension Dictionary {
   /// Same values, corresponding to mapped keys.
-  func mapKeys<Transformed>(
+  public func mapKeys<Transformed>(
     _ transform: (Key) throws -> Transformed
   ) rethrows -> [Transformed: Value] {
     .init(

Sources/JExtractSwift/Convenience/String+Extensions.swift

@@ -22,4 +22,4 @@ extension String {
 
     return "\(f.uppercased())\(String(dropFirst()))"
   }
-}
+}

Sources/JExtractSwift/Convenience/SwiftSyntax+Extensions.swift

@@ -79,11 +79,8 @@ extension DeclSyntaxProtocol {
 extension DeclModifierSyntax {
   var isAccessControl: Bool {
     switch self.name.tokenKind {
-    case .keyword(.private): fallthrough
-    case .keyword(.fileprivate): fallthrough
-    case .keyword(.internal): fallthrough
-    case .keyword(.package): fallthrough
-    case .keyword(.public):
+    case .keyword(.private), .keyword(.fileprivate), .keyword(.internal), .keyword(.package),
+      .keyword(.public):
       return true
     default:
       return false

Sources/JExtractSwift/ImportedDecls+Printing.swift

@@ -14,8 +14,8 @@
 
 import Foundation
 import JavaTypes
-import SwiftSyntax
 import OrderedCollections
+import SwiftSyntax
 
 extension ImportedFunc {
   /// Render a `@{@snippet ... }` comment section that can be put inside a JavaDoc comment
@@ -37,8 +37,8 @@ extension VariableAccessorKind {
 
   public var fieldSuffix: String {
     switch self {
-      case .get: "_GET"
-      case .set: "_SET"
+    case .get: "_GET"
+    case .set: "_SET"
     }
   }
 
@@ -99,4 +99,4 @@ extension Optional where Wrapped == VariableAccessorKind {
   func renderMethodName(_ decl: ImportedFunc) -> String {
     self?.renderMethodName(decl) ?? decl.baseIdentifier
   }
-}
+}

Sources/JExtractSwift/ImportedDecls.swift

@@ -14,8 +14,8 @@
 
 import Foundation
 import JavaTypes
-import SwiftSyntax
 import OrderedCollections
+import SwiftSyntax
 
 /// Any imported (Swift) declaration
 protocol ImportedDecl {
@@ -36,7 +36,10 @@ public struct ImportedNominalType: ImportedDecl {
   public var methods: [ImportedFunc] = []
   public var variables: [ImportedVariable] = []
 
-  public init(swiftTypeName: String, javaType: JavaType, swiftMangledName: String? = nil, kind: NominalTypeKind) {
+  public init(
+    swiftTypeName: String, javaType: JavaType, swiftMangledName: String? = nil,
+    kind: NominalTypeKind
+  ) {
     self.swiftTypeName = swiftTypeName
     self.javaType = javaType
     self.swiftMangledName = swiftMangledName
@@ -56,7 +59,7 @@ public struct ImportedNominalType: ImportedDecl {
   /// The Java class name without the package.
   public var javaClassName: String {
     switch javaType {
-    case .class(package: _, name: let name): name
+    case .class(package: _, let name): name
     default: javaType.description
     }
   }
@@ -250,7 +253,7 @@ public struct ImportedVariable: ImportedDecl, CustomStringConvertible {
   public var identifier: String
 
   /// Which accessors are we able to expose.
-  /// 
+  ///
   /// Usually this will be all the accessors the variable declares,
   /// however if the getter is async or throwing we may not be able to import it
   /// (yet), and therefore would skip it from the supported set.
@@ -285,13 +288,14 @@ public struct ImportedVariable: ImportedDecl, CustomStringConvertible {
 
     switch kind {
     case .set:
-      let newValueParam: FunctionParameterSyntax = "_ newValue: \(self.returnType.cCompatibleSwiftType)"
+      let newValueParam: FunctionParameterSyntax =
+        "_ newValue: \(self.returnType.cCompatibleSwiftType)"
       var funcDecl = ImportedFunc(
         parentName: self.parentName,
         identifier: self.identifier,
         returnType: TranslatedType.void,
         parameters: [.init(param: newValueParam, type: self.returnType)])
-      funcDecl.swiftMangledName = self.swiftMangledName + "s" // form mangled name of the getter by adding the suffix
+      funcDecl.swiftMangledName = self.swiftMangledName + "s"  // form mangled name of the getter by adding the suffix
       return funcDecl
 
     case .get:
@@ -300,21 +304,24 @@ public struct ImportedVariable: ImportedDecl, CustomStringConvertible {
         identifier: self.identifier,
         returnType: self.returnType,
         parameters: [])
-      funcDecl.swiftMangledName = self.swiftMangledName + "g" // form mangled name of the getter by adding the suffix
+      funcDecl.swiftMangledName = self.swiftMangledName + "g"  // form mangled name of the getter by adding the suffix
       return funcDecl
     }
   }
 
-  public func effectiveAccessorParameters(_ kind: VariableAccessorKind, selfVariant: SelfParameterVariant?) -> [ImportedParam] {
+  public func effectiveAccessorParameters(
+    _ kind: VariableAccessorKind, selfVariant: SelfParameterVariant?
+  ) -> [ImportedParam] {
     var params: [ImportedParam] = []
 
     if kind == .set {
-      let newValueParam: FunctionParameterSyntax = "_ newValue: \(raw: self.returnType.swiftTypeName)"
+      let newValueParam: FunctionParameterSyntax =
+        "_ newValue: \(raw: self.returnType.swiftTypeName)"
       params.append(
         ImportedParam(
           param: newValueParam,
           type: self.returnType)
-        )
+      )
     }
 
     if let parentName {

Sources/JExtractSwift/JavaType+Printing.swift

@@ -13,10 +13,10 @@
 //===----------------------------------------------------------------------===//
 
 import Foundation
+import JavaTypes
 import SwiftBasicFormat
 import SwiftParser
 import SwiftSyntax
-import JavaTypes
 
 extension JavaType {
   /// Returns a 'handle' name to pass to the `invoke` call as well as the
@@ -42,6 +42,7 @@ extension JavaType {
         """
     }
 
-    fatalError("Cannot render closure downcall handle for: \(self), in: \(decl), parameter: \(parameter)")
+    fatalError(
+      "Cannot render closure downcall handle for: \(self), in: \(decl), parameter: \(parameter)")
   }
 }