On Sat, Feb 06, 2016 at 11:08:16AM +0000, Rostislav Pehlivanov wrote:
> On 6 February 2016 at 03:37, Timothy Gu <timothyg...@gmail.com> wrote:
> 
> > ---
> >  libavcodec/dirac_dwt.c          | 13 ++++++++++---
> >  libavcodec/dirac_dwt_template.c | 25 +++++++++----------------
> >  2 files changed, 19 insertions(+), 19 deletions(-)
> >
> >
> LGTM

Thanks, pushed.

> 
> d->temp = (uint8_t *)(((TYPE *)d->temp) + 8); might appear confusing
> to someone as it modifies the actual pointer but that's okay since the
> pointer itself is supplied upon DWT init which happens on every single
> frame (since the wavelet transforms can differ between frames).

I think that's clear enough since the actual modification happens in the init
function as well.

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

Reply via email to