[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