[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