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.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to