Make it Swift 5.8 compatible by trivial syntactic changes (#94)

Just removes `if` and `switch` *expressions*

Author: kateinoigakukun

Sources/AsyncProcess/ChunkSequence.swift

@@ -49,7 +49,7 @@ public struct ChunkSequence: AsyncSequence & Sendable {
 
     public mutating func next() async throws -> Element? {
       if self.underlyingIterator != nil {
-        try await self.underlyingIterator!.next()
+        return try await self.underlyingIterator!.next()
       } else {
         throw IllegalStreamConsumptionError(
           description: """

Sources/AsyncProcess/FileContentStream.swift

@@ -183,11 +183,11 @@ private final class ReadIntoAsyncChannelHandler: ChannelDuplexHandler {
   private var shouldRead: Bool {
     switch self.state {
     case .idle:
-      true
+      return true
     case .error:
-      false
+      return false
     case .sending:
-      false
+      return false
     }
   }
 

Sources/AsyncProcess/ProcessExecutor+Convenience.swift

@@ -37,18 +37,18 @@ public struct OutputLoggingSettings: Sendable {
   func logMessage(line: String) -> Logger.Message {
     switch self.to {
     case .logMessage:
-      "\(line)"
+      return "\(line)"
     case .metadata(logMessage: let message, key: _):
-      message
+      return message
     }
   }
 
   func metadata(stream: ProcessOutputStream, line: String) -> Logger.Metadata {
     switch self.to {
     case .logMessage:
-      ["stream": "\(stream.description)"]
+      return ["stream": "\(stream.description)"]
     case .metadata(logMessage: _, let key):
-      [key: "\(line)"]
+      return [key: "\(line)"]
     }
   }
 }

Sources/AsyncProcess/ProcessExecutor.swift

@@ -34,9 +34,9 @@ public struct ProcessOutputStream: Sendable & Hashable & CustomStringConvertible
   public var description: String {
     switch self.backing {
     case .standardOutput:
-      "stdout"
+      return "stdout"
     case .standardError:
-      "stderr"
+      return "stderr"
     }
   }
 }

Sources/GeneratorCLI/GeneratorCLI.swift

@@ -198,11 +198,12 @@ extension GeneratorCLI {
       if isInvokedAsDefaultSubcommand() {
         print("deprecated: Please explicity specify the subcommand to run. For example: $ swift-sdk-generator make-linux-sdk")
       }
-      let linuxDistributionDefaultVersion = switch self.linuxDistributionName {
+      let linuxDistributionDefaultVersion: String
+      switch self.linuxDistributionName {
       case .rhel:
-        "ubi9"
+        linuxDistributionDefaultVersion = "ubi9"
       case .ubuntu:
-        "22.04"
+        linuxDistributionDefaultVersion = "22.04"
       }
       let linuxDistributionVersion = self.linuxDistributionVersion ?? linuxDistributionDefaultVersion
       let linuxDistribution = try LinuxDistribution(name: linuxDistributionName, version: linuxDistributionVersion)

Sources/GeneratorEngine/Cache/SQLite.swift

@@ -135,11 +135,11 @@ public final class SQLite {
     var pathString: String {
       switch self {
       case let .path(path):
-        path.string
+        return path.string
       case .memory:
-        ":memory:"
+        return ":memory:"
       case .temporary:
-        ""
+        return ""
       }
     }
   }

Sources/GeneratorEngine/FileSystem/FileSystem.swift

@@ -28,9 +28,9 @@ enum FileSystemError: Error {
 extension Error {
   func attach(path: FilePath) -> any Error {
     if let error = self as? Errno {
-      FileSystemError.systemError(path, error)
+      return FileSystemError.systemError(path, error)
     } else {
-      self
+      return self
     }
   }
 }

Sources/GeneratorEngine/FileSystem/OpenReadableFile.swift

@@ -25,9 +25,9 @@ public struct OpenReadableFile {
   func read() async throws -> ReadableFileStream {
     switch self.fileHandle {
     case let .local(fileDescriptor):
-      ReadableFileStream.local(.init(fileDescriptor: fileDescriptor, readChunkSize: self.readChunkSize))
+      return ReadableFileStream.local(.init(fileDescriptor: fileDescriptor, readChunkSize: self.readChunkSize))
     case let .virtual(array):
-      ReadableFileStream.virtual(.init(bytes: array))
+      return ReadableFileStream.virtual(.init(bytes: array))
     }
   }
 

Sources/GeneratorEngine/FileSystem/ReadableFileStream.swift

@@ -25,19 +25,19 @@ public enum ReadableFileStream: AsyncSequence {
     public func next() async throws -> [UInt8]? {
       switch self {
       case let .local(local):
-        try await local.next()
+        return try await local.next()
       case let .virtual(virtual):
-        try await virtual.next()
+        return try await virtual.next()
       }
     }
   }
 
   public func makeAsyncIterator() -> Iterator {
     switch self {
     case let .local(local):
-      .local(local.makeAsyncIterator())
+      return .local(local.makeAsyncIterator())
     case let .virtual(virtual):
-      .virtual(virtual.makeAsyncIterator())
+      return .virtual(virtual.makeAsyncIterator())
     }
   }
 }

Sources/SwiftSDKGenerator/Artifacts/DownloadableArtifacts.swift

@@ -17,10 +17,10 @@ import struct SystemPackage.FilePath
 private extension Triple {
   var llvmBinaryURLSuffix: String {
     switch (self.os, self.arch) {
-    case (.linux, .aarch64): "aarch64-linux-gnu"
-    case (.linux, .x86_64): "x86_64-linux-gnu-ubuntu-22.04"
-    case (.macosx, .aarch64): "arm64-apple-darwin22.0"
-    case (.macosx, .x86_64): "x86_64-apple-darwin22.0"
+    case (.linux, .aarch64): return "aarch64-linux-gnu"
+    case (.linux, .x86_64): return "x86_64-linux-gnu-ubuntu-22.04"
+    case (.macosx, .aarch64): return "arm64-apple-darwin22.0"
+    case (.macosx, .x86_64): return "x86_64-apple-darwin22.0"
     default: fatalError("\(self) is not supported as LLVM host platform yet")
     }
   }