Hi, On Sat, Jul 16, 2011 at 2:08 PM, Stefano Sabatini <[email protected]> wrote: > Also remove now unused variable. > --- > doc/ffplay.texi | 3 ++- > ffplay.c | 6 +++--- > 2 files changed, 5 insertions(+), 4 deletions(-) [..] > static int opt_frame_pix_fmt(const char *opt, const char *arg) > { > - frame_pix_fmt = av_get_pix_fmt(arg); > - return 0; > + av_log(NULL, AV_LOG_ERROR, > + "Option '%s' has been removed, use private format options > instead\n"); > + return AVERROR(EINVAL); > }
I don't like this. I've said it before, I want to keep application compatibility for a while to assist users that are upgrading. We do the same for API, and it's not very hard. Putting it under a proper FF_API_* automates removing it in 1-2 ABI bumps. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
