Skip to content

Ignore comma-separated rid alternatives. - #2813

Merged
jan-ivar merged 2 commits into
w3c:mainfrom
jan-ivar:alternatives
Jan 11, 2023
Merged

Ignore comma-separated rid alternatives.#2813
jan-ivar merged 2 commits into
w3c:mainfrom
jan-ivar:alternatives

Conversation

@jan-ivar

@jan-ivar jan-ivar commented Jan 5, 2023

Copy link
Copy Markdown
Member

Fixes #2769. @docfaraday PTAL


Preview | Diff

@jan-ivar jan-ivar self-assigned this Jan 5, 2023
@jan-ivar

jan-ivar commented Jan 5, 2023

Copy link
Copy Markdown
Member Author

There was a choice here between removing alternatives before or after removing duplicates. I.e.

a=simulcast:recv 1,2,3;2,4,5;3,5,6

...could either reduce to:

a=simulcast:recv 1;4;6

...or

a=simulcast:recv 1;2;3

I chose the latter, because it seems sane.

@dontcallmedom dontcallmedom added the Simulcast Issue relating to Simulcast label Jan 5, 2023
@jan-ivar
jan-ivar merged commit a34a4cf into w3c:main Jan 11, 2023
@jan-ivar
jan-ivar deleted the alternatives branch January 11, 2023 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Editors can integrate Simulcast Issue relating to Simulcast

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handling of simulcast attributes with multiple choices in a version seems to be underspecified

3 participants