Re: wineaudiofile.acm: Initial commit

[Henri Verbeet]

Not claiming to know a whole lot about OS X or audio, but these are
some general things I noticed:

On 12 March 2010 16:41, Aric Stewart <aric(a)codeweavers.com> wrote:
> +/* leopad functions */
> Typo.
> +static tAudioFileStreamOpen pAudioFileStreamOpen = NULL;
> +static tAudioFileStreamSeek pAudioFileStreamSeek = NULL;
> +static tAudioFileStreamParseBytes  pAudioFileStreamParseBytes = NULL;
> +static tAudioFileStreamClose pAudioFileStreamClose = NULL;
> Explicitly initializing these to NULL is superfluous.
> +typedef	struct
> +{
> +    int		nChannels;
> +    int		nBits;
> +    int		rate;
> +} Format;
> You have some stray tabs (not just here).
> +    TRACE("ndst %i %p  <-  %i %p\n",*ndst,dst,*nsrc, src);
> *ndst and *nsrc are unsigned.
> +        ERR("Feed Error %c%c%c%c\n", (char) (ret>> 24),
> +                                     (char) (ret>> 16),
> +                                     (char) (ret>> 8),
> +                                     (char) ret);
> debugstr_an((const char *)&ret, 4); may be easier / nicer.
> +OSStatus Mp3AudioConverterComplexInputDataProc (
> +void Mp3PropertyListenerProc ( void *inClientData, void* inAudioFileStream,
> +void Mp3PacketsProc (
> These should probably be static.