Skip to content

Commit

Permalink
Merge pull request #225 from shiguredo/feature/deprecated-multistream…
Browse files Browse the repository at this point in the history
…-option

multistreamEnabled を非推奨扱いにする
  • Loading branch information
zztkm authored Jan 27, 2025
2 parents e4146d7 + acda327 commit e27eca4
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 7 deletions.
5 changes: 5 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,11 @@

## develop

- [UPDATE] multistreamEnabled を非推奨扱いにする
- multistreamEnabled の設定が不要なイニシャライザを `Configuration` に追加する
- ドキュメントコメントに非推奨扱いの旨を追加する
- @zztkm

## 2025.1.1

**リリース日**: 2025-01-23
Expand Down
15 changes: 9 additions & 6 deletions Sora/Configuration.swift
Original file line number Diff line number Diff line change
Expand Up @@ -86,13 +86,16 @@ public struct Configuration {
public var role: Role

/// マルチストリームの可否
public var multistreamEnabled: Bool
///
/// レガシーストリーム機能は 2025 年 6 月リリースの Sora にて廃止します
/// そのため、multistreamEnabled の使用は非推奨です
public var multistreamEnabled: Bool?

/// :nodoc:
var isMultistream: Bool {
switch role {
default:
return multistreamEnabled
return multistreamEnabled ?? true
}
}

Expand Down Expand Up @@ -226,12 +229,12 @@ public struct Configuration {
- parameter url: サーバーの URL
- parameter channelId: チャネル ID
- parameter role: ロール
- parameter multistreamEnabled: マルチストリームの可否
- parameter multistreamEnabled: マルチストリームの可否(デフォルトは指定なし)
*/
public init(url: URL,
channelId: String,
role: Role,
multistreamEnabled: Bool)
multistreamEnabled: Bool? = nil)
{
urlCandidates = [url]
self.channelId = channelId
Expand All @@ -244,12 +247,12 @@ public struct Configuration {
- parameter urlCandidates: シグナリングに利用する URL の候補
- parameter channelId: チャネル ID
- parameter role: ロール
- parameter multistreamEnabled: マルチストリームの可否
- parameter multistreamEnabled: マルチストリームの可否(デフォルトは指定なし)
*/
public init(urlCandidates: [URL],
channelId: String,
role: Role,
multistreamEnabled: Bool)
multistreamEnabled: Bool? = nil)
{
self.urlCandidates = urlCandidates
self.channelId = channelId
Expand Down
5 changes: 4 additions & 1 deletion Sora/PeerChannel.swift
Original file line number Diff line number Diff line change
Expand Up @@ -294,7 +294,10 @@ class PeerChannel: NSObject, RTCPeerConnectionDelegate {
message: "did connect to signaling channel")

var role: SignalingRole
var multistream = configuration.multistreamEnabled || configuration.spotlightEnabled == .enabled
var multistream = configuration.multistreamEnabled
if configuration.spotlightEnabled == .enabled {
multistream = true
}
switch configuration.role {
case .sendonly:
role = .sendonly
Expand Down
3 changes: 3 additions & 0 deletions Sora/Signaling.swift
Original file line number Diff line number Diff line change
Expand Up @@ -270,6 +270,9 @@ public struct SignalingConnect {
public var sdp: String?

/// マルチストリームの可否
///
/// レガシーストリーム機能は 2025 年 6 月リリースの Sora にて廃止します
/// そのため、multistreamEnabled の使用は非推奨です
public var multistreamEnabled: Bool?

/// 映像の可否
Expand Down

0 comments on commit e27eca4

Please sign in to comment.