Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Juan Carlos Castro y Castro <jcastro@instant.com.br>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@lists.sourceforge.net
Subject: Re: Another asinine question
Date: Wed, 17 May 2006 13:22:04 -0300	[thread overview]
Message-ID: <446B4DAC.1060906@instant.com.br> (raw)
In-Reply-To: <s5hmzdg3c93.wl%tiwai@suse.de>

Takashi Iwai wrote:

>>>>Another thing: you'll see I just "hung" another snd_pcm_file_t structure at the bottom of the original one. I'll understand if that offends people's coding sensibilities -- it offended mine. :-/
>>>>        
>>>>
>>>Does't only one file descriptor instead of the whole snd_pcm_file struct suffice?  Actually, it's used only in readi().
>>>      
>>>
>>Not really, especially when I finally tackle _readn. And even in the noninterleaved case, there's always the possibility read() will return a number of bytes that's not a multiple of the frame size (especially if the file is a named pipe). I'll need ->wbuf and some of those indexes to keep the remaining bytes for the next read.
>>    
>>
>Then let's create another object type.  It's confusing to use the existing object for completly another purpose.
>
>Also, "nextfile" doesn't sound intuitive in this case.  (it sounds like a linked chain of data.)
>  
>
Agree and agree. I DID say it was offensive. In my defense, boss was 
swinging the whip and I had to churn out code rather quickly. OK, that 
was a lame excuse. :-/

For the noninterleaved case, I think a plain dumb byte FIFO (buffer, 
head, tail) would be enough. I'll only be able to code on that tomorrow 
-- got an unrelated deadline to meet today.

Juan


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642

  reply	other threads:[~2006-05-17 16:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-17 14:48 Another asinine question Juan Carlos Castro y Castro
2006-05-17 15:24 ` Takashi Iwai
2006-05-17 16:22   ` Juan Carlos Castro y Castro [this message]
  -- strict thread matches above, loose matches on Subject: below --
2006-05-12 15:06 Juan Carlos Castro y Castro
2006-05-16 13:38 ` Takashi Iwai
2006-05-16 14:33   ` Juan Carlos Castro y Castro
2006-05-16 14:38     ` Takashi Iwai
2006-05-16 16:28       ` Juan Carlos Castro y Castro
2006-05-17 13:26         ` Takashi Iwai
2006-05-16 18:44     ` Juan Carlos Castro y Castro

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=446B4DAC.1060906@instant.com.br \
    --to=jcastro@instant.com.br \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox