On Sun, Apr 14, 2013 at 8:52 PM, Junio C Hamano <[email protected]> wrote: > Jeff King <[email protected]> writes: > >> On Sun, Apr 14, 2013 at 06:38:27PM -0500, Felipe Contreras wrote: >> >>> + if (want_color(branch_use_color)) >>> + snprintf(fancy, sizeof(fancy), "%s%s%s", >>> + >>> branch_get_color(BRANCH_COLOR_UPSTREAM), >>> + ref, >>> branch_get_color(BRANCH_COLOR_RESET)); >>> + else >>> + strncpy(fancy, ref, sizeof(fancy)); >> >> $ man strncpy | grep -C1 Warning >> The strncpy() function is similar, except that at most n bytes of src are >> copied. Warning: If there is no null byte among the first n bytes of src, >> the string placed in dest will not be null-terminated. > > Is there a reason to avoid strbuf_addf() here? We are talking about > output for human consumption and not performance critical code, no?
It's not about the performance, it's about the amount of code. But I already agreed to do this. -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
