Bug#831529: libavcodec57: broken option parsing with LANGs with decimal mark different from .

[Sebastian Ramacher]

Control: tags -1 + patch

Hi Carl

On 2016-08-02 17:51:47, Carl Eugen Hoyos wrote:
> Please someone test attached patch, I cannot reproduce on any of my systems.

The patch fixes the issue here.

Cheers

> diff --git a/libavcodec/vaapi_encode_h264.c b/libavcodec/vaapi_encode_h264.c
> index 979cf37..5bed4e4 100644
> --- a/libavcodec/vaapi_encode_h264.c
> +++ b/libavcodec/vaapi_encode_h264.c
> @@ -1323,10 +1323,10 @@ static const AVCodecDefault vaapi_encode_h264_defaults[] = {
>      { "b",              "0"   },
>      { "bf",             "2"   },
>      { "g",              "120" },
> -    { "i_qfactor",      "1.0" },
> -    { "i_qoffset",      "0.0" },
> -    { "b_qfactor",      "1.2" },
> -    { "b_qoffset",      "0.0" },
> +    { "i_qfactor",      "1"   },
> +    { "i_qoffset",      "0"   },
> +    { "b_qfactor",      "6/5" },
> +    { "b_qoffset",      "0"   },
>      { NULL },
>  };
>  
> diff --git a/libavcodec/vaapi_encode_h265.c b/libavcodec/vaapi_encode_h265.c
> index 1ef968c..45f6f6d 100644
> --- a/libavcodec/vaapi_encode_h265.c
> +++ b/libavcodec/vaapi_encode_h265.c
> @@ -1341,10 +1341,10 @@ static const AVCodecDefault vaapi_encode_h265_defaults[] = {
>      { "b",              "0"   },
>      { "bf",             "2"   },
>      { "g",              "120" },
> -    { "i_qfactor",      "1.0" },
> -    { "i_qoffset",      "0.0" },
> -    { "b_qfactor",      "1.2" },
> -    { "b_qoffset",      "0.0" },
> +    { "i_qfactor",      "1"   },
> +    { "i_qoffset",      "0"   },
> +    { "b_qfactor",      "6/5" },
> +    { "b_qoffset",      "0"   },
>      { NULL },
>  };
>  


-- 
Sebastian Ramacher
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20160802/ab920cf2/attachment.sig>