Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-06 Thread Michael Niedermayer
On Sat, Dec 06, 2014 at 05:52:21PM +0100, Lukasz Marek wrote: > On 06.12.2014 15:59, Michael Niedermayer wrote: > >On Fri, Dec 05, 2014 at 12:21:21AM +0100, Michael Niedermayer wrote: > >>On Thu, Dec 04, 2014 at 12:50:49AM +0100, Lukasz Marek wrote: > >>[...] > >> > >>>index 49c8b40..4148752 100644

Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-06 Thread Lukasz Marek
On 06.12.2014 15:59, Michael Niedermayer wrote: On Fri, Dec 05, 2014 at 12:21:21AM +0100, Michael Niedermayer wrote: On Thu, Dec 04, 2014 at 12:50:49AM +0100, Lukasz Marek wrote: [...] index 49c8b40..4148752 100644 --- a/libavcodec/options.c +++ b/libavcodec/options.c @@ -303,3 +303,176 @@ con

Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-06 Thread Michael Niedermayer
On Fri, Dec 05, 2014 at 12:21:21AM +0100, Michael Niedermayer wrote: > On Thu, Dec 04, 2014 at 12:50:49AM +0100, Lukasz Marek wrote: > [...] > > > index 49c8b40..4148752 100644 > > --- a/libavcodec/options.c > > +++ b/libavcodec/options.c > > @@ -303,3 +303,176 @@ const AVClass *avcodec_get_subtit

Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-04 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 12:50:49AM +0100, Lukasz Marek wrote: [...] > index 49c8b40..4148752 100644 > --- a/libavcodec/options.c > +++ b/libavcodec/options.c > @@ -303,3 +303,176 @@ const AVClass *avcodec_get_subtitle_rect_class(void) > { > return &av_subtitle_rect_class; > } > + > +#define

Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-03 Thread Lukasz Marek
On 04.12.2014 00:50, Lukasz Marek wrote: This test doesn't cover every possible issue with this function. It covers options management only. Signed-off-by: Lukasz Marek --- libavcodec/Makefile | 2 + libavcodec/options.c | 173 +++ 2 files

[FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-03 Thread Lukasz Marek
This test doesn't cover every possible issue with this function. It covers options management only. Signed-off-by: Lukasz Marek --- libavcodec/Makefile | 2 + libavcodec/options.c | 173 +++ 2 files changed, 175 insertions(+) diff --git a/libav