A personal pet peeve of mine when reading diffs, is when a file has some functions and you insert one and instead of looking like this:
int someOldFunction()
{
// Function body
}
+
+int newFunction()
+{
+ // New function body
+}
It looks like this:
int someOldFunction()
{
// Function body
+}
+
+int newFunction()
+{
+ // New function body
}
It's a small thing, but given that these diffs are equivalent, the one that balances the curly braces within added blocks should be favored. But diff utilities seem to get this pretty consistently wrong.