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@ao2.it

src/seq/seq.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/seq/seq.c b/src/seq/seq.c index d5ed1c6a..808c7915 100644 --- a/src/seq/seq.c +++ b/src/seq/seq.c @@ -452,13 +452,13 @@ For setting these tempo parameters, use #snd_seq_queue_tempo_t record. For example, to set the tempo of the queue <code>q</code> to 48 PPQ, 60 BPM, \code -void set_tempo(snd_seq_t *handle) +void set_tempo(snd_seq_t *handle, int queue) { snd_seq_queue_tempo_t *tempo; snd_seq_queue_tempo_alloca(&tempo); snd_seq_queue_tempo_set_tempo(tempo, 1000000); // 60 BPM snd_seq_queue_tempo_set_ppq(tempo, 48); // 48 PPQ

•    snd_seq_set_queue_tempo(handle, tempo);
  

•    snd_seq_set_queue_tempo(handle, queue, tempo);
  

  } \endcode

This looks good catch for documentation.

Reviewed-by: Takashi Sakamoto o-takashi@sakamocchi.jp

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.

Thanks

Takashi Sakamoto