Updates:
        Status: Confirmed
        Labels: Interdiffs Component-DiffViewer

Comment #2 on issue 1224 by chipx86: yellow diff highlighting is off by one  
line
http://code.google.com/p/reviewboard/issues/detail?id=1224

Been looking into this. It's not just those lines. Most of the lines are  
off. Any
replace line, in fact.

This seems to be due to our markup and text lines being off. We compare the  
plain
text lines and then generate the interline diff highlighting, and then  
apply the
highlighting to the HTML lines. However, we're applying them to a line too  
early,
implying that the HTML line is off by one compared to the plain text lines.

This all appears to start with a block that looks somewhat like:

/*
  * NOTE: Blah blah blah...
  *       Blah blah blah...
  *       total of 6 lines of this...
  */

The highlighting here is wrong. It starts at the line before the comment,  
actually.
While both diffs show a blank line there, the diff viewer shows a replace  
line. This
implies that the lines are not equal.

I suspect there's a line termination issue here. Why it starts on that  
line, I do not
know. Perhaps there's something screwy in this particular change with the  
newlines.
We would need a good, self-contained test case for this.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to