Ehh, I was only kidding, though...

On Wed, Sep 9, 2015 at 10:37 AM, Karthik Nayak <karthik....@gmail.com> wrote:
> On Tue, Sep 8, 2015 at 11:50 PM, Junio C Hamano <gits...@pobox.com> wrote:
>> Matthieu Moy <matthieu....@grenoble-inp.fr> writes:
>>
>>>>> This line still looks overlong.  Would it help to stop spelling this
>>>>> as a double "a = b = overlong expression" assignment?
>>>>>
>>>>
>>>> I'm not sure, I get what you mean.
>>>
>>> I guess
>>>
>>> format = xstrfmt("%%(align:15,left)%%(refname:short)%%(end) 
>>> %%(contents:lines=%d)",
>>>                  filter->lines);
>>> to_free = format;
>>>
>>> (still 83 columns + indentation, but that's a bit shorter than your
>>> version).
>>
>> Sure.  This may also be possible
>>
>>         xstrfmt("%s %%(contents:lines=%d)",
>>                 "%(align:15,left)%(refname:short)%(end)", filter->lines);
>>
>> and highlights that the only thing that is variable is the number
>> of lines, which might be better.
>
> Yeah looks better, thanks.
>
> --
> Regards,
> Karthik Nayak
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to