[FFmpeg-devel] [PATCH] write AC-3 metadata
Wed Dec 22 19:26:13 CET 2010
On 12/21/2010 08:25 PM, Justin Ruggles wrote:
> On 12/21/2010 06:13 PM, Justin Ruggles wrote:
>> On 12/21/2010 05:20 PM, Kieran Kunhya wrote:
>>>> Date: Tuesday, 21 December, 2010, 21:56
>>>> This patch adds writing of AC-3 metadata to the AC-3
>>>> I chose to use AVOption instead of trying to move the
>>>> metadata API to
>>>> libavcodec because the AC-3 metadata isn't just regular
>>>> metadata. Even
>>>> though it is not audio data, many of the fields actually
>>>> affect decoding.
>>>> Some of the options use integers to set a specific code
>>>> value rather
>>>> than using a float value directly (e.g. mixing
>>>> levels). It looks a bit
>>>> ugly, but I chose to do this because there are only a few
>>>> valid values.
>>>> Trying to determine which value the user wants by
>>>> comparing floats
>>>> seems more complicated. Plus this allows the help
>>>> text to tell the user
>>>> what the valid values are for those options.
>>> Is there any possibility of making these metadata options changeable per frame? Quite useful for broadcast applications when the metadata needs to be different during advertising or during specific programming.
>> If you're changing programs, you should probably re-initialize encoding
>> anyway. Other things can change too like bitrate, number of channels,
>> As for setting metadata per-frame, it is possible. New patch attached.
>> It adds an option to enable per-frame validation of metadata so that it
>> won't significantly affect performance when the option is not enabled.
> oops. disregard this patch. i forgot the most important part. i'll send
> a new one in a little while.
New patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 19582 bytes
Desc: not available
More information about the ffmpeg-devel