On Tue, Feb 14, 2012 at 02:45:34PM +0100, Janne Grunau wrote: > On 2012-02-14 07:21:09 +0100, Kostya Shishkov wrote: > > On Mon, Feb 13, 2012 at 10:04:37PM +0100, Janne Grunau wrote: > > > Fixes bug 220. > > > --- > > > libavcodec/rv34.c | 134 > > > +++++++++++++++++++++++++++++++---------------------- > > > 1 files changed, 79 insertions(+), 55 deletions(-) > > > > nit: freeing buffers can be factored out into separate function too > > indeed, see below. > > ---8<--- > Factors all context dynamic memory handling to its own functions. > Fixes bug 220. > --- > libavcodec/rv34.c | 146 > ++++++++++++++++++++++++++++++----------------------- > 1 files changed, 82 insertions(+), 64 deletions(-)
lgtm _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
