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
-~--~~~~--~~--~--~---
