On Thu, Dec 18, 2014 at 12:12:14AM +0100, Stefano Sabatini wrote: [...] > > +static void mul_thrmat_c(FSPPContext *p, int q) > > +{ > > + int a; > > + for (a = 0; a < 64; a++) > > + ((short*)p->threshold_mtx)[a] = q * > > ((short*)p->threshold_mtx_noq)[a];//ints faster in C > > short -> int8_t (probably more readable)
whatever effect it may have on readability it is wrong and not working Please take more time for the reviews, i think its importat that the suggestions you make do not add bugs. It can take alot of time to find such bugs later [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User questions about the command line tools should be sent to the ffmpeg-user ML. And questions about how to use libav* should be sent to the libav-user ML.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel