[FFmpeg-devel] [PATCH] align pix_fmt list
Justin Ruggles
justin.ruggles
Fri Sep 25 03:36:55 CEST 2009
Michael Niedermayer wrote:
> On Sun, Sep 20, 2009 at 01:00:26PM -0400, Justin Ruggles wrote:
>> Justin Ruggles wrote:
>>
>>> Ramiro Polla wrote:
>>>
>>>> On Sun, Sep 20, 2009 at 11:52 AM, Justin Ruggles
>>>> <justin.ruggles at gmail.com> wrote:
>>>>> There are some pixel format names with more than 10 characters, so they
>>>>> mess up the list alignment. Patch attached.
>>>> [...]
>>>>
>>>>> diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c
>>>>> index ecadd63..8e0490a 100644
>>>>> --- a/libavcodec/imgconvert.c
>>>>> +++ b/libavcodec/imgconvert.c
>>>>> @@ -585,7 +585,7 @@ void avcodec_pix_fmt_string (char *buf, int buf_size, enum PixelFormat pix_fmt)
>>>>> char is_alpha_char= info.is_alpha ? 'y' : 'n';
>>>>>
>>>>> snprintf (buf, buf_size,
>>>>> - "%-10s" " %1d " " %2d " " %c ",
>>>>> + "%-11s" " %1d " " %2d " " %c ",
>>>>> info.name,
>>>>> info.nb_channels,
>>>>> info.depth,
>>>> That's a weird format string. Why isn't it simply "%-16s%-8s%-6s%c" ?
>>>> It gets filled up with whitespace anyways.
>>> It is weird... Your suggestion is not quite the same, but here is a new
>>> patch that is equivalent except for the trailing spaces.
>> or this patch, which allows for the pixel format name to grow some more.
>
> i do not want it to grow further!
> the +1 size is of course ok though
Applied the simple size increase. Attached patch to simplify the format
string.
-Justin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pix_fmt_list_align.patch
Type: text/x-diff
Size: 502 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090924/6edeb2c6/attachment.patch>
More information about the ffmpeg-devel
mailing list