[FFmpeg-devel] [PATCH v2 8/8] doc/encoders: Fix libkvazaar documentation
Arttu Ylä-Outinen
arttu.yla-outinen at tut.fi
Tue Sep 29 15:29:15 CEST 2015
The -threads option is ignored with libkvazaar since it does not have
any of the AV_CODEC_CAP_{FRAME,SLICE,AUTO}_THREADS capabilities. This
commit removes the incorrect documentation as well as the no-op of
setting the number of threads in libkvazaar encoder.
Signed-off-by: Arttu Ylä-Outinen <arttu.yla-outinen at tut.fi>
---
v2: Rewrite the commit message to communicate why the parameter did not
work and that the patch also touches libkvazaar.c.
---
doc/encoders.texi | 3 ---
libavcodec/libkvazaar.c | 1 -
2 files changed, 4 deletions(-)
diff --git a/doc/encoders.texi b/doc/encoders.texi
index 3550bcc..bf364fd 100644
--- a/doc/encoders.texi
+++ b/doc/encoders.texi
@@ -2397,9 +2397,6 @@ configuration. You need to explicitly configure the build with
@item b
Set target video bitrate in bit/s and enable rate control.
- at item threads
-Set number of encoding threads.
-
@item kvazaar-params
Set kvazaar parameters as a list of @var{name}=@var{value} pairs separated
by commas (,). See kvazaar documentation for a list of options.
diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.c
index 0cf890f..3000f6a 100644
--- a/libavcodec/libkvazaar.c
+++ b/libavcodec/libkvazaar.c
@@ -74,7 +74,6 @@ static av_cold int libkvazaar_init(AVCodecContext *avctx)
cfg->height = avctx->height;
cfg->framerate =
avctx->time_base.den / (double)(avctx->time_base.num * avctx->ticks_per_frame);
- cfg->threads = avctx->thread_count;
cfg->target_bitrate = avctx->bit_rate;
cfg->vui.sar_width = avctx->sample_aspect_ratio.num;
cfg->vui.sar_height = avctx->sample_aspect_ratio.den;
--
1.7.9.5
More information about the ffmpeg-devel
mailing list