[flac-dev] Patch to add Unicode filename support for win32 flac

JonY 10walls at gmail.com
Sun Mar 17 15:10:44 PDT 2013


On 3/17/2013 23:01, LRN wrote:
>> All those ifdefs will at least be confined rather than spread out 
>> through the code.
> I did it plibc-style:
> 
> in compat.h:
> #if defined(_WIN32)
> #define FOPEN grabbag__fopen_utf8_wrapper
> #else
> #define FOPEN fopen
> #endif
> 
> in grabbag:
> #if defined(_WIN32)
> <implement grabbag__fopen_utf8_wrapper, which has the same signature
> as fopen, but does utf8->utf16 conversion internally, then calls wfopen>
> #endif
> 
> and replace "fopen" with "FOPEN" everywhere else.

Don't do that, it leaks into the system headers and breaks mingw if
FLAC_USE_FOPEN_UTF8 is defined.

Call the wrappers directly instead of using a macro.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: OpenPGP digital signature
Url : http://lists.xiph.org/pipermail/flac-dev/attachments/20130318/c58ff9fc/attachment.pgp 


More information about the flac-dev mailing list