[flac-dev] test_streams dependencies

Brian Willoughby brianw at audiobanshee.com
Sun Jan 31 11:26:40 PST 2016

My assumption is that the code was written to call flac_fopen() so that it is portable to every operating system, even those without fopen(). If you replace flac_fopen() with fopen(), then the tests won't compile on systems that don't have fopen().

You could simply have the following:

#if defined _MSC_VER || defined __MINGW32__
#define flac_fopen fopen

In a header, or simply at the top of test_streams/main.c where there's already some conditional compilation.


On Jan 31, 2016, at 7:45 AM, lvqcl <lvqcl.mail at gmail.com> wrote:
> test_streams currently depends on grabbag and (on Windows) on win_utf8_io libs.
> It depends on win_utf8_io only because it uses flac_fopen() function.
> It will become to depend on libFLAC when all file functions will be moved
> from win_utf8_io to libFLAC. Not a big problem, but it is possible to avoid
> this dependency by replacing flac_fopen() with fopen().
> test_streams doesn't open/create files with unicode filenames, so it won't
> affect its functionality.
> So, what's better:
> a) replace all flac_fopen() calls inside test_streams/main.c with fopen()
> b) leave it as is, so that on Windows build of test_streams will depend on libFLAC

More information about the flac-dev mailing list