Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* Is this a bug?
@ 2002-07-15 13:57 Tisserand Patrice
  2002-07-15 14:08 ` Takashi Iwai
  0 siblings, 1 reply; 2+ messages in thread
From: Tisserand Patrice @ 2002-07-15 13:57 UTC (permalink / raw)
  To: alsa-devel

Hi,

I wasn't enable to get poll descriptors for a sequencer opened for
output only,
so I checked alsa-lib/seq/seq.c and found on line 320 in function
snd_seq_poll_descriptors:

if ((events & POLLOUT) && space >= 1) {
  assert(seq->streams & SND_SEQ_OPEN_INPUT);
  revents |= POLLOUT|POLLERR;
 }

Shouldn't it read: assert(seq->streams & SND_SEQ_OPEN_OUTPUT); ?

Regards,

--
Patrice Tisserand --- email:mailto:Patrice.Tisserand@ircam.fr
IRCAM 1 place Stravinsky 75004 Paris FRANCE





-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Is this a bug?
  2002-07-15 13:57 Is this a bug? Tisserand Patrice
@ 2002-07-15 14:08 ` Takashi Iwai
  0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2002-07-15 14:08 UTC (permalink / raw)
  To: Tisserand Patrice; +Cc: alsa-devel

At Mon, 15 Jul 2002 15:57:35 +0200,
Tisserand Patrice wrote:
> 
> Hi,
> 
> I wasn't enable to get poll descriptors for a sequencer opened for
> output only,
> so I checked alsa-lib/seq/seq.c and found on line 320 in function
> snd_seq_poll_descriptors:
> 
> if ((events & POLLOUT) && space >= 1) {
>   assert(seq->streams & SND_SEQ_OPEN_INPUT);
>   revents |= POLLOUT|POLLERR;
>  }
> 
> Shouldn't it read: assert(seq->streams & SND_SEQ_OPEN_OUTPUT); ?

yep, an obvious typo.  now fixed on cvs.

thanks,

Takashi


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2002-07-15 14:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-07-15 13:57 Is this a bug? Tisserand Patrice
2002-07-15 14:08 ` Takashi Iwai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox