On Fri, 8 Feb 2013 11:04:25 +0100, Kostya Shishkov <[email protected]> 
wrote:
> On Fri, Feb 08, 2013 at 07:26:34AM +0100, Anton Khirnov wrote:
> > ---
> > * moved ER_ defines to error_resilience.h
> > * removed error_status_table from MpegEncContext
> > * check for field_picture, not interlaced_frame in ff_er_frame_end.
> >   this is what the previous code did
> > * in h264 decode_mb, don't set the parameters that won't get used
> > ---
> >  libavcodec/error_resilience.c |  403 
> > ++++++++++++++++-------------------------
> >  libavcodec/error_resilience.h |   76 ++++++++
> >  libavcodec/h261dec.c          |    2 +-
> >  libavcodec/h263dec.c          |   20 +-
> >  libavcodec/h264.c             |   63 +++++--
> >  libavcodec/intrax8.c          |    2 +-
> >  libavcodec/mpeg12.c           |   20 +-
> >  libavcodec/mpeg4videodec.c    |   10 +-
> >  libavcodec/mpegvideo.c        |   95 +++++++++-
> >  libavcodec/mpegvideo.h        |   23 +--
> >  libavcodec/mpegvideo_enc.c    |    2 +-
> >  libavcodec/mss2.c             |    4 +-
> >  libavcodec/rv10.c             |    8 +-
> >  libavcodec/rv34.c             |   12 +-
> >  libavcodec/vc1dec.c           |   22 +--
> >  15 files changed, 427 insertions(+), 335 deletions(-)
> >  create mode 100644 libavcodec/error_resilience.h
> 
> doesn't look that horrible but it would be much easier to review if you'd be
> less lazy and kept old names for picture pointers

Not really, because they weren't pointers before. They were Pictures.
So I had to touch all those lines anyway.

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

Reply via email to