This is whitespace damaged in a funny way (some lines do not have leading whitespace while some do). log_tree_diff() hence log_tree_commit() is caled by codepaths other than "git log -p", and I am not convinced that this patch would not break them. Worrysome. I wonder why we did not actually remove the duplicate parents from "commit->parents" list after history simplification when we originally did 88494239, instead of filtering only on the output path. Anybody remember the reason? - 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