[FFmpeg-devel] [PATCH] make packet_size in AVFormatContext unsigned

Ronald S. Bultje rsbultje
Mon Jun 22 18:57:04 CEST 2009


On Mon, Jun 22, 2009 at 1:38 AM, Reinhard Tartler <siretart at tauware.de>wrote:

> This change can only break applications that check if packet_size in
> AVFormatContext is negative. If you expect that to cause problems in the
> wild, you'd need to bump LIBAVFORMAT_VERSION_MAJOR. Do you think it is
> worth forcing all applications to be recompiled against this new
> libavformat? TBH, I doubt that.
> For LIBAVFORMAT_VERSION_MINOR, I see no reason why applications built
> against the current libavformat would break when being used with this
> new libavformat.

You're entirely correct, I just figured how this is defined in commonly
large projects (GNOME, Apache): MAJOR if you break backwards compatibility,
i.e. if an app compiled with the old lib won't work with the new lib, and
MINOR if you break forward compatibility, i.e. if an app using new API
compiled with the new lib won't work with the old lib. Neither applies here,
so I should update MICRO: API change which affects neither backward nor
forward compatibility.

Thanks for the hint & patch applied,

More information about the ffmpeg-devel mailing list