Browse Source

vaapi_encode_mjpeg: Use common quality option

Doesn't change anything, but makes the behaviour better match that of the
other codecs (the CONSTANT_QUALITY_ONLY flag already ensures that CQP is
the only RC mode selectable for MJPEG).
tags/n4.2
Mark Thompson 7 years ago
parent
commit
fda6dcd0b0
1 changed files with 3 additions and 2 deletions
  1. +3
    -2
      libavcodec/vaapi_encode_mjpeg.c

+ 3
- 2
libavcodec/vaapi_encode_mjpeg.c View File

@@ -438,7 +438,7 @@ static av_cold int vaapi_encode_mjpeg_configure(AVCodecContext *avctx)
VAAPIEncodeMJPEGContext *priv = avctx->priv_data;
int err;

priv->quality = avctx->global_quality;
priv->quality = ctx->rc_quality;
if (priv->quality < 1 || priv->quality > 100) {
av_log(avctx, AV_LOG_ERROR, "Invalid quality value %d "
"(must be 1-100).\n", priv->quality);
@@ -483,6 +483,8 @@ static const VAAPIEncodeType vaapi_encode_type_mjpeg = {

.configure = &vaapi_encode_mjpeg_configure,

.default_quality = 80,

.picture_params_size = sizeof(VAEncPictureParameterBufferJPEG),
.init_picture_params = &vaapi_encode_mjpeg_init_picture_params,

@@ -537,7 +539,6 @@ static const AVOption vaapi_encode_mjpeg_options[] = {
};

static const AVCodecDefault vaapi_encode_mjpeg_defaults[] = {
{ "global_quality", "80" },
{ "b", "0" },
{ NULL },
};


Loading…
Cancel
Save