[FFmpeg-devel] [PATCH] do not #include assert.h directly

Måns Rullgård mans
Wed May 14 00:49:18 CEST 2008


Michael Niedermayer <michaelni at gmx.at> writes:

> On Tue, May 13, 2008 at 07:47:56PM +0100, Mans Rullgard wrote:
>> libavutil/internal.h includes assert.h, defining NDEBUG if DEBUG is
>> not defined.  Including assert.h again has no effect, and is possibly
>> confusing.
>
> IMHO internal.h should not #include assert.h

I figured it might be a good idea to have the same assert() behaviour
(enabled/disabled) everywhere.  Either way, it should be consistent.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-devel mailing list