[FFmpeg-devel] [PATCH 2/3] simple_idct12: align C and x86
michael at niedermayer.cc
Wed Oct 14 13:39:22 CEST 2015
On Wed, Oct 14, 2015 at 08:46:22AM +0200, Christophe Gisquet wrote:
> > also IIUC this is just to make C and x86 match, so it could just be
> > skiped with no ill effects except that tnen x86 and C would not be
> > bitexact matches ?
> Yes, and the 12bpp.jpg fate should not be added then, because it'd
> test just the C idct.
testing just the C idct is still better than testing none
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel