[FFmpeg-devel] [PATCH] */version.h: Add note/recommandition about bumping major
andreas.cadhalpun at googlemail.com
Tue Aug 18 23:52:39 CEST 2015
On 18.08.2015 12:28, Michael Niedermayer wrote:
> From: Michael Niedermayer <michael at niedermayer.cc>
> If preferred, i can also apply this after the bump, in case its felt that
> this would cause too much delay/work
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> libavcodec/version.h | 4 ++++
> libavformat/version.h | 5 +++++
> libavutil/version.h | 4 ++++
> 3 files changed, 13 insertions(+)
> diff --git a/libavcodec/version.h b/libavcodec/version.h
> index 1b37a9e..cf9c924 100644
> --- a/libavcodec/version.h
> +++ b/libavcodec/version.h
> @@ -46,6 +46,10 @@
> * FF_API_* defines may be placed below to indicate public API that will be
> * dropped at a future version bump. The defines themselves are not part of
> * the public API and may change, break or disappear at any time.
> + *
> + * @note, when bumping the major version it is recommandeded to manually
> + * disable each FF_API_* in its own commit instead of disabling them all
> + * at once through the bump. This improves the git bissect-ability of the change.
I think that is a good idea, but instead of disabling the FF_API_* defines
they should be removed together with the code guarded by them.
Otherwise chances are that the old, disabled code will never get removed,
see e.g. FF_API_OLD_GRAPH_PARSE.
More information about the ffmpeg-devel