Re: [PATCH] seq: fix snd_seq_set_queue_tempo() usage example in the documentation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 29 Sep 2017 11:04:25 +0900
Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx> 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@xxxxxx>

[...]
> This looks good catch for documentation.
> 
> Reviewed-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx>
>

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?
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux