Hi, On Fri, May 27, 2011 at 9:13 PM, Ronald S. Bultje <[email protected]> wrote: > On Fri, May 27, 2011 at 7:53 PM, Stefano Sabatini > <[email protected]> wrote: >> Make ff* tools only accept opt_* functions taking two arguments. >> >> The distinction between functions with one and two arguments is quite >> pointless. Simplify parse_options() code. >> --- >> cmdutils.c | 6 +- >> cmdutils.h | 10 +-- >> cmdutils_common_opts.h | 2 +- >> ffmpeg.c | 215 >> ++++++++++++++++++++++++++---------------------- >> ffplay.c | 33 ++++---- >> ffprobe.c | 7 +- >> 6 files changed, 144 insertions(+), 129 deletions(-) > > This looks OK. Could you consider numbering your patches (git > send-email should do that automatically...?) so it's easy to figure > out for us in which order to apply them? Sometimes they are > interdependent.
And queued. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
