Yair K. wrote:
> Hi,
>
> The attached patch does the following:
> !) Merge ossplay and ossrecord into a single executable ('ossplay' which is
> symlinked to 'ossrecord'). Behaviour is decided by looking at argv[0].
> Reason is that if a graphical interface is ever written, we'll probably
> want to have both play and record in the same app.
> 2) Add IMA-ADPCM decoding, header variations for WAVs and AIFCs are
> supported.
> 3) Add AIFF writer.
> 4) Various misc. changes.
>
Ok. Applied. Now in hg.
> P.S. I've noticed several issues with soundcard.h:
> 1. Use of C++ comments.
> 2. long long type is used once (oss_midi_time_t typedef) without
> OSS_NO_LONG_LONG define to shield it.
>
Both fixed.
> 3. OSS_DEVNODE_SIZE is a narrow fit. e.g. "/dev/oss/usb046d08a9-1/pcmin0"
> is 30 chars + 1 for null.
>
This cannot be changed because it will break binary compatibility of the
API. Driver developers need to keep this limitation in mind when
selecting the device node names. However I can add a check to the
framework for this.
Best regards,
Hannu
_______________________________________________
oss-devel mailing list
[email protected]
http://mailman.opensound.com/mailman/listinfo/oss-devel
