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

2015-03-07 Thread Rainer Hochecker
Michael Niedermayer michaelni at gmx.at writes: ive already applied wm4s patch yesterday, so the next ffmpeg release (2.6) should contain avformat_flush() if further changes are needed please tell me [...] Thanks for notification. I will switch Kodi to 2.6 this weekend.

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

2015-03-05 Thread Bálint Réczey
Hi, 2015-03-05 0:02 GMT+01:00 Andreas Cadhalpun andreas.cadhal...@googlemail.com: On 04.03.2015 23:13, Luca Barbato wrote: On 04/03/15 21:49, Andreas Cadhalpun wrote: So what do you think about resurrecting this patch (add avformat_flush)? I see no patch about it, my email client chewed

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

2015-03-04 Thread Andreas Cadhalpun
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.

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

2015-03-04 Thread wm4
On Wed, 04 Mar 2015 20:21:26 +0100 Andreas Cadhalpun andreas.cadhal...@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

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

2015-03-04 Thread Michael Niedermayer
On Wed, Mar 04, 2015 at 09:49:17PM +0100, Andreas Cadhalpun wrote: On 04.03.2015 21:14, wm4 wrote: On Wed, 4 Mar 2015 21:05:04 +0100 Hendrik Leppkes h.lepp...@gmail.com wrote: On Wed, Mar 4, 2015 at 8:59 PM, wm4 nfx...@googlemail.com wrote: On Wed, 04 Mar 2015 20:21:26 +0100 Andreas

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

2015-03-04 Thread Derek Buitenhuis
On 3/4/2015 7:21 PM, Andreas Cadhalpun wrote: 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. Why exactly should we care about a project that uses explicitly private

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

2015-03-04 Thread Andreas Cadhalpun
On 04.03.2015 21:14, wm4 wrote: On Wed, 4 Mar 2015 21:05:04 +0100 Hendrik Leppkes h.lepp...@gmail.com wrote: On Wed, Mar 4, 2015 at 8:59 PM, wm4 nfx...@googlemail.com wrote: On Wed, 04 Mar 2015 20:21:26 +0100 Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Unfortunately XBMC is

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

2015-03-04 Thread Hendrik Leppkes
On Wed, Mar 4, 2015 at 8:59 PM, wm4 nfx...@googlemail.com wrote: On Wed, 04 Mar 2015 20:21:26 +0100 Andreas Cadhalpun andreas.cadhal...@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

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

2015-03-04 Thread wm4
On Wed, 4 Mar 2015 21:05:04 +0100 Hendrik Leppkes h.lepp...@gmail.com wrote: On Wed, Mar 4, 2015 at 8:59 PM, wm4 nfx...@googlemail.com wrote: On Wed, 04 Mar 2015 20:21:26 +0100 Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Hi, On 06.02.2015 14:53, wm4 wrote: This is

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

2015-03-04 Thread wm4
On Wed, 4 Mar 2015 22:33:57 +0100 Michael Niedermayer michae...@gmx.at wrote: On Wed, Mar 04, 2015 at 09:49:17PM +0100, Andreas Cadhalpun wrote: On 04.03.2015 21:14, wm4 wrote: On Wed, 4 Mar 2015 21:05:04 +0100 Hendrik Leppkes h.lepp...@gmail.com wrote: On Wed, Mar 4, 2015 at 8:59 PM,

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

2015-03-04 Thread wm4
On Wed, 04 Mar 2015 20:35:51 + Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 3/4/2015 7:21 PM, Andreas Cadhalpun wrote: 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

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

2015-03-04 Thread Andreas Cadhalpun
On 04.03.2015 23:13, Luca Barbato wrote: On 04/03/15 21:49, Andreas Cadhalpun wrote: So what do you think about resurrecting this patch (add avformat_flush)? I see no patch about it, my email client chewed it? It has been a while ago [1]. It just adds a public API for ff_read_frame_flush.