[SCM] libav/experimental: dvenc: initialize the profile only once, at init
siretart at users.alioth.debian.org
siretart at users.alioth.debian.org
Sun Aug 10 16:03:32 UTC 2014
The following commit has been merged in the experimental branch:
commit 778111592bf5f38630858ee6dfcfd097cd6c6da9
Author: Anton Khirnov <anton at khirnov.net>
Date: Wed Jun 18 10:39:13 2014 +0200
dvenc: initialize the profile only once, at init
We do not allow the frame parameters to change during encoding, so there
is no need to do this for each frame.
diff --git a/libavcodec/dvenc.c b/libavcodec/dvenc.c
index 6338dee..552f691 100644
--- a/libavcodec/dvenc.c
+++ b/libavcodec/dvenc.c
@@ -35,13 +35,22 @@
static av_cold int dvvideo_encode_init(AVCodecContext *avctx)
{
- if (!avpriv_dv_codec_profile(avctx)) {
+ DVVideoContext *s = avctx->priv_data;
+ int ret;
+
+ s->sys = avpriv_dv_codec_profile(avctx);
+ if (!s->sys) {
av_log(avctx, AV_LOG_ERROR, "Found no DV profile for %ix%i %s video. "
"Valid DV profiles are:\n",
avctx->width, avctx->height, av_get_pix_fmt_name(avctx->pix_fmt));
ff_dv_print_profiles(avctx, AV_LOG_ERROR);
return AVERROR(EINVAL);
}
+ ret = ff_dv_init_dynamic_tables(s->sys);
+ if (ret < 0) {
+ av_log(avctx, AV_LOG_ERROR, "Error initializing work tables.\n");
+ return ret;
+ }
avctx->coded_frame = av_frame_alloc();
if (!avctx->coded_frame)
@@ -660,9 +669,6 @@ static int dvvideo_encode_frame(AVCodecContext *c, AVPacket *pkt,
DVVideoContext *s = c->priv_data;
int ret;
- s->sys = avpriv_dv_codec_profile(c);
- if (!s->sys || ff_dv_init_dynamic_tables(s->sys))
- return -1;
if ((ret = ff_alloc_packet(pkt, s->sys->frame_size)) < 0) {
av_log(c, AV_LOG_ERROR, "Error getting output packet.\n");
return ret;
--
Libav/FFmpeg packaging
More information about the pkg-multimedia-commits
mailing list