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  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.

wineaudiofile.acm: Initial commit

[Aric Stewart]

Thanks!

Henri Verbeet wrote:
> > -
> >    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.

the problem is that that gets the error codes backwards ('typ?' outputs 
as '?pyt')

-aric

wineaudiofile.acm: Initial commit

[Ken Thomases]

On Mar 12, 2010, at 12:12 PM, Aric Stewart wrote:
> Henri Verbeet wrote:
> > > -
> > >    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.
>
> the problem is that that gets the error codes backwards ('typ?' outputs as '?pyt')

Take a look at wine_dbgstr_fourcc() in dlls/winecoreaudio.drv/coreaudio.h.

-Ken