[libav-devel] [PATCH 2/2] avutil: Move memcpy_backptr() to mem.c

2012-10-22 Thread Diego Biurrun
The function is used elsewhere and does not belong with the LZO code. --- libavcodec/dfa.c |3 ++- libavcodec/eatgv.c|2 +- libavcodec/g723_1.c |2 +- libavcodec/lcldec.c |1 + libavcodec/wmavoice.c |2 +- libavcodec/xan.c |3 +-- libavcodec/xxan.c |

Re: [libav-devel] [PATCH 2/2] avutil: Move memcpy_backptr() to mem.c

2012-10-22 Thread Diego Elio Pettenò
On 22/10/2012 10:18, Diego Biurrun wrote: -#include libavutil/lzo.h // for av_memcpy_backptr +#include libavutil/lzo.h +#include libavutil/mem.h Is lzo.h still required here? -- Diego Elio Pettenò — Flameeyes flamee...@flameeyes.eu — http://blog.flameeyes.eu/

Re: [libav-devel] [PATCH 2/2] avutil: Move memcpy_backptr() to mem.c

2012-10-22 Thread Diego Biurrun
On Mon, Oct 22, 2012 at 11:43:00AM -0700, Diego Elio Pettenò wrote: On 22/10/2012 10:18, Diego Biurrun wrote: -#include libavutil/lzo.h // for av_memcpy_backptr +#include libavutil/lzo.h +#include libavutil/mem.h Is lzo.h still required here? Yes, for AV_LZO_OUTPUT_PADDING. Now I'm

Re: [libav-devel] [PATCH 2/2] avutil: Move memcpy_backptr() to mem.c

2012-10-22 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Mon, Oct 22, 2012 at 11:43:00AM -0700, Diego Elio Pettenò wrote: On 22/10/2012 10:18, Diego Biurrun wrote: -#include libavutil/lzo.h // for av_memcpy_backptr +#include libavutil/lzo.h +#include libavutil/mem.h Is lzo.h still required here?