Re: [PATCH 5/7] help: reuse struct column_layout

From: Nguyen Thai Ngoc Duy <pclouds@gmail.com>
Date: Feb 9, 2011, 11:21 UTC
Message-ID: <AANLkTinTvTdng2MpVmPfF3O1c08mFEEr1x_pv1_1unBX@mail.gmail.com>
In-Reply-To: <20110209073953.GB2135@elie>

2011/2/9 Jonathan Nieder <jrnieder@gmail.com>:

Show 14 quoted lines

> Nguyễn Thái Ngọc Duy wrote:
>
> [...]
>> +++ b/help.c
>> @@ -72,29 +73,17 @@ void exclude_cmds(struct cmdnames *cmds, struct cmdnames *excludes)
> [...]
>> +     struct column_layout c;
>> +     int i;
>> +
>> +     memset(&c, 0, sizeof(c));
>
> Might be nice to have a COLUMN_LAYOUT_INIT, I suppose.
>
> Looks sane.  What output changes would be noticeable, if any?

Two spaces between column instead of one. Will fix.

-- 
Duy