[FFmpeg-devel] [libav-devel] [PATCH 2/2] lavf: move internal fields from public to internal context

wm4 nfxjfg at googlemail.com
Wed Mar 4 20:59:13 CET 2015


On Wed, 04 Mar 2015 20:21:26 +0100
Andreas Cadhalpun <andreas.cadhalpun at googlemail.com> wrote:

> Hi,
> 
> On 06.02.2015 14:53, wm4 wrote:
> > This is not an API change; the fields were explicitly declared private
> > before.
> 
> Unfortunately XBMC is using these semi-private fields, so it gets broken by this 
> change. Therefore I think it would be better to postpone this until after a 
> SOVERSION bump.
> 
> Looking a bit closer at the source [1], it seems XBMC uses this only to provide 
> a ff_read_frame_flush equivalent called xbmc_read_frame_flush.
> 
> To avoid having to do that in XBMC I suggest to rename ff_read_frame_flush back 
> to av_read_frame_flush and add it to the public API. Thoughts?
> 

I don't think this is a valid excuse. Tell XBMC to fix their code.

Or do I get to have "API exceptions" with my own project too?


More information about the ffmpeg-devel mailing list