Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-27 Thread Benoit Fouet
Hi, On 25/06/2016 14:15, Clément Bœsch wrote: On Fri, Jun 24, 2016 at 09:20:35AM +0200, Benoit Fouet wrote: [...] Any objection to this patch now? iam ok with the patch, maybe give others a bit of time to reply before applying though Yeah, I'm in no hurry, I just saw this FIXME in the context

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-25 Thread Clément Bœsch
On Fri, Jun 24, 2016 at 09:20:35AM +0200, Benoit Fouet wrote: [...] > >>Any objection to this patch now? > >iam ok with the patch, maybe give others a bit of time to reply > >before applying though > > Yeah, I'm in no hurry, I just saw this FIXME in the context of one of > Mateo's patches. > I was

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-24 Thread Benoit Fouet
Hi, On 23/06/2016 22:37, Michael Niedermayer wrote: On Thu, Jun 23, 2016 at 03:28:10PM +0200, Benoit Fouet wrote: Hi, On 21/06/2016 16:42, Benoit Fouet wrote: Hi, On 21/06/2016 16:29, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: Hi, On 21/06/2016 14:52, H

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-23 Thread Michael Niedermayer
On Thu, Jun 23, 2016 at 03:28:10PM +0200, Benoit Fouet wrote: > Hi, > > > On 21/06/2016 16:42, Benoit Fouet wrote: > >Hi, > > > >On 21/06/2016 16:29, Hendrik Leppkes wrote: > >>On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet > >> wrote: > >>>Hi, > >>> > >>> > >>>On 21/06/2016 14:52, Hendrik Leppkes

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-23 Thread Benoit Fouet
Hi, On 21/06/2016 16:42, Benoit Fouet wrote: Hi, On 21/06/2016 16:29, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: Hi, On 21/06/2016 14:52, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: On Tue, Jun 21, 2016 at 02:34:33PM +02

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, On 21/06/2016 16:29, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: Hi, On 21/06/2016 14:52, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: Hi, Unless I totally m

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Hendrik Leppkes
On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: > Hi, > > > On 21/06/2016 14:52, Hendrik Leppkes wrote: >> >> On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: >>> >>> On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: Hi, Unless I totally missed something,

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, On 21/06/2016 14:52, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: Hi, Unless I totally missed something, the FIXME in H264ParamSets structure should be fixed by attached patch. -- Ben From 28

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Hendrik Leppkes
On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: > On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: >> Hi, >> >> Unless I totally missed something, the FIXME in H264ParamSets structure >> should be fixed by attached patch. >> >> -- >> Ben >> > >> From 28ae10498f81070539bdb8f40236

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Clément Bœsch
On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: > Hi, > > Unless I totally missed something, the FIXME in H264ParamSets structure > should be fixed by attached patch. > > -- > Ben > > From 28ae10498f81070539bdb8f40236326743350101 Mon Sep 17 00:00:00 2001 > From: Benoit Fouet > D

[FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, Unless I totally missed something, the FIXME in H264ParamSets structure should be fixed by attached patch. -- Ben From 28ae10498f81070539bdb8f40236326743350101 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Tue, 21 Jun 2016 14:17:13 +0200 Subject: [PATCH] h264: make H264ParamSets sps