Ben Wing <benwing666(a)gmail.com> wrote:
yes, as long as the user can override it.
So it seems that ALSA should be an alternative to native sound, rather
than a type of native sound as my initial patch made it. That way, the
existing configure options would make it easy to not use ALSA if
somebody so desired. I would like to check for ALSA first, before the
native check, to fix the problem I am experiencing.
A drawback to that approach is that I'll need a new config.h symbol,
HAVE_ALSA_SOUND, and I'll have to add "|| defined(HAVE_ALSA_SOUND)"
clauses to the existing "defined(HAVE_NATIVE_SOUND)" clauses in sound.c.
Does anybody object to this approach?
and your code looks ok to me.
Great. Thanks for the review. I've since figured out how to set the
volume, by the way, so I think the actual code is good to go. I just
need to get configure right now, and then modify the documentation.
Also, ALSA is capable of playing a few more formats than my detector
detects (IEC-958, MPEG, GSM). I know how to tell when a file is in one
of those formats, but I don't know how to extract the bitrate and the
number of tracks/channels in those formats. If anybody in the audience
has that information, please let me know.
--
Jerry James, Assistant Professor james(a)xemacs.org
Computer Science Department
http://www.cs.usu.edu/~jerry/
Utah State University