Hi, On Thu, Jun 2, 2011 at 12:14 AM, Kostya <[email protected]> wrote: > On Wed, Jun 01, 2011 at 11:57:00PM -0700, Ronald S. Bultje wrote: >> From: Alexander Strange <[email protected]> >> >> --- >> libavcodec/error_resilience.c | 45 +++- >> libavcodec/h263dec.c | 8 + >> libavcodec/h264.c | 736 >> +++++++++++++++++++++++++++++++---------- >> libavcodec/h264.h | 2 + >> libavcodec/h264_direct.c | 41 ++- >> libavcodec/mpeg12.c | 36 ++- >> libavcodec/mpeg4videodec.c | 20 +- >> libavcodec/mpegvideo.c | 259 ++++++++++++--- >> libavcodec/mpegvideo.h | 18 +- >> libavcodec/mpegvideo_enc.c | 24 +- >> libavcodec/pthread.c | 2 +- >> libavcodec/utils.c | 4 +- >> tests/fate/h264.mak | 204 ++++++------ >> tests/ref/vsynth1/error | 6 +- >> tests/ref/vsynth2/error | 6 +- >> tests/regression-funcs.sh | 2 +- >> 16 files changed, 1058 insertions(+), 355 deletions(-) > > Looks mostly good (except for some strange FIXMEs in error_resilience.c). > And somebody should reformat all source files after this according to modern > guidelines.
Thanks for the review, I'll queue and push this later today. I agree with all you just said. :-). Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
