W3C home > Mailing lists > Public > public-webrtc-logs@w3.org > April 2018

Re: [webrtc-pc] Split out RTCSendRtpParameters and mark parameters as required.

From: Taylor Brandstetter via GitHub <sysbot+gh@w3.org>
Date: Mon, 23 Apr 2018 20:33:36 +0000
To: public-webrtc-logs@w3.org
Message-ID: <issue_comment.created-383713355-1524515614-sysbot+gh@w3.org>
I assumed "no value" was different than "some value" (`null` != "foo") and so removing a read-only parameter would result in `InvalidModificationError`. Though the way it's worded ("*has a value* that is different...") may not make that clear enough. Is that the hole you're referring to? If so, doesn't it still exist, since there are still things (like `RTCRtpCodecParameters` fields) that aren't `required`?

-- 
GitHub Notification of comment by taylor-b
Please view or discuss this issue at https://github.com/w3c/webrtc-pc/pull/1805#issuecomment-383713355 using your GitHub account
Received on Monday, 23 April 2018 20:33:38 UTC

This archive was generated by hypermail 2.4.0 : Friday, 17 January 2020 19:22:01 UTC