[FFmpeg-devel] [PATCH] AVFrame width/height are now useless except for internal purposes.
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Sun May 5 09:36:54 CEST 2013
At least document this change, which for VP6 with cropping triggered
a few sanity checks in MPlayer.
---
doc/APIchanges | 5 +++++
libavutil/frame.h | 4 +++-
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/doc/APIchanges b/doc/APIchanges
index 763ccf0..a6d3b72 100644
--- a/doc/APIchanges
+++ b/doc/APIchanges
@@ -15,6 +15,11 @@ libavutil: 2012-10-22
API changes, most recent first:
+2013-??-?? - xxxxxxx - lavu ??.?.???
+ AVFrame width and height no longer match AVCodecContext width and height.
+ They contain now purely internal values that have no relevance to users,
+ you should not use them for any purpose.
+
2013-04-19 - xxxxxxx - lavc 55.4.100
Add AV_CODEC_PROP_TEXT_SUB property for text based subtitles codec.
diff --git a/libavutil/frame.h b/libavutil/frame.h
index 39a664f..8b1112c 100644
--- a/libavutil/frame.h
+++ b/libavutil/frame.h
@@ -117,7 +117,9 @@ typedef struct AVFrame {
uint8_t **extended_data;
/**
- * width and height of the video frame
+ * allocated width and height of the video frame
+ * these have nothing to do with the actual visible size of the
+ * frame data, that information is not available
*/
int width, height;
--
1.7.10.4
More information about the ffmpeg-devel
mailing list