On Sun, Apr 14, 2013 at 5:31 PM, Duy Nguyen <pclo...@gmail.com> wrote:
> On Sun, Apr 14, 2013 at 11:46 AM, Felipe Contreras
> <felipe.contre...@gmail.com> wrote:
>> + char fancy[80];
>>
>> - if (show_upstream_ref)
>> + if (show_upstream_ref) {
>> ref = shorten_unambiguous_ref(branch->merge[0]->dst, 0);
>> + if (want_color(branch_use_color))
>> + snprintf(fancy, sizeof(fancy), "%s%s%s",
>> + GIT_COLOR_BLUE, ref,
>> GIT_COLOR_RESET);
>> + else
>> + strncpy(fancy, ref, sizeof(fancy));
>> + }
>> +
>
> Please use strbuf for "fancy".
Why? We would need to initialize and free it. What's the advantage?
--
Felipe Contreras
--
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
