On Sat, 1 Mar 2008, Denis Cheng wrote: > > rev->abbrev = DEFAULT_ABBREV; > - rev->commit_format = CMIT_FMT_DEFAULT; > + if (fmt_pretty) > + rev->commit_format = get_commit_format(fmt_pretty); Umm. Now it looks like commit_format isn't initialized at all if fmt_pretty hasn't been set. Now, it looks like it will have been initialized properly in "init_revisions()", but your commit log doesn't mention that, so it was harder to review this patch than necessary. Also, can you describe what the background for this is? The reason I ask is that if anybody ever sets that default commit format to anythign else, it will now *seriously* confuse not just users but potentially other git tools too (at least gitk uses "--pretty=raw", but who knows what other tools/scripts are around that just expected the default format). Linus -- 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
