Open Source Telephony
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH v0 1/7] doc: Add ECNR to Handsfree
Date: Fri, 19 Apr 2013 13:01:10 -0500	[thread overview]
Message-ID: <51718666.7090902@gmail.com> (raw)
In-Reply-To: <CAKT1EBcTAEp_e=jNCfDEc2FwBwKr=7N3fB2BqvK=pkQp1R_9_Q@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 345 bytes --]

Hi Claudio,

>> I really prefer we used full words instead of an acronym.  You'll find we
>> have no acronyms in the oFono APIs.
>
> The feature's name doesn't help, which name do you prefer?
>
> * EchoCancelingAndNoiseReduction
> * EchoNoise
>
> Any other suggestion?
>

I'd go for EchoCancelingNoiseReduction

Regards,
-Denis

  reply	other threads:[~2013-04-19 18:01 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-19 17:34 [PATCH v0 1/7] doc: Add ECNR to Handsfree Claudio Takahasi
2013-04-19 15:24 ` Denis Kenzior
2013-04-19 21:17   ` Claudio Takahasi
2013-04-19 18:01     ` Denis Kenzior [this message]
2013-04-19 17:34 ` [PATCH v0 2/7] core: Add ECNR to Handsfree Features Claudio Takahasi
2013-04-19 17:34 ` [PATCH v0 3/7] core: Add ECNR in Handsfree GetProperty Claudio Takahasi
2013-04-19 17:34 ` [PATCH v0 4/7] core: Set ECNR to TRUE by default Claudio Takahasi
2013-04-19 17:34 ` [PATCH v0 5/7] include: Add ECNR to handsfree driver Claudio Takahasi
2013-04-19 17:34 ` [PATCH v0 6/7] hfpmodem: Add ECNR implementation Claudio Takahasi
2013-04-19 17:34 ` [PATCH v0 7/7] core: Add SetProperty for ECNR Claudio Takahasi
2013-04-19 20:11 ` [PATCH v0 1/7] doc: Add ECNR to Handsfree Johan Hedberg
2013-04-19 20:43   ` Claudio Takahasi
2013-04-22 12:58 ` [PATCH v1 1/7] doc: Add EchoCancelingNoiseReduction " Claudio Takahasi
2013-04-22  9:04   ` Denis Kenzior
2013-04-22 16:46     ` Claudio Takahasi
2013-04-22 11:56       ` Denis Kenzior
2013-04-22 12:58   ` [PATCH v1 2/7] core: Add "echo-canceling-and-noise-reduction" Claudio Takahasi
2013-04-22 12:58   ` [PATCH v1 3/7] core: Add EchoCancelingNoiseReduction to GetProperties Claudio Takahasi
2013-04-22 12:58   ` [PATCH v1 4/7] core: Set EchoCancelingNoiseReduction to TRUE by default Claudio Takahasi
2013-04-22 12:58   ` [PATCH v1 5/7] include: Add EchoCancelingNoiseReduction to handsfree driver Claudio Takahasi
2013-04-22  9:00     ` Denis Kenzior
2013-04-22 12:58   ` [PATCH v1 6/7] hfpmodem: Add EchoCancelingNoiseReduction Claudio Takahasi
2013-04-22 12:58   ` [PATCH v1 7/7] core: Add SetProperty for EchoCancelingNoiseReduction Claudio Takahasi
2013-04-23 12:44   ` [PATCH v2 1/7] doc: Add EchoCancelingNoiseReduction to Handsfree Claudio Takahasi
2013-04-23 12:44     ` [PATCH v2 2/7] core: Add "echo-canceling-and-noise-reduction" Claudio Takahasi
2013-04-23 12:44     ` [PATCH v2 3/7] core: Add EchoCancelingNoiseReduction to GetProperties Claudio Takahasi
2013-04-23 12:44     ` [PATCH v2 4/7] core: Set EchoCancelingNoiseReduction to TRUE by default Claudio Takahasi
2013-04-23 12:44     ` [PATCH v2 5/7] include: Add EchoCancelingNoiseReduction to handsfree driver Claudio Takahasi
2013-04-23 12:44     ` [PATCH v2 6/7] hfpmodem: Add EchoCancelingNoiseReduction Claudio Takahasi
2013-04-23 12:44     ` [PATCH v2 7/7] core: Add SetProperty for EchoCancelingNoiseReduction Claudio Takahasi
2013-04-23 17:21     ` [PATCH v3 1/7] doc: Add EchoCancelingNoiseReduction to Handsfree Claudio Takahasi
2013-04-23 15:30       ` Denis Kenzior
2013-04-23 17:21       ` [PATCH v3 2/7] core: Add "echo-canceling-and-noise-reduction" Claudio Takahasi
2013-04-23 17:21       ` [PATCH v3 3/7] core: Add EchoCancelingNoiseReduction to GetProperties Claudio Takahasi
2013-04-23 17:21       ` [PATCH v3 4/7] core: Set EchoCancelingNoiseReduction to TRUE by default Claudio Takahasi
2013-04-23 17:21       ` [PATCH v3 5/7] include: Add EchoCancelingNoiseReduction to handsfree driver Claudio Takahasi
2013-04-23 17:21       ` [PATCH v3 6/7] hfpmodem: Add EchoCancelingNoiseReduction Claudio Takahasi
2013-04-23 17:21       ` [PATCH v3 7/7] core: Add SetProperty for EchoCancelingNoiseReduction Claudio Takahasi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51718666.7090902@gmail.com \
    --to=denkenz@gmail.com \
    --cc=ofono@ofono.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox