Anton Khirnov <[email protected]> writes: > There is no real reason for it to be 64bit, it's just a plain int in the > rest of Libav. > --- > The first version didn't work > --- > libavfilter/avfilter.h | 6 +++++- > libavfilter/version.h | 3 +++ > 2 files changed, 8 insertions(+), 1 deletion(-) > > diff --git a/libavfilter/avfilter.h b/libavfilter/avfilter.h > index 357ce34..df1c4f2 100644 > --- a/libavfilter/avfilter.h > +++ b/libavfilter/avfilter.h > @@ -596,7 +596,11 @@ struct AVFilterLink { > AVRational sample_aspect_ratio; ///< agreed upon sample aspect ratio > /* These two parameters apply only to audio */ > uint64_t channel_layout; ///< channel layout of current buffer (see > libavutil/audioconvert.h) > - int64_t sample_rate; ///< samples per second > +#if FF_API_SAMPLERATE64 > + int64_t sample_rate; ///< samples per second > +#else > + int sample_rate; ///< samples per second > +#endif
The comments are now misaligned. > int format; ///< agreed upon media format > > diff --git a/libavfilter/version.h b/libavfilter/version.h > index 718ed78..ae172be 100644 > --- a/libavfilter/version.h > +++ b/libavfilter/version.h > @@ -47,5 +47,8 @@ > #ifndef FF_API_GRAPH_AVCLASS > #define FF_API_GRAPH_AVCLASS (LIBAVFILTER_VERSION_MAJOR > 2) > #endif > +#ifndef FF_API_SAMPLERATE64 > +#define FF_API_SAMPLERATE64 (LIBAVFILTER_VERSION_MAJOR < 3) > +#endif Inconsistent whitespace. LGTM otherwise. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
