[FFmpeg-devel] [PATCH 5/5] doc/bitstream_filters: Add av1_metadata

[James Almer]

On 10/16/2018 6:15 PM, Mark Thompson wrote:
> On 05/10/18 00:09, Mark Thompson wrote:
>> ---
>>  doc/bitstream_filters.texi | 50 ++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 50 insertions(+)
>>
>> diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi
>> index d948c6d658..66df24fd3d 100644
>> --- a/doc/bitstream_filters.texi
>> +++ b/doc/bitstream_filters.texi
>> @@ -37,6 +37,56 @@ raw ADTS AAC or an MPEG-TS container to MP4A-LATM, to an FLV file, or
>>  to MOV/MP4 files and related formats such as 3GP or M4A. Please note
>>  that it is auto-inserted for MP4A-LATM and MOV/MP4 and related formats.
>>  
>> + at section av1_metadata
>> +
>> +Modify metadata embedded in an AV1 stream.
>> +
>> + at table @option
>> + at item td
>> +Insert or remove temporal delimiter OBUs in all temporal units of the
>> +stream.
>> +
>> + at table @samp
>> + at item insert

Maybe mention that it inserts one if there's none already at the
beginning of the TU, and does nothing otherwise. Same with remove below.

>> + at item remove
>> + at end table
>> +
>> + at item color_primaries
>> + at item transfer_characteristics
>> + at item matrix_coefficients
>> +Set the color description fields in the stream (see AV1 section 6.4.2).
>> +
>> + at item color_range
>> +Set the color range in the stream (see AV1 section 6.4.2; note that
>> +this cannot be set for streams using BT.709 primaries, sRGB transfer
>> +characteristic and identity (RGB) matrix coefficients).
>> + at table @samp
>> + at item tv
>> +Limited range.
>> + at item pc
>> +Full range.
>> + at end table
>> +
>> + at item chroma_sample_position
>> +Set the chroma sample location in the stream (see AV1 section 6.4.2).
>> +This can only be set for 4:2:0 streams.
>> +
>> + at table @samp
>> + at item vertical
>> +Left position (matching the default in MPEG-2 and H.264).
>> + at item colocated
>> +Top-left position.
>> + at end table
>> +
>> + at item tick_rate
>> +Set the tick rate (@emph{num_units_in_display_tick / time_scale}) in
>> +the timing info in the sequence header.
>> + at item num_ticks_per_picture
>> +Set the number of ticks in each picture, to indicate that the stream
>> +has a fixed framerate.  Ignored if @option{tick_rate} is not also set.
>> +
>> + at end table
>> +
>>  @section chomp
>>  
>>  Remove zero padding at the end of a packet.
>>
> 
> Ping.
> 
> Thanks,
> 
> - Mark

LGTM with or without the above change.