Re: [libav-devel] [PATCH] x86/cpu: Include system headers before local headers

2012-08-05 Thread Martin Storsjö
On Sun, 5 Aug 2012, Måns Rullgård wrote: Martin Storsjö writes: An inline function in MSVC system headers included by these headers use free(). Which function? It's in _freea in the MSVC malloc.h (which is included implicitly). It's not used by us, but we enable the equivalent of -Werro

Re: [libav-devel] [PATCH] x86/cpu: Include system headers before local headers

2012-08-05 Thread Måns Rullgård
Martin Storsjö writes: > An inline function in MSVC system headers included by these > headers use free(). Which function? > The local headers (after 239fdf1b) include internal.h that redirect > free to please_use_av_free_instead_of_free. That is because avutil.h foolishly includes common.h.

[libav-devel] [PATCH] x86/cpu: Include system headers before local headers

2012-08-05 Thread Martin Storsjö
An inline function in MSVC system headers included by these headers use free(). The local headers (after 239fdf1b) include internal.h that redirect free to please_use_av_free_instead_of_free. --- libavutil/x86/cpu.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --gi