fmt-lint で警告された一部を修正

Author: zztkm

Sora/ConnectionTimer.swift

@@ -6,20 +6,20 @@ enum ConnectionMonitor {
 
   var state: ConnectionState {
     switch self {
-    case let .signalingChannel(chan):
+    case .signalingChannel(let chan):
       return chan.state
-    case let .peerChannel(chan):
+    case .peerChannel(let chan):
       return ConnectionState(chan.state)
     }
   }
 
   func disconnect() {
     let error = SoraError.connectionTimeout
     switch self {
-    case let .signalingChannel(chan):
+    case .signalingChannel(let chan):
       // タイムアウトはシグナリングのエラーと考える
       chan.disconnect(error: error, reason: .signalingFailure)
-    case let .peerChannel(chan):
+    case .peerChannel(let chan):
       // タイムアウトはシグナリングのエラーと考える
       chan.disconnect(error: error, reason: .signalingFailure)
     }

Sora/DataChannel.swift

@@ -200,9 +200,9 @@ class BasicDataChannelDelegate: NSObject, RTCDataChannelDelegate {
 
       case "signaling", "push", "notify":
         switch Signaling.decode(data) {
-        case let .success(signaling):
+        case .success(let signaling):
           peerChannel.handleSignalingOverDataChannel(signaling)
-        case let .failure(error):
+        case .failure(let error):
           Logger.error(
             type: .dataChannel,
             message: "decode failed (\(error.localizedDescription)) => ")

Sora/Logger.swift

@@ -47,7 +47,7 @@ extension LogType: CustomStringConvertible {
       return "VideoRenderer"
     case .videoView:
       return "VideoView"
-    case let .user(name):
+    case .user(let name):
       return name
     case .configurationViewController:
       return "ConfigurationViewController"

Sora/MediaStream.swift

@@ -47,26 +47,20 @@ public protocol MediaStream: AnyObject {
 
   // MARK: - 映像と音声の可否
 
-  /**
-     映像の可否。
-     ``false`` をセットすると、サーバーへの映像の送受信を停止します。
-     ``true`` をセットすると送受信を再開します。
-     */
+  /// 映像の可否。
+  /// ``false`` をセットすると、サーバーへの映像の送受信を停止します。
+  /// ``true`` をセットすると送受信を再開します。
   var videoEnabled: Bool { get set }
 
-  /**
-     音声の可否。
-     ``false`` をセットすると、サーバーへの音声の送受信を停止します。
-     ``true`` をセットすると送受信を再開します。
-
-     サーバーへの送受信を停止しても、マイクはミュートされませんので注意してください。
-     */
+  /// 音声の可否。
+  /// ``false`` をセットすると、サーバーへの音声の送受信を停止します。
+  /// ``true`` をセットすると送受信を再開します。
+  ///
+  /// サーバーへの送受信を停止しても、マイクはミュートされませんので注意してください。
   var audioEnabled: Bool { get set }
 
-  /**
-     受信した音声のボリューム。 0 から 10 (含む) までの値をセットします。
-     このプロパティはロールがサブスクライバーの場合のみ有効です。
-     */
+  /// 受信した音声のボリューム。 0 から 10 (含む) までの値をセットします。
+  /// このプロパティはロールがサブスクライバーの場合のみ有効です。
   var remoteAudioVolume: Double? { get set }
 
   // MARK: 映像フレームの送信
@@ -77,22 +71,18 @@ public protocol MediaStream: AnyObject {
   /// 映像レンダラー。
   var videoRenderer: VideoRenderer? { get set }
 
-  /**
-     映像フレームをサーバーに送信します。
-     送信される映像フレームは映像フィルターを通して加工されます。
-     映像レンダラーがセットされていれば、加工後の映像フレームが
-     映像レンダラーによって描画されます。
-
-     - parameter videoFrame: 描画する映像フレーム。
-                             `nil` を指定すると空の映像フレームを送信します。
-     */
+  /// 映像フレームをサーバーに送信します。
+  /// 送信される映像フレームは映像フィルターを通して加工されます。
+  /// 映像レンダラーがセットされていれば、加工後の映像フレームが
+  /// 映像レンダラーによって描画されます。
+  ///
+  /// - parameter videoFrame: 描画する映像フレーム。
+  ///                         `nil` を指定すると空の映像フレームを送信します。
   func send(videoFrame: VideoFrame?)
 
   // MARK: 終了処理
 
-  /**
-     ストリームの終了処理を行います。
-     */
+  /// ストリームの終了処理を行います。
   func terminate()
 }
 
@@ -237,7 +227,7 @@ class BasicMediaStream: MediaStream {
       // フィルターを通す
       let frame = videoFilter?.filter(videoFrame: frame) ?? frame
       switch frame {
-      case let .native(capturer: capturer, frame: nativeFrame):
+      case let .native(capturer: let capturer, frame: let nativeFrame):
         // RTCVideoSource.capturer(_:didCapture:) の最初の引数は
         // 現在使われてないのでダミーでも可？ -> ダミーにしました
         nativeVideoSource?.capturer(

Sora/PeerChannel.swift

@@ -610,7 +610,7 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
     }
   }
 
-  /** `initializeSenderStream()` にて生成されたリソースを開放するための、対になるメソッドです。 */
+  /// `initializeSenderStream()` にて生成されたリソースを開放するための、対になるメソッドです。
   private func terminateSenderStream() {
     if configuration.videoEnabled || configuration.cameraSettings.isEnabled {
       // CameraVideoCapturer が起動中の場合は停止する
@@ -910,7 +910,7 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
       type: .mediaStream,
       message: "handle signaling over WebSocket => \(signaling.typeName())")
     switch signaling {
-    case let .offer(offer):
+    case .offer(let offer):
       signalingChannel.setConnectedUrl()
 
       clientId = offer.clientId
@@ -927,14 +927,14 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
       }
 
       createAndSendAnswer(offer: offer)
-    case let .update(update):
+    case .update(let update):
       if configuration.isMultistream {
         createAndSendUpdateAnswer(forOffer: update.sdp)
       }
-    case let .reOffer(reOffer):
+    case .reOffer(let reOffer):
       createAndSendReAnswer(forReOffer: reOffer.sdp)
 
-    case let .ping(ping):
+    case .ping(let ping):
       let pong = SignalingPong()
       if ping.statisticsEnabled == true {
         nativeChannel?.statistics { report in
@@ -956,7 +956,7 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
       } else {
         signalingChannel.send(message: .pong(pong))
       }
-    case let .switched(switched):
+    case .switched(let switched):
       switchedToDataChannel = true
       signalingChannel.ignoreDisconnectWebSocket = switched.ignoreDisconnectWebSocket ?? false
       if signalingChannel.ignoreDisconnectWebSocket {
@@ -968,7 +968,7 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
       if let mediaChannel, let onDataChannel = mediaChannel.handlers.onDataChannel {
         onDataChannel(mediaChannel)
       }
-    case let .redirect(redirect):
+    case .redirect(let redirect):
       signalingChannel.redirect(location: redirect.location)
     default:
       break
@@ -983,7 +983,7 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
       type: .mediaStream,
       message: "handle signaling over DataChannel => \(signaling.typeName())")
     switch signaling {
-    case let .reOffer(reOffer):
+    case .reOffer(let reOffer):
       createAndSendReAnswerOverDataChannel(forReOffer: reOffer.sdp)
     case .push, .notify:
       // 処理は不要

Sora/SignalingChannel.swift

@@ -130,11 +130,11 @@ class SignalingChannel {
       }
       // 採用された WebSocket 以外を切断してから webSocketChannelCandidates をクリアする
       weakSelf.webSocketChannelCandidates.removeAll { $0 == webSocketChannel }
-      weakSelf.webSocketChannelCandidates.forEach {
+      for candidate in weakSelf.webSocketChannelCandidates {
         Logger.debug(
           type: .signalingChannel,
-          message: "closeing connection to \(String(describing: $0.host))")
-        $0.disconnect(error: nil)
+          message: "closeing connection to \(String(describing: candidate.host))")
+        candidate.disconnect(error: nil)
       }
       weakSelf.webSocketChannelCandidates.removeAll()
 
@@ -250,7 +250,9 @@ class SignalingChannel {
 
       state = .disconnecting
       webSocketChannel?.disconnect(error: nil)
-      webSocketChannelCandidates.forEach { $0.disconnect(error: nil) }
+      for candidate in webSocketChannelCandidates {
+        candidate.disconnect(error: nil)
+      }
       state = .disconnected
 
       Logger.debug(type: .signalingChannel, message: "call onDisconnect")