[FFmpeg-devel] [PATCH] avformat/dashenc: Format VP9 level as decimal instead of hexadecimal
Jeyapal, Karthick
kjeyapal at akamai.com
Mon Sep 17 11:05:45 EEST 2018
On 9/17/18 1:12 PM, Hendrik Leppkes wrote:
> On Mon, Sep 17, 2018 at 9:28 AM Karthick J <kjeyapal at akamai.com> wrote:
>>
>> From: Karthick Jeyapal <kjeyapal at akamai.com>
>>
>> Commit ID 63c69d51c7532fb6c2460076329b50ec51a0f290 fixed the bug in vpcc, get_vp9_level() function, causing this change.
>> ---
>> libavformat/dashenc.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c
>> index 9a33321..f429ebc 100644
>> --- a/libavformat/dashenc.c
>> +++ b/libavformat/dashenc.c
>> @@ -211,7 +211,7 @@ static void set_vp9_codec_str(AVFormatContext *s, AVCodecParameters *par,
>> VPCC vpcc;
>> int ret = ff_isom_get_vpcc_features(s, par, frame_rate, &vpcc);
>> if (ret == 0) {
>> - av_strlcatf(str, size, "vp09.%02x.%02x.%02x",
>> + av_strlcatf(str, size, "vp09.%02x.%02d.%02x",
>> vpcc.profile, vpcc.level, vpcc.bitdepth);
>> } else {
>> // Default to just vp9 in case of error while finding out profile or level
>
> Shouldn't all of those be decimals, instead of hex? For profile its
> never going to matter since the values are all low, but bitdepth
> should be 10 instead of 0A.
You are right. I have sent another patch to fix it.
Thanks for pointing it out.
Regards,
Karthick
>
> - Hendrik
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
More information about the ffmpeg-devel
mailing list