On Mon, Sep 05, 2016 at 06:18:42AM -0500, Burt P wrote: > Signed-off-by: Burt P <pbu...@gmail.com> > --- > libavfilter/af_hdcd.c | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/libavfilter/af_hdcd.c b/libavfilter/af_hdcd.c > index cde2340..c8bda82 100644 > --- a/libavfilter/af_hdcd.c > +++ b/libavfilter/af_hdcd.c > @@ -1316,11 +1316,10 @@ static void hdcd_analyze_prepare(hdcd_state *state, > int32_t *samples, int count, > /** encode a value in the given sample by adjusting the amplitude */ > static int32_t hdcd_analyze_gen(int32_t sample, unsigned int v, unsigned int > maxv) > { > - float sflt = sample, vv = v; > - vv /= maxv; > - if (vv > 1.0) vv = 1.0; > - sflt *= 1.0 + (vv * 18); > - return (int32_t)sflt; > + static const int r = 18, m = 1024; > + int64_t s64 = sample; > + v = m + (v * r * m / maxv);
maxv is always a constant (for each call) and division is slow, maybe you want to replace it by multiplication by its inverse and adjust the /m later (that is fast as m is a constant power of 2 in unsigned context) [...[ -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Breaking DRM is a little like attempting to break through a door even though the window is wide open and the only thing in the house is a bunch of things you dont want and which you would get tomorrow for free anyway
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel