On 2012-12-08 03:16:18 +0100, Luca Barbato wrote: > On 12/8/12 2:51 AM, Luca Barbato wrote: > > On 12/8/12 2:45 AM, Clément Bœsch wrote: > >> On Sat, Dec 08, 2012 at 02:17:04AM +0100, Luca Barbato wrote: > >> Review in form of patches mostly welcome for this one > > > > It is an uncrustify pass with some of the glaring ugliness removed, I > > was expecting to miss some and thus I asked for updated patches as > > review process to speed up cleaning up the cleaning up. > > > > An update will follow up tomorrow I guess. > > Patch postponed pending Janne work on h264 for now.
You misunderstood me. This file is save. Current diffstat for adaptive-mt h264 is libavcodec/h264.c | 479 +++++++++++++++++++++++++++++++++++++++++++++-------------------------- libavcodec/h264.h | 6 + libavcodec/h264_direct.c | 4 +- libavcodec/h264_ps.c | 3 + libavcodec/h264_refs.c | 30 +++-- libavcodec/mpegvideo.c | 14 ++- libavcodec/svq3.c | 7 +- libavformat/mov.c | 2 +- 8 files changed, 350 insertions(+), 195 deletions(-) So reindenting h264.c and maybe h264_refs.c would be annoying, the rest is manegable. Janne _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
