Diego Biurrun <di...@biurrun.de> writes:

> This copies the value of AV_LZO_OUTPUT_PADDING because the LZO code uses
> av_memcpy_backptr internally, but is semantically independent.
> ---
>  libavcodec/dfa.c    |    3 ++-
>  libavcodec/eatgv.c  |    4 ++--
>  libavcodec/lcldec.c |    4 +++-
>  libavutil/lzo.h     |    6 +++++-
>  4 files changed, 12 insertions(+), 5 deletions(-)

I'd much rather fix the code so it doesn't need the silly padding.
There are even comments (in dfa.c) stating that this function can't be
used due to the padding requirement, so it seems rather stupid to have
it that way.

-- 
Måns Rullgård
m...@mansr.com
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to