Re: [FFmpeg-devel] [PATCH] avutil/frame: Assert that width/height/channels is 0 for the destination of av_frame*_ref()

[Gonzalo]

El 08/03/16 a las 17:14, Michael Niedermayer escribió:
diff --git a/libavutil/frame.c b/libavutil/frame.c
index 5607206..dde32b0 100644
--- a/libavutil/frame.c
+++ b/libavutil/frame.c
@@ -375,6 +375,9 @@ int av_frame_ref(AVFrame *dst, const AVFrame *src)
  {
      int i, ret = 0;
  
+    av_assert0(dst->width == 0 && dst->height == 0);
+    av_assert0(dst->channels == 0);
+
      dst->format         = src->format;
      dst->width          = src->width;
      dst->height         = src->height;
Why does it matter what dst->width/height is originally?  They are 
overwritten right after.

--
Gonzalo Garramuño
ggarr...@gmail.com

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel