[FFmpeg-devel] [PATCH] align pix_fmt list

[Justin Ruggles]

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.

-Justin

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