[FFmpeg-devel] [PATCH] FFV1 specification: Add more details about the configuration record

[Michael Niedermayer]

On Wed, May 13, 2015 at 06:47:11PM +0200, Jerome Martinez wrote:
[...]

> @@ -2661,7 +2728,7 @@ if( keyframe ) {
>  \begin_inset space ~
>  \end_inset
>  
> -if( version < 2 )

[...]
> +key "AVI"
> +
> +\end_inset
> +
> + for more information about chunks.
> +\end_layout
> +
> +\begin_layout Description
> +NumBytes is defined as the size, in bytes, of the strf chunk indicated in
> + the chunk header minus the size of the stream format structure.
> +\end_layout
> +
> +\begin_layout Subsubsection
> +In ISO/IEC 14496-12 (MP4 File Format)
> +\end_layout
> +
> +\begin_layout Standard
> +The Configuration Record extends the sample description box (

Does the text somewhere say why just avi and mp4 are listed as
containers ? (i didnt spot that but i might have missed it)

It should make it clear that these are not the only containers
supported but that nearly any container can be used

[...]
> @@ -2728,7 +3056,7 @@ if( version < 2 )
>  \begin_inset space ~
>  \end_inset
>  
> -FrameHeader01( )
> +if( keyframe && !ConfigurationRecordIsPresent)

is it better to add indirection here instead of spelling out that
its version < 2 ?




-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150513/8d656938/attachment.asc>