Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 360384 - Problems in Java structure compare and compare editor in comparing two commits
Summary: Problems in Java structure compare and compare editor in comparing two commits
Status: CLOSED DUPLICATE of bug 350623
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 1.1   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-10 02:31 EDT by Ayushman Jain CLA
Modified: 2012-11-05 17:59 EST (History)
7 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ayushman Jain CLA 2011-10-10 02:31:18 EDT 
version 1.1.0.201109151100-r

From History View> Select the last two commits> Compare with each other

Open the compare editor by double clicking on any file

The compare editors show the wrong commit numbers on top. The last commit is shown on the left editor and the second to last on the right. However, the changes in the right one reflect the changes done by last commit.

Also, the Java Structure compare shows the APIs added in the last commit as "removed" instead of being "added".
Comment 1 Remy Suen CLA 2011-10-10 06:51:17 EDT 
This bug seems to be about the 'History' view and not the 'Synchronize' view. How does this bug depend on bug 358898?
Comment 2 Ayushman Jain CLA 2011-10-10 06:56:07 EDT 
(In reply to comment #1)
> This bug seems to be about the 'History' view and not the 'Synchronize' view.
> How does this bug depend on bug 358898?

No, it doesn't. Sorry, I just cloned this bug off of 358898 since it was already open in my browser, and forgot to erase the "depends on" field. :(
Comment 3 Deepak Azad CLA 2011-10-11 01:04:48 EDT 
(In reply to comment #0)
> The compare editors show the wrong commit numbers on top. 
Given that synchronize view is unusable (bug 358898), I use the history view to compare, but the wrong commit numbers can be quite annoying.

Also, I think it would be handy to show the branch names in addition to commit id. I often compare a local branch with master and reading the commit ids is a bit cumbersome. (Should this request go in a separate bug report?)
Comment 4 Robin Stocker CLA 2012-11-05 17:59:01 EST 
This has been fixed with bug 350623 in EGit 2.0.

(In reply to comment #3)
> Also, I think it would be handy to show the branch names in addition to
> commit id. I often compare a local branch with master and reading the commit
> ids is a bit cumbersome. (Should this request go in a separate bug report?)

Yes, please open a separate bug report about that.

*** This bug has been marked as a duplicate of bug 350623 ***