[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