Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 359313 - NPE when trying to compare against multiple commits
Summary: NPE when trying to compare against multiple commits
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 1.1   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.2-M1   Edit
Assignee: Remy Suen CLA
QA Contact: Remy Suen CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-28 18:02 EDT by Remy Suen CLA
Modified: 2011-11-02 08:40 EDT (History)
0 users

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 Remy Suen CLA 2011-09-28 18:02:16 EDT 
1. Select a project > Context menu > Compare With > Commit...
2. Select two or more commits > Enter
3. The 'Git Tree Compare' view opens but nothing happens. An NPE is logged.

java.lang.NullPointerException
	at org.eclipse.egit.ui.internal.actions.CompareWithCommitActionHandler.execute(CompareWithCommitActionHandler.java:85)
	at org.eclipse.egit.ui.internal.actions.RepositoryAction.run(RepositoryAction.java:86)
	at org.eclipse.ui.internal.PluginAction.runWithEvent(PluginAction.java:251)
	at org.eclipse.jface.action.ActionContributionItem.handleWidgetSelection(ActionContributionItem.java:584)
	at org.eclipse.jface.action.ActionContributionItem.access$2(ActionContributionItem.java:501)
	at org.eclipse.jface.action.ActionContributionItem$5.handleEvent(ActionContributionItem.java:411)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1053)
	at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4165)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3754)
	at org.eclipse.ui.internal.Workbench.runEventLoop(Workbench.java:2696)
	at org.eclipse.ui.internal.Workbench.runUI(Workbench.java:2660)
Comment 1 Remy Suen CLA 2011-10-23 09:01:26 EDT 
A change set has been pushed to Gerrit for review.
http://egit.eclipse.org/r/#change,4397