[FFmpeg-devel] [PATCH 3/3] add digital cinema frame sizes
Dave Rice
dave at dericed.com
Sat Feb 2 16:51:12 CET 2013
On Feb 2, 2013, at 10:21 AM, Stefano Sabatini <stefasab at gmail.com> wrote:
> 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.
Sorry about that, wrong character. It is here replaced with 'x' 0x78. The character I used before was 0xc397.
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
+2048x1080
+ at item 4K
+4096x2160
@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 */
};
>> @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.
Here is my reference for the 2K and 4K frame sizes: http://dcimovies.com/specification/DCI_DCSS_v12_with_errata_2012-1010.pdf.
Any style preference for "2K" or "2k"?
Dave Rice
More information about the ffmpeg-devel
mailing list