[FFmpeg-devel] [PATCH]lavf/mov: Export vendor metadata

[Carl Eugen Hoyos]

2017-01-27 10:42 GMT+01:00 wm4 <nfxjfg at googlemail.com>:
> On Fri, 27 Jan 2017 10:38:23 +0100
> Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>
>> 2017-01-27 10:29 GMT+01:00 wm4 <nfxjfg at googlemail.com>:
>> > On Fri, 27 Jan 2017 10:19:32 +0100
>> > Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>> >
>> >> 2017-01-27 10:09 GMT+01:00 wm4 <nfxjfg at googlemail.com>:
>> >> > On Fri, 27 Jan 2017 09:39:03 +0100
>> >> > Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>> >> >
>> >> >> 2017-01-27 9:17 GMT+01:00 wm4 <nfxjfg at googlemail.com>:
>> >> >>
>> >> >> > You're completely misunderstanding.
>> >> >>
>> >> >> Would you mind to elaborate?
>> >> >
>> >> > FFmpeg shouldn't mux codec-specific tags into a different
>> >> > container.
>> >>
>> >> This is not codec-specific, at least not in the sense that
>> >> would make a difference for other containers.
>> >>
>> >> > Your ffmpeg.c patch works for transcoding only, not remuxing.
>> >>
>> >> That's because it makes sense to remux "vendor" metadata.
>> >
>> > No, technical metadata that is "hidden" is different from user-visible
>> > tags that contain non-technical information about the media.
>>
>> This is non-technical information that should be user-visible.
>
> The vendor tag in your fate change has the value "appl". How should
> that be user-visible?

You mean the information is encoded in such a difficult manner
that no user will be able to decipher it?
I really wish you were a little more serious when trying to slow down
FFmpeg development.

> You can't seriously tell me that this should show
> up in a non-technical UI view.

Since you are blocking much more important patches, I should
probably not spend so much time discussing this one. But the
fact that I needed this patch for debugging and that I found  out
afterwards that other developers need it too sufficiently indicates
to me showing, exporting and remuxing the vendor tag makes sense.

Carl Eugen