[alsa-devel] [PATCH] seq: fix snd_seq_set_queue_tempo() usage example in the documentation

Antonio Ospite ao2 at ao2.it
Sun Oct 1 11:23:05 CEST 2017


On Fri, 29 Sep 2017 11:04:25 +0900
Takashi Sakamoto <o-takashi at sakamocchi.jp> wrote:

> Hi,
> 
> On Sep 28 2017 22:46, Antonio Ospite wrote:
> > snd_seq_set_queue_tempo() requires a queue id as the second argument,
> > fix the example in documentation to reflect that.
> > 
> > Also add the queue id as an argument of the set_tempo() function, just
> > to keep the whole example compilable.
> > 
> > Signed-off-by: Antonio Ospite <ao2 at ao2.it>

[...]
> This looks good catch for documentation.
> 
> Reviewed-by: Takashi Sakamoto <o-takashi at sakamocchi.jp>
>

Thanks Takashi.

> Unfortunately, a maintainer of this subsystem is absent this week. Your 
> patch is going to put into merge queue after he's back. It may takes a 
> bit longer.
> 

No problem about the delay, it's surely not a critical fix. :)

Ciao ciao,
   Antonio

-- 
Antonio Ospite
https://ao2.it
https://twitter.com/ao2it

A: Because it messes up the order in which people normally read text.
   See http://en.wikipedia.org/wiki/Posting_style
Q: Why is top-posting such a bad thing?


More information about the Alsa-devel mailing list