Re: [FFmpeg-devel] [PATCH] avutil/frame: fix av_frame_copy for unknown layouts

2017-01-30 Thread Marton Balint
On Mon, 30 Jan 2017, Paul B Mahol wrote: On 1/30/17, wm4 wrote: On Mon, 30 Jan 2017 08:47:49 +0100 Hendrik Leppkes wrote: On Mon, Jan 30, 2017 at 8:40 AM, wm4 wrote: > On Mon, 30 Jan 2017 01:37:02 +0100 > Marton Balint wrote: > >> I wonder how unknown layouts ever worked without this? >>

Re: [FFmpeg-devel] [PATCH] avutil/frame: fix av_frame_copy for unknown layouts

2017-01-30 Thread Paul B Mahol
On 1/30/17, wm4 wrote: > On Mon, 30 Jan 2017 08:47:49 +0100 > Hendrik Leppkes wrote: > >> On Mon, Jan 30, 2017 at 8:40 AM, wm4 wrote: >> > On Mon, 30 Jan 2017 01:37:02 +0100 >> > Marton Balint wrote: >> > >> >> I wonder how unknown layouts ever worked without this? >> >> >> >> Signed-off-by: Ma

Re: [FFmpeg-devel] [PATCH] avutil/frame: fix av_frame_copy for unknown layouts

2017-01-30 Thread wm4
On Mon, 30 Jan 2017 08:47:49 +0100 Hendrik Leppkes wrote: > On Mon, Jan 30, 2017 at 8:40 AM, wm4 wrote: > > On Mon, 30 Jan 2017 01:37:02 +0100 > > Marton Balint wrote: > > > >> I wonder how unknown layouts ever worked without this? > >> > >> Signed-off-by: Marton Balint > >> --- > >> libavu

Re: [FFmpeg-devel] [PATCH] avutil/frame: fix av_frame_copy for unknown layouts

2017-01-29 Thread Hendrik Leppkes
On Mon, Jan 30, 2017 at 8:40 AM, wm4 wrote: > On Mon, 30 Jan 2017 01:37:02 +0100 > Marton Balint wrote: > >> I wonder how unknown layouts ever worked without this? >> >> Signed-off-by: Marton Balint >> --- >> libavutil/frame.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff

Re: [FFmpeg-devel] [PATCH] avutil/frame: fix av_frame_copy for unknown layouts

2017-01-29 Thread wm4
On Mon, 30 Jan 2017 01:37:02 +0100 Marton Balint wrote: > I wonder how unknown layouts ever worked without this? > > Signed-off-by: Marton Balint > --- > libavutil/frame.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/frame.c b/libavutil/frame.c > index c2

Re: [FFmpeg-devel] [PATCH] avutil/frame: fix av_frame_copy for unknown layouts

2017-01-29 Thread Hendrik Leppkes
On Mon, Jan 30, 2017 at 1:37 AM, Marton Balint wrote: > I wonder how unknown layouts ever worked without this? > > Signed-off-by: Marton Balint > --- > libavutil/frame.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/frame.c b/libavutil/frame.c > index c2f5509

Re: [FFmpeg-devel] [PATCH] avutil/frame: fix av_frame_copy for unknown layouts

2017-01-29 Thread Nicolas George
Le primidi 11 pluviôse, an CCXXV, Marton Balint a écrit : > I wonder how unknown layouts ever worked without this? > > Signed-off-by: Marton Balint > --- > libavutil/frame.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM, but I do not maintain that file. Regards, -- Nicolas G

[FFmpeg-devel] [PATCH] avutil/frame: fix av_frame_copy for unknown layouts

2017-01-29 Thread Marton Balint
I wonder how unknown layouts ever worked without this? Signed-off-by: Marton Balint --- libavutil/frame.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/frame.c b/libavutil/frame.c index c2f5509..a08e0c5 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c @@ -72