Re: [flac-dev] GCC/clang compilation issues

2014-12-07 Thread Oliver Stöneberg
> I don't know, its your build system and I haven't seen it, nor do > I have the time and motivation to look at it (unless of course you > were paying me for my time). I willing give my time to maintain > FLAC. I have not signed up to spend my time on your project. And I am willing to looking at

Re: [flac-dev] GCC/clang compilation issues

2014-12-07 Thread Oliver Stöneberg
> Oliver Stöneberg wrote: > > > > > > Oliver Stöneberg wrote: > > > > > > > I finally got around to trying to update FLAC for the MAME/MESS > > > > project again. There were several issues I was able to fix and will > > > >

Re: [flac-dev] GCC/clang compilation issues

2014-12-06 Thread Oliver Stöneberg
> Oliver Stöneberg wrote: > > > I finally got around to trying to update FLAC for the MAME/MESS > > project again. There were several issues I was able to fix and will > > submit patches later, but I hit one roadblock with GCC and clang: > > > > src/lib/l

[flac-dev] GCC/clang compilation issues

2014-12-06 Thread Oliver Stöneberg
Hi, I finally got around to trying to update FLAC for the MAME/MESS project again. There were several issues I was able to fix and will submit patches later, but I hit one roadblock with GCC and clang: src/lib/libflac/libFLAC/stream_encoder.c:1696:43: error: cast from function call of typ

Re: [flac-dev] #410 Incorrect HAVE_CONFIG_H checks

2014-03-24 Thread Oliver Stöneberg
> Oliver Stöneberg wrote: > > > None of those. Just pulling out the files necessary and compiling > > them into a library - that's the way it was done when it was added to > > MAME/MESS. > > That's the beautiful thing about open source. When

Re: [flac-dev] #410 Incorrect HAVE_CONFIG_H checks

2014-03-23 Thread Oliver Stöneberg
> Oliver Stöneberg wrote: > > > Copying my report from sf.net: > > > > The source uses a mixture of "#ifdef HAVE_CONFIG_H" and "#if HAVE_CONFIG_H" > > checks. Judging from > > > > DEFS=-DHAVE_CONFIG_H > > > > in the con

[flac-dev] #410 Incorrect HAVE_CONFIG_H checks

2014-03-23 Thread Oliver Stöneberg
Copying my report from sf.net: The source uses a mixture of "#ifdef HAVE_CONFIG_H" and "#if HAVE_CONFIG_H" checks. Judging from DEFS=-DHAVE_CONFIG_H in the configure I supposed it should be "#ifdef" in all cases. These incorrect checks cause build errors since it tries to include the config.h

Re: [flac-dev] PATCH for cpu.c

2014-03-23 Thread Oliver Stöneberg
> Oliver Stöneberg wrote: > > >> Unfortunately it breaks x86_64 build (where FLAC__CPU_X86_64 is defined, > >> and > >> FLAC__CPU_IA32 isn't). Maybe it's simpler to turn them into #defines such > >> as > >> > >>

Re: [flac-dev] PATCH for cpu.c

2014-03-23 Thread Oliver Stöneberg
> Oliver Stöneberg wrote: > > > This is simply fixed by putting those unused constants into the > > proper defines. I attached a patch against git 70b078c. > > Unfortunately it breaks x86_64 build (where FLAC__CPU_X86_64 is defined, and > FLAC__CPU_IA32 isn't).

[flac-dev] PATCH for cpu.c

2014-03-22 Thread Oliver Stöneberg
Hi, after having some discussion of the FPU/SSE behavior on the sf.net bug tracker as well as having some other changes we made for the MAME/MESS project sitting around I thought about joining this list to make things easier. First one is the latest modification we had to make to compile with