[FFmpeg-devel] [PATCH] Make AVRtpPayloadTypes static and constant

[Luca Abeni]

Hi Diego,

On Sat, 2008-01-05 at 13:34 +0100, Diego 'Flameeyes' Petten? wrote:
> The attached patch changes the way AVRtpPayloadTypes is defined.
[...]

I'm all in favour of this change, but AVRtpPayloadTypes is unfortunately
declared as extern in rtp.h, which is an exported header... So, your
patch would break libavformat's API and ABI. At least, it needs an
increment in libavformat's version.

Maybe this is a good candidate for the usual "#if
LIBAVFORMAT_VERSION_INT > ..." stuff? What do other people think? I
believe that we should not export AVRtpPayloadTypes.


			Thanks,
				Luca