feature: Code enhancement

Samples/SwiftKitSampleApp/src/test/java/org/swift/swiftkit/SwiftArenaTest.java

@@ -63,7 +63,7 @@ public void arena_markAsDestroyed_preventUseAfterFree_class() {
 
         try {
             unsafelyEscapedOutsideArenaScope.echoIntMethod(1);
-            fail("Expected exception to be thrown! Object was suposed to be dead.");
+            fail("Expected exception to be thrown! Object was supposed to be dead.");
         } catch (IllegalStateException ex) {
             return;
         }
@@ -82,7 +82,7 @@ public void arena_markAsDestroyed_preventUseAfterFree_struct() {
 
         try {
             unsafelyEscapedOutsideArenaScope.echoIntMethod(1);
-            fail("Expected exception to be thrown! Object was suposed to be dead.");
+            fail("Expected exception to be thrown! Object was supposed to be dead.");
         } catch (IllegalStateException ex) {
             return;
         }

Sources/JExtractSwiftLib/CodePrinter.swift

@@ -53,14 +53,14 @@ public struct CodePrinter {
     self.mode = mode
   }
 
-  internal mutating func append(_ text: String) {
+  mutating func append(_ text: String) {
     contents.append(text)
     if self.verbose {
       Swift.print(text, terminator: "")
     }
   }
 
-  internal mutating func append<S>(contentsOf text: S)
+  mutating func append<S>(contentsOf text: S)
   where S: Sequence, S.Element == Character {
     contents.append(contentsOf: text)
     if self.verbose {

Sources/JExtractSwiftLib/Convenience/SwiftSyntax+Extensions.swift

@@ -16,15 +16,15 @@ import SwiftDiagnostics
 import SwiftSyntax
 
 extension WithModifiersSyntax {
-  internal var accessControlModifiers: DeclModifierListSyntax {
+  var accessControlModifiers: DeclModifierListSyntax {
     modifiers.filter { modifier in
       modifier.isAccessControl
     }
   }
 }
 
 extension ImplicitlyUnwrappedOptionalTypeSyntax {
-  internal var asOptionalTypeSyntax: any TypeSyntaxProtocol {
+  var asOptionalTypeSyntax: any TypeSyntaxProtocol {
     OptionalTypeSyntax(
       leadingTrivia: leadingTrivia,
       unexpectedBeforeWrappedType,

Sources/JExtractSwiftLib/Logger.swift

@@ -111,7 +111,7 @@ extension Logger.Level: ExpressibleByArgument {
 }
 
 extension Logger.Level {
-  internal var naturalIntegralValue: Int {
+  var naturalIntegralValue: Int {
     switch self {
     case .trace:
       return 0

Sources/JavaKit/JavaKitVM/LockedState.swift

@@ -13,9 +13,9 @@
 //===----------------------------------------------------------------------===//
 
 #if canImport(os)
-internal import os
+import os
 #if FOUNDATION_FRAMEWORK && canImport(C.os.lock)
-internal import C.os.lock
+import C.os.lock
 #endif
 #elseif canImport(Bionic)
 import Bionic

Sources/JavaKit/JavaValue.swift

@@ -34,7 +34,7 @@ import JavaTypes
 /// The protocol provides operations to bridge values in both directions:
 /// - `getJNIValue(in:)`: produces the JNI value (of type `JNIType`) for the
 ///   `self` Swift value in the given JNI environment.
-/// - `init(fromJNI:in:)`: intializes a Swift value from the JNI value (of
+/// - `init(fromJNI:in:)`: initializes a Swift value from the JNI value (of
 ///   type `JNIType`) in the given JNI environment.
 ///
 /// The protocol also provides hooks to tie into JNI, including operations to

Sources/SwiftKitSwift/SwiftKit.swift

@@ -34,7 +34,7 @@ public func _swiftjava_swift_isUniquelyReferenced(object: UnsafeMutableRawPointe
 
 
  @_alwaysEmitIntoClient @_transparent
- internal func _swiftjava_withHeapObject<R>(
+func _swiftjava_withHeapObject<R>(
    of object: AnyObject,
    _ body: (UnsafeMutableRawPointer) -> R
  ) -> R {

Sources/_Subprocess/LockedState.swift

@@ -13,9 +13,9 @@
 //===----------------------------------------------------------------------===//
 
 #if canImport(os)
-internal import os
+import os
 #if FOUNDATION_FRAMEWORK && canImport(C.os.lock)
-internal import C.os.lock
+import C.os.lock
 #endif
 #elseif canImport(Bionic)
 import Bionic

Sources/_Subprocess/Platforms/Subprocess+Darwin.swift

@@ -32,9 +32,9 @@ import _CShims
 
 // Darwin specific implementation
 extension Subprocess.Configuration {
-    internal typealias StringOrRawBytes = Subprocess.StringOrRawBytes
+    typealias StringOrRawBytes = Subprocess.StringOrRawBytes
 
-    internal func spawn(
+    func spawn(
         withInput input: Subprocess.ExecutionInput,
         output: Subprocess.ExecutionOutput,
         error: Subprocess.ExecutionOutput
@@ -224,7 +224,7 @@ extension Subprocess {
         public var launchRequirementData: Data? = nil
         /// An ordered list of steps in order to tear down the child
         /// process in case the parent task is cancelled before
-        /// the child proces terminates.
+        /// the child process terminates.
         /// Always ends in sending a `.kill` signal at the end.
         public var teardownSequence: [TeardownStep] = []
         /// A closure to configure platform-specific
@@ -288,7 +288,7 @@ extension Subprocess.PlatformOptions: Hashable {
 }
 
 extension Subprocess.PlatformOptions : CustomStringConvertible, CustomDebugStringConvertible {
-    internal func description(withIndent indent: Int) -> String {
+    func description(withIndent indent: Int) -> String {
         let indent = String(repeating: " ", count: indent * 4)
         return """
 PlatformOptions(
@@ -315,7 +315,7 @@ PlatformOptions(
 
 // MARK: - Process Monitoring
 @Sendable
-internal func monitorProcessTermination(
+func monitorProcessTermination(
     forProcessWithIdentifier pid: Subprocess.ProcessIdentifier
 ) async throws -> Subprocess.TerminationStatus {
     return try await withCheckedThrowingContinuation { continuation in

Sources/_Subprocess/Platforms/Subprocess+Linux.swift

@@ -22,9 +22,9 @@ import _CShims
 
 // Linux specific implementations
 extension Subprocess.Configuration {
-    internal typealias StringOrRawBytes = Subprocess.StringOrRawBytes
+    typealias StringOrRawBytes = Subprocess.StringOrRawBytes
 
-    internal func spawn(
+    func spawn(
         withInput input: Subprocess.ExecutionInput,
         output: Subprocess.ExecutionOutput,
         error: Subprocess.ExecutionOutput
@@ -123,7 +123,7 @@ extension Subprocess {
         public var createSession: Bool = false
         /// An ordered list of steps in order to tear down the child
         /// process in case the parent task is cancelled before
-        /// the child proces terminates.
+        /// the child process terminates.
         /// Always ends in sending a `.kill` signal at the end.
         public var teardownSequence: [TeardownStep] = []
         /// A closure to configure platform-specific
@@ -180,7 +180,7 @@ extension Subprocess.PlatformOptions: Hashable {
 }
 
 extension Subprocess.PlatformOptions : CustomStringConvertible, CustomDebugStringConvertible {
-    internal func description(withIndent indent: Int) -> String {
+    func description(withIndent indent: Int) -> String {
         let indent = String(repeating: " ", count: indent * 4)
         return """
 PlatformOptions(
@@ -210,7 +210,7 @@ extension String {
 
 // MARK: - Process Monitoring
 @Sendable
-internal func monitorProcessTermination(
+func monitorProcessTermination(
     forProcessWithIdentifier pid: Subprocess.ProcessIdentifier
 ) async throws -> Subprocess.TerminationStatus {
     return try await withCheckedThrowingContinuation { continuation in
@@ -262,7 +262,7 @@ private let setup: () = {
                 default:
                     fatalError("Unexpected exit status: \(siginfo.si_code)")
                 }
-                if let status = status {
+                if let status {
                     let pid = siginfo._sifields._sigchld.si_pid
                     if let existing = continuations.removeValue(forKey: pid),
                        case .continuation(let c) = existing {

Sources/_Subprocess/Platforms/Subprocess+Unix.swift

@@ -106,7 +106,7 @@ extension Subprocess {
         }
     }
 
-    internal func tryTerminate() -> Error? {
+    func tryTerminate() -> Error? {
         do {
             try self.send(.kill, toProcessGroup: true)
         } catch {
@@ -124,9 +124,9 @@ extension Subprocess {
 
 // MARK: - Environment Resolution
 extension Subprocess.Environment {
-    internal static let pathEnvironmentVariableName = "PATH"
+    static let pathEnvironmentVariableName = "PATH"
 
-    internal func pathValue() -> String? {
+    func pathValue() -> String? {
         switch self.config {
         case .inherit(let overrides):
             // If PATH value exists in overrides, use it
@@ -145,7 +145,7 @@ extension Subprocess.Environment {
 
     // This method follows the standard "create" rule: `env` needs to be
     // manually deallocated
-    internal func createEnv() -> [UnsafeMutablePointer<CChar>?] {
+    func createEnv() -> [UnsafeMutablePointer<CChar>?] {
         func createFullCString(
             fromKey keyContainer: Subprocess.StringOrRawBytes,
             value valueContainer: Subprocess.StringOrRawBytes
@@ -212,7 +212,7 @@ extension Subprocess.Environment {
 extension Subprocess.Arguments {
     // This method follows the standard "create" rule: `args` needs to be
     // manually deallocated
-    internal func createArgs(withExecutablePath executablePath: String) -> [UnsafeMutablePointer<CChar>?] {
+    func createArgs(withExecutablePath executablePath: String) -> [UnsafeMutablePointer<CChar>?] {
         var argv: [UnsafeMutablePointer<CChar>?] = self.storage.map { $0.createRawBytes() }
         // argv[0] = executable path
         if let override = self.executablePathOverride {
@@ -246,7 +246,7 @@ extension Subprocess.ProcessIdentifier : CustomStringConvertible, CustomDebugStr
 
 // MARK: -  Executable Searching
 extension Subprocess.Executable {
-    internal static var defaultSearchPaths: Set<String> {
+    static var defaultSearchPaths: Set<String> {
         return Set([
             "/usr/bin",
             "/bin",
@@ -256,7 +256,7 @@ extension Subprocess.Executable {
         ])
     }
 
-    internal func resolveExecutablePath(withPathValue pathValue: String?) -> String? {
+    func resolveExecutablePath(withPathValue pathValue: String?) -> String? {
         switch self.storage {
         case .executable(let executableName):
             // If the executableName in is already a full path, return it directly
@@ -289,7 +289,7 @@ extension Subprocess.Executable {
 
 // MARK: - Configuration
 extension Subprocess.Configuration {
-    internal func preSpawn() throws -> (
+    func preSpawn() throws -> (
         executablePath: String,
         env: [UnsafeMutablePointer<CChar>?],
         argv: [UnsafeMutablePointer<CChar>?],
@@ -343,7 +343,7 @@ extension Subprocess.Configuration {
         )
     }
 
-    internal static func pathAccessible(_ path: String, mode: Int32) -> Bool {
+    static func pathAccessible(_ path: String, mode: Int32) -> Bool {
         return path.withCString {
             return access($0, mode) == 0
         }
@@ -352,18 +352,18 @@ extension Subprocess.Configuration {
 
 // MARK: - FileDescriptor extensions
 extension FileDescriptor {
-    internal static func openDevNull(
+    static func openDevNull(
         withAcessMode mode: FileDescriptor.AccessMode
     ) throws -> FileDescriptor {
         let devnull: FileDescriptor = try .open("/dev/null", mode)
         return devnull
     }
 
-    internal var platformDescriptor: Subprocess.PlatformFileDescriptor {
+    var platformDescriptor: Subprocess.PlatformFileDescriptor {
         return self
     }
 
-    internal func readChunk(upToLength maxLength: Int) async throws -> Data? {
+    func readChunk(upToLength maxLength: Int) async throws -> Data? {
         return try await withCheckedThrowingContinuation { continuation in
             DispatchIO.read(
                 fromFileDescriptor: self.rawValue,
@@ -383,7 +383,7 @@ extension FileDescriptor {
         }
     }
 
-    internal func readUntilEOF(upToLength maxLength: Int) async throws -> Data {
+    func readUntilEOF(upToLength maxLength: Int) async throws -> Data {
         return try await withCheckedThrowingContinuation { continuation in
             let dispatchIO = DispatchIO(
                 type: .stream,
@@ -404,7 +404,7 @@ extension FileDescriptor {
                     continuation.resume(throwing: POSIXError(.init(rawValue: error) ?? .ENODEV))
                     return
                 }
-                if let data = data {
+                if let data {
                     buffer += Data(data)
                 }
                 if done {
@@ -415,7 +415,7 @@ extension FileDescriptor {
         }
     }
 
-    internal func write<S: Sequence>(_ data: S) async throws where S.Element == UInt8 {
+    func write<S: Sequence>(_ data: S) async throws where S.Element == UInt8 {
         try await withCheckedThrowingContinuation { (continuation: CheckedContinuation<Void, Error>) -> Void in
             let dispatchData: DispatchData = Array(data).withUnsafeBytes {
                 return DispatchData(bytes: $0)
@@ -439,13 +439,13 @@ extension FileDescriptor {
 }
 
 extension Subprocess {
-    internal typealias PlatformFileDescriptor = FileDescriptor
+    typealias PlatformFileDescriptor = FileDescriptor
 }
 
 // MARK: - Read Buffer Size
 extension Subprocess {
     @inline(__always)
-    internal static var readBufferSize: Int {
+    static var readBufferSize: Int {
 #if canImport(Darwin)
         return 16384
 #else