[PATCH] winemp3.acm: Fixes compilation for older mpg123 library versions

Bruno Jesus 00cpxxx at gmail.com
Sun Feb 19 13:04:01 CST 2017


On Sun, Feb 19, 2017 at 3:57 PM, Sebastian Lackner <sebastian at fds-team.de>
wrote:

> On 19.02.2017 19:54, Bruno Jesus wrote:
> > From: Bruno Jesus <00cpxxx at gmail.com>
> >
> > Fixes bug https://bugs.winehq.org/show_bug.cgi?id=42483
> >
> > Signed-off-by: Bruno Jesus <00cpxxx at gmail.com>
> > ---
> >  dlls/winemp3.acm/mpegl3.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/dlls/winemp3.acm/mpegl3.c b/dlls/winemp3.acm/mpegl3.c
> > index 28a1998..100aa49 100644
> > --- a/dlls/winemp3.acm/mpegl3.c
> > +++ b/dlls/winemp3.acm/mpegl3.c
> > @@ -249,12 +249,14 @@ static  LRESULT MPEG3_StreamOpen(PACMDRVSTREAMINSTANCE
> adsi)
> >          aad->mh = mpg123_new(NULL,&err);
> >          mpg123_open_feed(aad->mh);
> >
> > +#ifdef MPG123_IGNORE_INFOFRAME
>
> Are you sure this works? On my system it is an enum, not a define.
>
>
You are absolutely right, it doesn't. I was a victim of my own "add error
inside #ifdef/else and recompile test", probably recompiled before saving.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20170219/4787df51/attachment.html>


More information about the wine-devel mailing list