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 christophe.gisq...@gmail.com: 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

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 kier...@obe.tv: +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

[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