[FFmpeg-cvslog] r25278 - in trunk: ffmpeg.c libavutil/Makefile libavutil/assert.h libavutil/avutil.h libavutil/rational.c

Ronald S. Bultje rsbultje
Fri Oct 1 01:37:36 CEST 2010


Hi,

2010/9/30 M?ns Rullg?rd <mans at mansr.com>:
> michael <subversion at mplayerhq.hu> writes:
>> -#include <assert.h>
>> +#include "assert.h"
>
> Using the same name as a standard header is practically guaranteed to
> cause confusion and mysterious bugs.

I second this, can you please rename this to avassert.h if nothing else?

(Review would've been nice - what problem does this solve exactly?)

Ronald



More information about the ffmpeg-cvslog mailing list