[vlc-devel] [PATCH] packetizer: hevc: read the sample aspect ratio

"zhilizhao(赵志立)" quinkblack at foxmail.com
Wed Nov 4 11:41:51 CET 2020



> On Nov 4, 2020, at 6:31 PM, Francois Cartegnie <fcvlcdev at free.fr> wrote:
> 
> Le 04/11/2020 à 08:00, Steve Lhomme a écrit :
>> ---
>>  modules/packetizer/hevc.c | 7 +++++++
>>  1 file changed, 7 insertions(+)
>> diff --git a/modules/packetizer/hevc.c b/modules/packetizer/hevc.c
>> index 33765c38d20..1142bea847a 100644
>> --- a/modules/packetizer/hevc.c
>> +++ b/modules/packetizer/hevc.c
>> @@ -630,6 +630,13 @@ static void ActivateSets(decoder_t *p_dec,
>>              }
>>          }
>>  +        unsigned num, den;
>> +        if (hevc_get_aspect_ratio(p_sps, &num, &den))
>> +        {
>> +            p_dec->fmt_out.video.i_sar_num = num;
>> +            p_dec->fmt_out.video.i_sar_den = den;
>> +        }
>> +
> 
> This is overriding the demuxer values

H.264 packtizer override sar too. Isn’t it the point to use a packtizer? Or check
fmt sar before overriding?

> 
> -- 
> Francois Cartegnie
> VideoLAN - VLC Developer
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel <https://mailman.videolan.org/listinfo/vlc-devel>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20201104/f36f1d77/attachment.html>


More information about the vlc-devel mailing list