[alsa-devel] [PATCH] ascenario: Add scenario support to alsa-lib

Mark Brown broonie at opensource.wolfsonmicro.com
Thu Oct 1 15:24:18 CEST 2009


On Thu, Oct 01, 2009 at 03:19:48PM +0200, Stefan Schmidt wrote:
> On Thu, 2009-10-01 at 13:28, Mark Brown wrote:

> > > +/** Scenario ID's
> > > + *
> > > + * Standard Scenario ID's - Add new scenarios at the end.
> > > + */

> > Extra 's here.

> That way?
> /**
>  * Scenario ID's
>  *
>  * Standard Scenario ID's - Add new scenarios at the end.
>  */

There shouldn't be any 's in the above at all ("Scenarios IDs" instead),
those are just plain plurals - just a trivial grammar nit.

> > I guess the idea is that in the future this will be removed and the
> > scenario API will use inotify or similar to pick up changes.

> Indeed. Certainly a Todo item for later enhancements. Do you guys prefer such
> todo items in the code or noted somewhere else?

I have no great preference either way.


More information about the Alsa-devel mailing list