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

Stefano Sabatini stefasab at gmail.com
Sat Feb 2 16:21:22 CET 2013


On date Friday 2013-02-01 21:13:36 -0500, Dave Rice encoded:
> diff --git a/doc/syntax.texi b/doc/syntax.texi
> index a3aabce..8134c78 100644
> --- a/doc/syntax.texi
> +++ b/doc/syntax.texi
> @@ -170,6 +186,10 @@ The following abbreviations are recognized:
>  1280x720
>  @item hd1080
>  1920x1080

> + at item 2K
> +2048×1080
> + at item 4K
> +4096×2160

I'm not sure '×' is correctly rendered in the MAN page.

>  @end table
> diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c
> index ca40569..29f6f0d 100644
> --- a/libavutil/parseutils.c
> +++ b/libavutil/parseutils.c
> @@ -109,6 +109,8 @@ static const VideoSizeAbbr video_size_abbrs[] = {
>      { "hd480",     852, 480 },
>      { "hd720",    1280, 720 },
>      { "hd1080",   1920,1080 },
> +    { "2K",       2048,1080 }, /* Digital Cinema System Specification */
> +    { "4K",       4096,2160 }, /* Digital Cinema System Specification */

LGTM after checking Wikipedia, I'll push the three patches tomorrow
(this one with a micro bump), if no-one thinks this is wrong.
-- 
FFmpeg = Forgiving and Funny Majestic Problematic Evil Genius


More information about the ffmpeg-devel mailing list