W3C home > Mailing lists > Public > public-webrtc@w3.org > November 2014

Re: Proposal for RtpSender.getParameters and .setParameters

From: Martin Thomson <martin.thomson@gmail.com>
Date: Mon, 3 Nov 2014 15:15:23 -0800
Message-ID: <CABkgnnUP4qhiym8=FHM2RbpAYexqjE=6QqXNNaQ7kLspjHUdkQ@mail.gmail.com>
To: Peter Thatcher <pthatcher@google.com>
Cc: "public-webrtc@w3.org" <public-webrtc@w3.org>
On 3 November 2014 13:16, Peter Thatcher <pthatcher@google.com> wrote:
>  RTCRtpParameters getParameters();

getCurrentParameters(); ?

>     unsigned long maxBitrate = null;

Don't assign a default to this one.  Presence of absence can be used
as a signal (and defaults ruin that; you'll copy defaults across.)

Didn't you have an SSRC in RTCRtpEncodingParameters?  With the
proposal as stated, it's hard to see why you might have this:
>  sequence<RTCRtpEncodingParameters> encodings;
Received on Monday, 3 November 2014 23:15:51 UTC

This archive was generated by hypermail 2.3.1 : Monday, 23 October 2017 15:19:42 UTC