My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 23: Highlight lines with trailing whitespace in side by side and unified diff views
  Back to list
Status:  Released
Owner:  code-rev...@gtempaccount.com
Closed:  Oct 2012


Sign in to add a comment
 
Reported by code-rev...@gtempaccount.com, Sep 24, 2009
Reported by Shawn Pearce <sop@google.com> on Thu Jan 15 07:51:14 PST 2009
Source: JIRA GERRIT-23
Affected Version: 2.0

git diff highlights trailing whitespace on lines with red background if
diff.color is enabled.  Gerrit should do the same thing so reviewers can be
easily warned if a change is going to introduce trailing whitespace.  Usually
trailing whitespace isn't desired, as it provides no semantic value, so
helping reviewers to spot it and keep it out of the code base is a good thing.
Sep 24, 2009
#1 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Thu Jan 15 07:51:26 PST 2009
Labels: -Priority-Major Priority-Trivial
Sep 24, 2009
#2 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Fri Feb 06 17:34:13 PST 2009

Fixed by http://review.source.android.com/8805
Sep 24, 2009
#3 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Fri Feb 06 17:34:13 PST 2009

Fixed in version 2.0.2.
Status: Fixed
Sep 25, 2009
#4 code-rev...@gtempaccount.com
(No comment was entered for this change.)
Labels: FixedIn-2.0.2
Oct 25, 2012
#5 sop@google.com
(No comment was entered for this change.)
Status: Released
Sign in to add a comment

Powered by Google Project Hosting