Hi, 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. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
