Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-17 Thread Michael Niedermayer
On Mon, Aug 17, 2015 at 11:30:15AM -0400, Ronald S. Bultje wrote: > Hi, > > On Mon, Aug 17, 2015 at 9:20 AM, Michael Niedermayer > wrote: > > > On Mon, Aug 17, 2015 at 07:48:07AM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Mon, Aug 17, 2015 at 6:40 AM, Michael Niedermayer > > > > >

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-17 Thread Hendrik Leppkes
On Mon, Aug 17, 2015 at 5:30 PM, Ronald S. Bultje wrote: > Hi, > > On Mon, Aug 17, 2015 at 9:20 AM, Michael Niedermayer > wrote: > >> On Mon, Aug 17, 2015 at 07:48:07AM -0400, Ronald S. Bultje wrote: >> > Hi, >> > >> > On Mon, Aug 17, 2015 at 6:40 AM, Michael Niedermayer >> > > > wrote: >> > >> >

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-17 Thread Ronald S. Bultje
Hi, On Mon, Aug 17, 2015 at 9:20 AM, Michael Niedermayer wrote: > On Mon, Aug 17, 2015 at 07:48:07AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Mon, Aug 17, 2015 at 6:40 AM, Michael Niedermayer > > > wrote: > > > > > On Sun, Aug 16, 2015 at 05:45:55PM -0400, Ronald S. Bultje wrote: > > >

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-17 Thread Michael Niedermayer
On Mon, Aug 17, 2015 at 09:28:10AM -0400, Ronald S. Bultje wrote: > Hi, > > On Mon, Aug 17, 2015 at 9:20 AM, Michael Niedermayer > wrote: > > > On Mon, Aug 17, 2015 at 07:48:07AM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Mon, Aug 17, 2015 at 6:40 AM, Michael Niedermayer > > > > >

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-17 Thread Ronald S. Bultje
Hi, On Mon, Aug 17, 2015 at 9:20 AM, Michael Niedermayer wrote: > On Mon, Aug 17, 2015 at 07:48:07AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Mon, Aug 17, 2015 at 6:40 AM, Michael Niedermayer > > > wrote: > > > > > On Sun, Aug 16, 2015 at 05:45:55PM -0400, Ronald S. Bultje wrote: > > >

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-17 Thread Michael Niedermayer
On Mon, Aug 17, 2015 at 07:48:07AM -0400, Ronald S. Bultje wrote: > Hi, > > On Mon, Aug 17, 2015 at 6:40 AM, Michael Niedermayer > wrote: > > > On Sun, Aug 16, 2015 at 05:45:55PM -0400, Ronald S. Bultje wrote: > > > Hi, > > > > > > On Sun, Aug 16, 2015 at 5:24 PM, Andreas Cadhalpun < > > > andre

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-17 Thread Ronald S. Bultje
Hi, On Mon, Aug 17, 2015 at 6:40 AM, Michael Niedermayer wrote: > On Sun, Aug 16, 2015 at 05:45:55PM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Sun, Aug 16, 2015 at 5:24 PM, Andreas Cadhalpun < > > andreas.cadhal...@googlemail.com> wrote: > > > > > On 16.08.2015 22:15, Ronald S. Bultje wr

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-17 Thread Michael Niedermayer
On Sun, Aug 16, 2015 at 05:45:55PM -0400, Ronald S. Bultje wrote: > Hi, > > On Sun, Aug 16, 2015 at 5:24 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: > > > On 16.08.2015 22:15, Ronald S. Bultje wrote: > > > Convert last users to av_opt_get_*() counterparts. > > > --- > > >

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-16 Thread Ronald S. Bultje
Hi, On Sun, Aug 16, 2015 at 5:24 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 16.08.2015 22:15, Ronald S. Bultje wrote: > > Convert last users to av_opt_get_*() counterparts. > > --- > > libavfilter/af_aresample.c | 17 + > > libavutil/opt.h|

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-16 Thread Andreas Cadhalpun
On 16.08.2015 22:15, Ronald S. Bultje wrote: > Convert last users to av_opt_get_*() counterparts. > --- > libavfilter/af_aresample.c | 17 + > libavutil/opt.h| 3 +++ > 2 files changed, 12 insertions(+), 8 deletions(-) I'm fine with this, but the patch is incomplete:

Re: [FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-16 Thread Michael Niedermayer
On Sun, Aug 16, 2015 at 04:15:47PM -0400, Ronald S. Bultje wrote: > Convert last users to av_opt_get_*() counterparts. > --- > libavfilter/af_aresample.c | 17 + > libavutil/opt.h| 3 +++ > 2 files changed, 12 insertions(+), 8 deletions(-) LGTM thanks [...] -- Mic

[FFmpeg-devel] [PATCH] options: mark av_get_{int, double, q} as deprecated.

2015-08-16 Thread Ronald S. Bultje
Convert last users to av_opt_get_*() counterparts. --- libavfilter/af_aresample.c | 17 + libavutil/opt.h| 3 +++ 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/libavfilter/af_aresample.c b/libavfilter/af_aresample.c index 6cb765d..f6fdd08 100644 --- a