[FFmpeg-devel] [PATCH] align pix_fmt list

[Justin Ruggles]

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.

-Justin

-------------- next part --------------
A non-text attachment was scrubbed...
Name: pix_fmt_list_align_3.patch
Type: text/x-diff
Size: 503 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090920/944ea58c/attachment.patch>