Re: [FFmpeg-devel] [PATCH] h264: Move AFD to side data to match MPEG-2

2014-08-22 Thread Christophe Gisquet
Hi, 2014-08-22 1:21 GMT+02:00 Christophe Gisquet : > Maybe all that side data stuff should be moved to the end of the > function first (but before the last thread-related line), and fixed, > then you also add this hunk there? Not a good thing to do either, after discussing with Michael. I'll send

Re: [FFmpeg-devel] [PATCH] h264: Move AFD to side data to match MPEG-2

2014-08-21 Thread Christophe Gisquet
Hi, 2014-08-09 2:45 GMT+02:00 Kieran Kunhya : > +AVFrameSideData *sd = > +av_frame_new_side_data(&cur->f, > + AV_FRAME_DATA_AFD, 1); > +if (!sd) > +return; You're not responsible for what has already been committed before,

Re: [FFmpeg-devel] [PATCH] h264: Move AFD to side data to match MPEG-2

2014-08-21 Thread Kieran Kunhya
On 9 August 2014 01:45, Kieran Kunhya wrote: > --- > libavcodec/h264.c | 11 +++ > libavcodec/h264.h |6 ++ > libavcodec/h264_sei.c |6 ++ > 3 files changed, 23 insertions(+) ping ___ ffmpeg-devel mailing list ffmpeg-d

[FFmpeg-devel] [PATCH] h264: Move AFD to side data to match MPEG-2

2014-08-08 Thread Kieran Kunhya
--- libavcodec/h264.c | 11 +++ libavcodec/h264.h |6 ++ libavcodec/h264_sei.c |6 ++ 3 files changed, 23 insertions(+) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 8fa35c7..e77b633 100644 --- a/libavcodec/h264.c +++ b/libavcodec/h264.c @@ -883,6 +883