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
next prev parent 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.