[FFmpeg-devel] [PATCH]Simplify ASSERT_LEVEL test in common.h

Michael Niedermayer michaelni at gmx.at
Tue Jan 8 02:47:51 CET 2013


On Sun, Jan 06, 2013 at 06:10:58PM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached matches avassert.h
> 
> Please comment, Carl Eugen

>  common.h |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 78d0c6ca5857778674da4e036dacd521404ccd85  patchassertlevel.diff

without the HAVE_AV_CONFIG_H protection i  think it could break a user
app that deines ASSERT_LEVEL to a non nummeric value.
its not called AV_ASSERT_LEVEL and not FF_ASSERT_LEVEL, so this
shouldt happen

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130108/baf500df/attachment.asc>


More information about the ffmpeg-devel mailing list