[FFmpeg-devel] [PATCH 4/4] mov: simplify mov_read_chapters() by using avio_get_str16be

Janne Grunau janne-ffmpeg
Tue Jan 25 21:14:05 CET 2011


On Mon, Jan 24, 2011 at 09:06:26PM -0800, Baptiste Coudurier wrote:
> On 01/24/2011 10:06 AM, Ronald S. Bultje wrote:
> > Hi,
> >
> > On Mon, Jan 24, 2011 at 12:56 PM, Reimar D?ffinger
> > <Reimar.Doeffinger at gmx.de>  wrote:
> >> On Sun, Jan 23, 2011 at 09:03:01PM +0100, Anton Khirnov wrote:
> >>>  From 8289638633f28bdb6fc99dd3ff8be39c7e6489a0 Mon Sep 17 00:00:00 2001
> >>> From: Anton Khirnov<anton at khirnov.net>
> >>> Date: Sun, 23 Jan 2011 12:35:23 +0100
> >>> Subject: [PATCH] mov: simplify mov_read_chapters() by using avio_get_str16be
> >>>
> >>> It probably also fixes a memleak or two.
> >>> ---
> >>>   libavformat/mov.c |   36 +++++++++++++++---------------------
> >>>   1 files changed, 15 insertions(+), 21 deletions(-)
> >>
> >> Looks ok to me, but I am not the maintainer.
> >
> > OK to me too. Baptiste, do you want to review more mov work?
> 
> Humm, yes, patch looks ok.

queued,

Janne



More information about the ffmpeg-devel mailing list