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

Re: Proposal: Fire a toneDiscarded event while discarding the invalid DTMF values.

From: Adam Bergkvist <adam.bergkvist@ericsson.com>
Date: Tue, 17 Jun 2014 08:41:30 +0200
Message-ID: <539FE31A.1090901@ericsson.com>
To: <kiran.guduru@samsung.com>, Martin Thomson <martin.thomson@gmail.com>, Harald Alvestrand <harald@alvestrand.no>
CC: Stefan Håkansson LK <stefan.lk.hakansson@ericsson.com>, tim panton <thp@westhawk.co.uk>, Justin Uberti <juberti@google.com>, "public-webrtc@w3.org" <public-webrtc@w3.org>
On 2014-06-16 12:05, Kiran Kumar Guduru wrote:
> ------- *Original Message* -------
>
> *Sender* : Adam Bergkvist<adam.bergkvist@ericsson.com>
>
> *Date* : Jun 16, 2014 16:24 (GMT+09:00)
>
> *Title* : Re: Proposal: Fire a toneDiscarded event while discarding the
> invalid DTMF values.
>
> On 2014-06-14 21:22, Martin Thomson wrote:
>  > On 14 June 2014 02:19, Harald Alvestrand wrote:
>  >>> FWIW, I'm with Tim on this one. I also think it makes it clearer -
>  >>> either the UA throws, or it sends all symbols.
>  >>
>  >> FWIW, I think the spec will be simpler if we throw
>  >> IllegalCharacterException on strings that contain any character except
>  >> [0-9A-Da-d*#,], and have no special treatment at sendout time (illegal
>  >> characters can't occur there).
>  >
>  >
>  > FWIW, FWIW isn't necessary.  This is simply good design.  Having
>  > something blow up early avoids nasty deferred surprises.
>
> So after a little detour we're back at the same set of recognized
> characters, but we throw instead of ignore. As a result, the problem
> that started this thread doesn't exist anymore.
>
> [Kiran] It means, does WebRTC going to accept "-", "(", ")"
> characters with the some text explaining it or is it going to raise
> expection?

We keep the same set of recognized characters ([0-9A-Da-d*#,] from 
Harald's post above), but we throw instead of silently ignore any other.

/Adam
Received on Tuesday, 17 June 2014 06:41:57 UTC

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