Hi, 2011/4/21 Aℓex Converse <aconve...@google.com>: > On Thu, Apr 21, 2011 at 11:02 AM, Diego Biurrun <di...@biurrun.de> wrote: >> On Thu, Apr 21, 2011 at 10:56:07AM -0700, Aℓex Converse wrote: >>> On Thu, Apr 21, 2011 at 10:10 AM, Diego Biurrun <di...@biurrun.de> wrote: >>> > Do not call av_log_ask_for_sample() from av_log_missing_feature() >>> > depending on >>> > the value of a flag parameter. Instead, call av_log_ask_for_sample() >>> > directly >>> > where this is desired. This simplifies av_log_missing_feature() and makes >>> > for >>> > a nicer API. >>> > --- >>> > libavcodec/aac_adtstoasc_bsf.c | 5 +++-- >>> > libavcodec/aacdec.c | 20 +++++++++++++------- >>> > libavcodec/aacsbr.c | 5 +++-- >>> > libavcodec/ac3dec.c | 3 ++- >>> > libavcodec/alsdec.c | 2 +- >>> > libavcodec/amrnbdec.c | 3 ++- >>> > libavcodec/amrwbdec.c | 6 ++++-- >>> > libavcodec/avcodec.h | 6 +----- >>> > libavcodec/eac3dec.c | 12 ++++++++---- >>> > libavcodec/flacdec.c | 2 +- >>> > libavcodec/mpc8.c | 3 ++- >>> > libavcodec/utils.c | 11 +++-------- >>> > libavcodec/vc1.c | 2 +- >>> > libavcodec/vp8.c | 6 ++++-- >>> > libavcodec/wmavoice.c | 3 ++- >>> > libavformat/rtpdec_latm.c | 8 +++++--- >>> > libavformat/rtpdec_qt.c | 9 ++++++--- >>> > libavformat/spdifdec.c | 5 +++-- >>> > 18 files changed, 64 insertions(+), 47 deletions(-) >>> > >>> > --- a/libavcodec/avcodec.h >>> > +++ b/libavcodec/avcodec.h >>> > @@ -4097,12 +4097,8 @@ unsigned int av_xiphlacing(unsigned char *s, >>> > unsigned int v); >>> > * @param[in] avc a pointer to an arbitrary struct of which the first >>> > field is >>> > * a pointer to an AVClass struct >>> > * @param[in] feature string containing the name of the missing feature >>> > - * @param[in] want_sample indicates if samples are wanted which exhibit >>> > this feature. >>> > - * If want_sample is non-zero, additional verbage will be added to the >>> > log >>> > - * message which tells the user how to report samples to the development >>> > - * mailing list. >>> > */ >>> > -void av_log_missing_feature(void *avc, const char *feature, int >>> > want_sample); >>> > +void av_log_missing_feature(void *avc, const char *feature); >>> > >>> > /** >>> > * Log a generic warning message asking for a sample. This function is >>> >>> [...] >>> >>> This is a public header. >> >> You mean that this would require a major/minor bump? We're in the period >> fter the bump that is officially declared unstable API time, plus those >> functions are not supposed to be used outside of libav* as the Doxygen >> explains: >> >> /** >> * Logs a generic warning message about a missing feature. This function is >> * intended to be used internally by Libav (libavcodec, libavformat, etc.) >> * only, and would normally not be used by applications. >> >> I didn't make this up, don't shoot the messenger :) > > Since libavformat and libavcodec were bumped at the same time and the > API is declared unstable I suppose this is ok. For extra karma > consider moving it to a not-installed header.
It needs to remain av_*() prefixed since it's shared between avf/avc, though. Ronald _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel