Jeff King <peff@xxxxxxxx> writes: >> Sure, the prefix could even be "prefix_path(): $message", I would >> think. > > I almost suggested that, but it not a change to prefix_path at all, but > rather to its callers. That may be getting nit-picky, though. :) Using "prefix_path(): free returned memory in the callers" as the title, with the body that explains that an earlier fix to make that a requirement but forgot to do so for a few that are fixed in this commit, would be sensible, no? It is not about how prefix_path() is implemented but its returned value, which is still about that function. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html