[FFmpeg-cvslog] Revert "lavc/utils: Do not require dimensions for PNG."

Derek Buitenhuis git at videolan.org
Sun Aug 10 16:58:55 CEST 2014


ffmpeg | branch: master | Derek Buitenhuis <derek.buitenhuis at gmail.com> | Sun Jul 13 19:32:56 2014 +0100| [43654a22c037a86c23e7aed90660c5ea3971e56d] | committer: Derek Buitenhuis

Revert "lavc/utils: Do not require dimensions for PNG."

mplayer-specifc hacks should not be in our codebase. mplayer should fix
its own code. It is not our responsibility to work around their broken
code.

This reverts commit e8e575633faf19711910cf9caf59f7db300a9ccd.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=43654a22c037a86c23e7aed90660c5ea3971e56d
---

 libavcodec/utils.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index 0bc0b38..62d4b26 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -1531,9 +1531,7 @@ int attribute_align_arg avcodec_open2(AVCodecContext *avctx, const AVCodec *code
         } else if (avctx->channel_layout) {
             avctx->channels = av_get_channel_layout_nb_channels(avctx->channel_layout);
         }
-        if(avctx->codec_type == AVMEDIA_TYPE_VIDEO &&
-           avctx->codec_id != AV_CODEC_ID_PNG // For mplayer
-        ) {
+        if(avctx->codec_type == AVMEDIA_TYPE_VIDEO) {
             if (avctx->width <= 0 || avctx->height <= 0) {
                 av_log(avctx, AV_LOG_ERROR, "dimensions not set\n");
                 ret = AVERROR(EINVAL);



More information about the ffmpeg-cvslog mailing list