[Ffmpeg-cvslog] r5740 - in trunk/libavutil: Makefile lls.c lls.h

Rich Felker dalias
Fri Jul 14 18:11:15 CEST 2006


On Fri, Jul 14, 2006 at 12:03:09PM +0200, michael wrote:
> +#undef NDEBUG // allways check asserts, the speed effect is far too small to disable them
> +#include <assert.h>

Yet there are no asserts in this file.

Also I think this is inappropriate practice. There may be a reason
that a caller does not want any chance of assert being called, or
maybe doesn't even want assert linked in for whatever strange reason.
Of course whenever you use assert it should be provable that the
assert is never failable anyway, but the compiler is not a theorem
prover and cannot solve the halting problem...

Rich





More information about the ffmpeg-cvslog mailing list