Thread (1 message) 1 message, 1 author, 2017-06-02
  • (off-list ancestor, not in this archive)
  • Re: Incorrect Diff! · Junio C Hamano <hidden> · 2017-06-02

Re: Incorrect Diff!

From: Junio C Hamano <hidden>
Date: 2017-06-02 04:55:26

Pulkit Goyal [off-list ref] writes:
I was working on a Django project when I got the following diff which
is very much confusing.

The diff is pasted at https://bpaste.net/show/c2cb00728ff0.

As you can see on bpaste that in line number 9,10 I have changed
"class activationview" to "class activateuser", but in line number 14
the diff is again showing me "class activationview". Maybe Git is
trying to be smart here, but the diff is wrong and I was confused.
I do not think Git is trying to be smart at all.  

The function context lines @@ .. @@ consistently is showing the name
in the preimage of the diff, no?  

The function/class header that appears in the preimage before the
hunk that begins at line 108 is activationview and that is shown
there.  So is the header on the hunk that begins at line 149.

So there is nothing remarkable in this patch as far as I can see.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help