[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