[FFmpeg-devel] [PATCH 3/3] add digital cinema frame sizes

Stefano Sabatini stefasab at gmail.com
Sun Feb 3 23:07:34 CET 2013


On date Sunday 2013-02-03 13:50:04 -0500, Dave Rice encoded: 
> On Feb 3, 2013, at 10:55 AM, Stefano Sabatini <stefasab at gmail.com> wrote:
[...]
> > Please send an updated patch.
> 
> Updated, thank Stefano.
> 
> diff --git a/doc/syntax.texi b/doc/syntax.texi
> index a3aabce..4dddc3c 100644
> --- a/doc/syntax.texi
> +++ b/doc/syntax.texi
> @@ -170,6 +186,18 @@ The following abbreviations are recognized:
>  1280x720
>  @item hd1080
>  1920x1080
> + at item 2k
> +2048x1080
> + at item 2kflat
> +1998x1080
> + at item 2kscope
> +2048x858
> + at item 4k
> +4096x2160
> + at item 4kflat
> +3996x2160
> + at item 4kscope
> +4096x1716
>  @end table
>  
>  @anchor{video rate syntax}
> diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c
> index ca40569..73e400a 100644
> --- a/libavutil/parseutils.c
> +++ b/libavutil/parseutils.c
> @@ -109,6 +109,12 @@ static const VideoSizeAbbr video_size_abbrs[] = {
>      { "hd480",     852, 480 },
>      { "hd720",    1280, 720 },
>      { "hd1080",   1920,1080 },
> +    { "2k",       2048,1080 }, /* Digital Cinema System Specification */
> +    { "2kflat",   1998,1080 },
> +    { "2kscope",  2048, 858 },
> +    { "4k",       4096,2160 }, /* Digital Cinema System Specification */
> +    { "4kflat",   3996,2160 },
> +    { "4kscope",  4096,1716 },
>  };

Thanks, applied.

PS For the future, please always send git-format patches so the
committer doesn't have to apply the patch by hand and there are less
chances to messup (I had to copy the inline patch to a file, edit it
by hand, apply with patch -p1 -u and finally add the commit message
and author).
-- 
FFmpeg = Foolish & Free Marvellous Programmable Epic Glue


More information about the ffmpeg-devel mailing list