On Tue, 28 Feb 2012 16:40:56 -0500, Justin Ruggles <[email protected]> 
wrote:
> On 02/28/2012 06:24 AM, Anton Khirnov wrote:
> 
> > it will be useful for attached pictures in ID3v2
> > ---
> >  libavformat/internal.h |    5 +++++
> >  libavformat/matroska.c |    2 +-
> >  libavformat/matroska.h |    8 ++------
> >  3 files changed, 8 insertions(+), 7 deletions(-)
> 
> 
> LGTM.
> 
> Although, do we really need to use FF prefix for internal struct names?
> 

Not really, and I agree that it looks ugly.
Removed locally.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to