[flac-dev] Include directories
hendricks266 at gmail.com
Mon Jun 23 14:39:34 PDT 2014
Thanks for looking over my patch.
> > * Account for the updates to UTF-8 on Windows.
> One of these updates, added an "#ifdef _WIN32" to the file
> src/share/win_utf8_io/win_utf8_io.c. Is this really necessary? Is
> it not possible to just not include that file when not compiling
> for windows?
You're right. I changed my patch to only specify "-lwin_utf8_io" under Windows.
> Those Makefile variables are somewhat non-standard. Is there any
> reason you chose CCFLACGS and CCCFLAGS instead of the more usual
> CFLAGS and CXXFLAGS?
I usually see the C++ compiler as CXX, while FLAC uses CCC. I suppose
that threw me off. Updated my patch to use CFLAGS and CXXFLAGS.
> I would appreciate it if you could update you patch to address
> the issues I raised above.
I attached a revised patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 4396 bytes
Desc: not available
Url : http://lists.xiph.org/pipermail/flac-dev/attachments/20140623/4a53d052/attachment.bin
More information about the flac-dev