[FFmpeg-devel] [PATCH] ffmpeg_opt: Allow program metadata other than title to be set.
michael at niedermayer.cc
Sun Jan 24 21:47:08 CET 2016
On Sun, Jan 24, 2016 at 04:03:30PM +0100, Vittorio Gambaletta (VittGam) wrote:
> On 24/01/2016 12:33:51 CET, Michael Niedermayer wrote:
> >On Sun, Jan 24, 2016 at 05:23:22AM +0100, Vittorio Gambaletta (VittGam) wrote:
> >>This commit makes it possible to add optional metadata (such as "service_provider") to an mpegts output stream composed by multiple programs (mpts).
> >>This is needed because the global metadata is not inherited by the single programs.
> >>Signed-off-by: Vittorio Gambaletta <ffmpeg-dev at vittgam.net>
> >> ffmpeg_opt.c | 10 +++-------
> >> 1 file changed, 3 insertions(+), 7 deletions(-)
> >with this change the whole namespace except "st"/"program_num" would
> >be used for metadata. This might be a problem in the future if we
> >want to add some non metadata key/value
> What would you suggest then? I have two ideas:
> 1. Move the 'process manually set programs' block above 'process
> manually set metadata' and have it handle `-metadata:p:<program_id>`
> 2. Use a prefix: `-program st=0:st=1:metadata=service_provider=HelloWorld`
> 3. Change mpegtsenc to look for global metadata if program metadata
> is not present
> The first is probably cleaner.
i have no real preferrance, depends mostly on the people what syntax
and variant is preferred
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel