3 ms·
I feel like my version control system is better about highlighting the changed characters than these solid green or red strings.
by Aeolun 3y ago
I feel like my version control system is better about highlighting the changed characters than these solid green or red strings.
- TheRealPomax 3y agoIt's git, so it can show a vastly better diff, just not from a URL with hardcoded diff settings.
- Aeolun 3y agoYeah, but I’m wondering where the code review was done. In a different context, would this be easier to spot?
- XorNot 3y agoI don't love unified diffs as a rule either. They're very noisy to read in general. A side by side in something like "meld" will highlight changes but also means you're reading the full context as it will exist in the code base. My number one complaint about "code review" is the number of people who simply run their eyes over the diff (in private orgs) because management says "code review" and that's as far as they ever take that in terms of defining outcomes.
- eru 3y agoI love unified diffs for many applications. You are right that side by side diffs have their uses, too.
- masklinn 3y agoIt’s a giant block of new code, what is it supposed to do beyond tell you it’s a giant block of new code? Note that this is C code inside if a cmake string, even if your diff can do highlighting the odds it would highlight that are low, and if it did highlighting is generally just lexing so there wouldn’t be much to show.