[FFmpeg-devel] [PATCH] #define LAVF_API_MAX_STREAMS
Michael Niedermayer
michaelni
Tue Aug 17 14:18:12 CEST 2010
On Wed, Aug 11, 2010 at 07:21:09PM +0200, Aurelien Jacobs wrote:
> Hi,
>
> We currently have quite a few new API (or API deprecation) in lavf/lavc
> which are waiting for a major version bump to be fully enabled. Some of
> those new API are fully functionnal and ready to be used, some of them
> are work in progress that don't even compile for now (eg. AVPalette
> removal).
> All of them are currently protected by some check like:
> #if LIBAVFORMAT_VERSION_MAJOR < 53
> So we now have quite of mess of MAJOR version check all around and it's
> not always easy to say which one match which API change. Moreover, it's
> not easy to enable just one of those API change to conduct some tests.
> Just bumping the major lead to a non-compilable source tree.
>
> So I propose to add one new define for each API transition we are
> working on. This would allow to trivially test each transition
> individually, and also to trivially re-schedule an (incomplete)
> transition to next major bump when bumping major.
>
> For example this allows testing the API transition with something like:
> ./configure --extra-cflags=-DLAVF_API_MAX_STREAMS=0
>
> Attached patch does this for the currently worked on MAX_STREAMS
> transition, and I plan to propose patches for other transitions if this
> one is accepted.
>
> Aurel
> avformat.h | 6 +++++-
> mpegts.c | 2 +-
> nutdec.c | 2 +-
> 3 files changed, 7 insertions(+), 3 deletions(-)
> e11c7306927589563fb0d12fc5b3aae2f54aac8c lavf_api_max_streams.diff
if people want that then ok
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 190 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100817/f915bdec/attachment.pgp>
More information about the ffmpeg-devel
mailing list