[FFmpeg-devel] [PATCH 6/9] ffprobe: use AVFrame.duration instead of AVFrame.pkt_duration

[James Almer]

On 7/14/2022 6:34 AM, Anton Khirnov wrote:
> Quoting James Almer (2022-07-13 14:39:20)
>> You should probably add new entries to the schema file instead.
>>
>> https://git.videolan.org/?p=ffmpeg.git;a=blob;f=doc/ffprobe.xsd;h=3af621a17ae884adfeacb7cd50c60e1553808188;hb=HEAD#l93
>>
>> Once frame->pkt_duration is gone, ffprobe shouldn't keep printing a
>> frame->duration value as "pkt_duration" and "pkt_duration_time".
>> Also, if frame->duration is supposed to be able to have values other
>> than those we wrote to pkt_duration, maybe keep printing the latter with
>> the existing schema entries until it's all gone (You can use
>> AV_NOWARN_DEPRECATED() to shut compilers up).
> 
> Well, the whole idea that values printed by ffprobe should precisely
> mirror the libav* API, up to the field names of our structs, seems quite
> questionable to me, IMO it makes more sense for it to provide a layer of
> abstraction over the libraries. But then again I neither maintain
> ffprobe, nor use it very much, so I wanted to keep changes to a minimum.
> If nobody has other plans for dealing with this then I can certainly
> make the changes you're sugesting.

ffprobe used to print pkt_pts until it was removed, so printing a field 
that does not exist (once it's removed) where the new values assigned to 
it have potentially slightly different semantics than the old ones does 
not seem like a good idea.