From 6fab1b0953b5b51ea3bce0a4a656dfee9a732683 Mon Sep 17 00:00:00 2001 From: zztkm Date: Mon, 22 Jul 2024 17:58:29 +0900 Subject: [PATCH 1/5] =?UTF-8?q?WrapperVideoEncoderFactory.shared.simulcast?= =?UTF-8?q?Enabled=20=E3=81=AE=E8=A8=AD=E5=AE=9A=E6=9D=A1=E4=BB=B6?= =?UTF-8?q?=E3=81=8B=E3=82=89=20spotlightEnabled=20=E3=82=92=E5=89=8A?= =?UTF-8?q?=E9=99=A4=E3=81=99=E3=82=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Sora/PeerChannel.swift | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Sora/PeerChannel.swift b/Sora/PeerChannel.swift index 4ab3472f..7b7204fe 100644 --- a/Sora/PeerChannel.swift +++ b/Sora/PeerChannel.swift @@ -195,9 +195,9 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate { lock.lock() onConnect = handler + // TODO (zztkm): WrapperVideoEncoderFactory は type: offer メッセージを受け取ったときに設定されるので、ここでの設定は不要かもしれない // サイマルキャストを利用する場合は、 RTCPeerConnection の生成前に WrapperVideoEncoderFactory を設定する必要がある - // また、スポットライトはサイマルキャストを利用しているため、同様に設定が必要になる - WrapperVideoEncoderFactory.shared.simulcastEnabled = configuration.simulcastEnabled || configuration.spotlightEnabled == .enabled + WrapperVideoEncoderFactory.shared.simulcastEnabled = configuration.simulcastEnabled signalingChannel.connect { [weak self] error in guard let weakSelf = self else { From 112b9e9b53a138d993be5ba18d57e5fd6da0ebb1 Mon Sep 17 00:00:00 2001 From: zztkm Date: Mon, 22 Jul 2024 18:17:40 +0900 Subject: [PATCH 2/5] =?UTF-8?q?format=20check=20=E3=81=A7=E3=81=AE?= =?UTF-8?q?=E6=8C=87=E6=91=98=E3=82=92=E4=BF=AE=E6=AD=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Sora/PeerChannel.swift | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Sora/PeerChannel.swift b/Sora/PeerChannel.swift index 7b7204fe..6c33f7f6 100644 --- a/Sora/PeerChannel.swift +++ b/Sora/PeerChannel.swift @@ -195,7 +195,7 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate { lock.lock() onConnect = handler - // TODO (zztkm): WrapperVideoEncoderFactory は type: offer メッセージを受け取ったときに設定されるので、ここでの設定は不要かもしれない + // TODO(zztkm): WrapperVideoEncoderFactory は type: offer メッセージを受け取ったときに設定されるので、ここでの設定は不要かもしれない // サイマルキャストを利用する場合は、 RTCPeerConnection の生成前に WrapperVideoEncoderFactory を設定する必要がある WrapperVideoEncoderFactory.shared.simulcastEnabled = configuration.simulcastEnabled From 389548a9c083dec4b7b021e2a6f43fdaa17a6c7a Mon Sep 17 00:00:00 2001 From: zztkm Date: Mon, 22 Jul 2024 18:30:18 +0900 Subject: [PATCH 3/5] =?UTF-8?q?CHANGES=20=E3=82=92=E8=A8=98=E8=BC=89?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- CHANGES.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/CHANGES.md b/CHANGES.md index 60606965..65019f27 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -24,6 +24,9 @@ - [FIX] `WrapperVideoEncoderFactory.shared.simulcastEnabled` の値を type: offer の際に設定される simulcast の値で上書きする - 認証ウェブフック成功時に払い出された type: offer の `simulcast` の値が反映されない不具合への対応 - @zztkm +- [FIX] `WrapperVideoEncoderFactory.shared.simulcastEnabled` の設定条件から `Configuration.spotlightEnabled` を削除する + - `Configuration.spotlightEnabled` はサイマルキャストを有効化するための条件ではないため削除する + - @zztkm ## 2024.2.0 From 57ff07c6a0b10f7acd5fa6a8ccab503f1af36dc9 Mon Sep 17 00:00:00 2001 From: zztkm Date: Mon, 22 Jul 2024 18:44:04 +0900 Subject: [PATCH 4/5] =?UTF-8?q?CHANGES=20=E3=81=AE=E6=9B=B8=E3=81=8D?= =?UTF-8?q?=E6=96=B9=E3=82=92=E4=BF=AE=E6=AD=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- CHANGES.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index 65019f27..0ab14df6 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -24,8 +24,9 @@ - [FIX] `WrapperVideoEncoderFactory.shared.simulcastEnabled` の値を type: offer の際に設定される simulcast の値で上書きする - 認証ウェブフック成功時に払い出された type: offer の `simulcast` の値が反映されない不具合への対応 - @zztkm -- [FIX] `WrapperVideoEncoderFactory.shared.simulcastEnabled` の設定条件から `Configuration.spotlightEnabled` を削除する - - `Configuration.spotlightEnabled` はサイマルキャストを有効化するための条件ではないため削除する +- [FIX] `Configuration.spotlightEnabled` はサイマルキャストを有効化するための条件ではないのに、判定条件に加わっていた問題を修正する + - `WrapperVideoEncoderFactory.shared.simulcastEnabled` の判定条件から `Configuration.spotlightEnabled` を削除する + - 本来は、 時点で修正すべきだったが、漏れていたので修正 - @zztkm ## 2024.2.0 From cf1a53813bf10b8b5c713d73976d6b1e7569ec4f Mon Sep 17 00:00:00 2001 From: zztkm Date: Mon, 22 Jul 2024 18:46:55 +0900 Subject: [PATCH 5/5] =?UTF-8?q?CHANGES=20=E5=86=85=E5=AE=B9=E3=81=AE?= =?UTF-8?q?=E4=BF=AE=E6=AD=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- CHANGES.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGES.md b/CHANGES.md index 0ab14df6..4bb46359 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -26,7 +26,7 @@ - @zztkm - [FIX] `Configuration.spotlightEnabled` はサイマルキャストを有効化するための条件ではないのに、判定条件に加わっていた問題を修正する - `WrapperVideoEncoderFactory.shared.simulcastEnabled` の判定条件から `Configuration.spotlightEnabled` を削除する - - 本来は、 時点で修正すべきだったが、漏れていたので修正 + - の修正漏れ - @zztkm ## 2024.2.0