[FFmpeg-devel] [PATCH] fate: enable asserts at the highest level.

Michael Niedermayer michaelni at gmx.at
Sat Jan 12 16:03:46 CET 2013


On Sat, Jan 12, 2013 at 10:22:38AM +0100, Nicolas George wrote:
> Le primidi 21 nivôse, an CCXXI, Michael Niedermayer a écrit :
> > does it make sense to turn this on for all fate machines ?
> > it would slow down any on which its turned on 
> 
> For me, the slowdown is about less than 1%, hardly visible compared to
> statistical differences. Assuming the cost is similar on most architectures
> (I do not see why not; at worst it is lower on architectures without
> assembly optimizations) I believe this is worth considered the additional
> coverage.

if nothing tests assert_level 0 and 1 then coverage isnt optimal

I think having a fate client at assert level 1 and one at 2 makes
more sense
Or do you expect that assert failures are platform specific ?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- 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/20130112/bb920a7c/attachment.asc>


More information about the ffmpeg-devel mailing list