Re: [FFmpeg-devel] [PATCH 2/3] avformat/mov: skip hoov box if strict >= normal

2021-12-25 Thread zhilizhao(赵志立)
> On Dec 25, 2021, at 3:08 AM, Derek Buitenhuis > wrote: > > On 12/24/2021 10:46 AM, Steven Liu wrote: >> I think about ignore hoov should not default, because it use hoov >> default before. > > I agree - we (Vimeo) get a ton of hoov files with no moov or broken moov - > much more than files

Re: [FFmpeg-devel] [PATCH 2/3] avformat/mov: skip hoov box if strict >= normal

2021-12-24 Thread Derek Buitenhuis
On 12/24/2021 10:46 AM, Steven Liu wrote: > I think about ignore hoov should not default, because it use hoov > default before. I agree - we (Vimeo) get a ton of hoov files with no moov or broken moov - much more than files with hoov and valid moov (I've never seen any). This would be quite a brea

Re: [FFmpeg-devel] [PATCH 2/3] avformat/mov: skip hoov box if strict >= normal

2021-12-24 Thread Steven Liu
"zhilizhao(赵志立)" 于2021年12月24日周五 18:38写道: > > > > > On Dec 24, 2021, at 6:19 PM, Steven Liu wrote: > > > >> 2021年12月24日 下午5:58,Zhao Zhili 写道: > >> > >> The samples I have got have hoov and moov both. Unknown boxes > >> should be skipped according to the spec. So don't treat hoov as > >> moov in n

Re: [FFmpeg-devel] [PATCH 2/3] avformat/mov: skip hoov box if strict >= normal

2021-12-24 Thread zhilizhao(赵志立)
> On Dec 24, 2021, at 6:19 PM, Steven Liu wrote: > >> 2021年12月24日 下午5:58,Zhao Zhili 写道: >> >> The samples I have got have hoov and moov both. Unknown boxes >> should be skipped according to the spec. So don't treat hoov as >> moov in normal mode. >> >> For backward compatible, a log message

Re: [FFmpeg-devel] [PATCH 2/3] avformat/mov: skip hoov box if strict >= normal

2021-12-24 Thread Steven Liu
> 2021年12月24日 下午5:58,Zhao Zhili 写道: > > The samples I have got have hoov and moov both. Unknown boxes > should be skipped according to the spec. So don't treat hoov as > moov in normal mode. > > For backward compatible, a log message has been added to notice > the user to relax the striction i

[FFmpeg-devel] [PATCH 2/3] avformat/mov: skip hoov box if strict >= normal

2021-12-24 Thread Zhao Zhili
The samples I have got have hoov and moov both. Unknown boxes should be skipped according to the spec. So don't treat hoov as moov in normal mode. For backward compatible, a log message has been added to notice the user to relax the striction if moov doesn't exist. Fix #8883. --- libavformat/mov