[FFmpeg-devel] [patch] allow wordexp globs in image2 file sequence import
Carl Eugen Hoyos
cehoyos
Sat Jan 1 22:44:39 CET 2011
Brian Olson <icic <at> bolson.org> writes:
> Carl Eugen had a comment about indention and line wraps. I wrapped one long
line to <80 chars.
I meant the line length in your mail.
> I found in doc/developer the guideline that long blocks (>5 lines) included
> in new if(){} blocks should have a separate indention-fixup patch. I have two
> changes right on the limit of those 5 lines, and I'd hope it's
> ultimately simpler to commit this all as one change.
Please remove those indentation changes from your patch (possibly after a real
review has been made).
Carl Eugen
More information about the ffmpeg-devel
mailing list