[FFmpeg-devel] [PATCH 4/4] concatdec: add support for injecting packet metadata

Marton Balint cus at passwd.hu
Thu Jul 9 21:03:26 CEST 2015


On Thu, 9 Jul 2015, Nicolas George wrote:

> Le decadi 20 messidor, an CCXXIII, Marton Balint a écrit :
>> I'd still use the metadata directive. If someone needs per-stream, or
>> per-packet metadata, he can implement selector directives for that. Like
>> metadata_stream or metadata_interval which will change the scope of the
>> following metadata directives.
>>
>> How about that?
>
> I am still uneasy about that. Selectors feel like making the parser more
> complex, and also there is the issue of self-documentation.
>
> When someone reads a concat script and see "metadata foo=bar", unless they
> already know exactly the directive, they will take a guess at what it does,
> and since packet metadata is rare, especially packet metadata that is
> duplicated over all the packets, they will probably not guess right.
>
> I really think a more specific name would be useful.

What about "file_packet_metadata"?

It clearly states that metadata is set on a file level, and it belongs to 
packets.

>
> This is an usability issue, though, so my role as maintainer is not very
> important: let us ear advice from other people.

Sure, anybody? :)

Thanks,
Marton


More information about the ffmpeg-devel mailing list