On Sun, Feb 19, 2017 at 4:04 PM, Bruno Jesus 00cpxxx@gmail.com wrote:
On Sun, Feb 19, 2017 at 3:57 PM, Sebastian Lackner sebastian@fds-team.de wrote:
On 19.02.2017 19:54, Bruno Jesus wrote:
From: Bruno Jesus 00cpxxx@gmail.com
Fixes bug https://bugs.winehq.org/show_bug.cgi?id=42483
Signed-off-by: Bruno Jesus 00cpxxx@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.
Would you mind helping me understand why the check at [1] fails with:
14734 configure:13848: checking for MPG123_IGNORE_INFOFRAME 14735 configure:13848: ccache gcc-4.6 -m32 -c -g -O0 conftest.c >&5 14736 configure:13848: $? = 0 14737 configure:13848: ccache gcc-4.6 -m32 -c -g -O0 conftest.c >&5 14738 configure:13848: $? = 0 14739 configure:13848: result: no
[1] http://pastebin.com/nveZQNDR
I never did this sort of change before and the best example I found was some libxml2 stuff inside configure.ac.
Thanks in advance, Bruno