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

[Sebastian Lackner]

On 19.02.2017 19:54, Bruno Jesus wrote:
> From: Bruno Jesus
>
> Fixes bug https://bugs.winehq.org/show_bug.cgi?id=42483
> Signed-off-by: Bruno Jesus
>
> 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.
>      /* mpg123 may find a XING header in the mp3 and use that information
>       * to ask for seeks in order to read specific frames in the file.
>       * We cannot allow that since the caller application is feeding us.
>       * This fixes problems for mp3 files encoded with LAME (bug 42361)
>       */
>      mpg123_param(aad->mh, MPG123_ADD_FLAGS, MPG123_IGNORE_INFOFRAME, 0);
>
> +#endif
>      }
>      /* no encoding yet
>      else if (adsi->pwfxSrc->wFormatTag == WAVE_FORMAT_PCM &&