My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 223: "Ignore white space" behaves strangely across multiple files
  Back to list
Status:  CannotReproduce
Owner:  code-rev...@gtempaccount.com
Closed:  Sep 2009


Sign in to add a comment
 
Reported by code-rev...@gtempaccount.com, Sep 24, 2009
Reported by Cedric Beust <cbeust@google.com> on Wed Jun 17 10:51:26 PDT 2009
Source: JIRA GERRIT-224

From Marco:

Go to https://android-git.corp.google.com/g/Gerrit#change,4475 and
look at any file in side-by-side mode.
Check the "ignore whitespace" box (it should show an empty diff), then
uncheck it again, and click the "next" link to go to the next file
(don't go up first, go straight to the next one).
Repeat 3 or 4 times, and you'll see that the diff starts to mess up.
Sometimes it'll show a full diff even with the box checked, sometimes
the left column will be empty, sometimes the "next" link suddenly
disappears and you can only go "up", even though you're not on the
last file yet.
Sep 24, 2009
#1 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Thu Jun 18 16:58:25 PDT 2009

I can't reproduce any of this behavior now.  I wonder if the issue was it
transitioned to unified mode when there was no difference in the file, and
Marco hit reload at some point, which then caused the left/right links to
disappear because the client no longer had the List<Patch> handy?
Sep 24, 2009
#2 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Thu Jun 18 16:58:25 PDT 2009
Status: CannotReproduce
Sign in to add a comment

Powered by Google Project Hosting