4 ms·
Am I missing something? Those diffs look identical. Edit: got it. Thanks!
by duhi88 6y ago
Am I missing something? Those diffs look identical.
Edit: got it. Thanks!
- Lammy 6y agoThe second one shows the changed lines' context as a method definition "def tickets_as_html" instead of the mostly-useless "class TicketPdf".
- duckerude 6y agoThe header that indicates context. Default: @@ -24,7 +24,7 @@ class TicketPdf Configured: @@ -24,7 +24,7 @@ def tickets_as_html
- duhi88 6y agoAh, thanks. I scanned those so many times but I never caught it. I think I'm used to not reading the header because, as this points out, it isn't always useful.
- ndesaulniers 6y agoI use these a lot when someone sends a patch, I'll use `vim -t <function>` to jump to the definition of <function> to better understand the change. (Where <function> is from the header of the hunk.)
- ngcazz 6y agoThe second diff identifies the target of the changes more precisely - method vs class
- renox 6y agoIn the first diff after the @@ there's a reference to the enclosing class, in the second diff it's a reference to the enclosing method. Which could help perhaps if the methods are moved?
- swrobel 6y agoWhat this really needed was a diff for their own example