Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 215908 - WizardDialog.run should not use null as the progress monitor
Summary: WizardDialog.run should not use null as the progress monitor
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: JFace (show other bugs)
Version: 1.0   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 1.1 M2   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-20 10:16 EST by Ed Merks CLA
Modified: 2008-09-24 05:38 EDT (History)
0 users

See Also:

Attachments
Changes to use in a null progress monitor rather than null. (1.15 KB, patch)
2008-01-20 10:16 EST, Ed Merks CLA 		jkrause: iplog+
Details | Diff
View All 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 Ed Merks CLA 2008-01-20 10:16:40 EST 
Created attachment 87351 [details]
Changes to use in a null progress monitor rather than null.

Since it calls

  ModalContext.run

which does

  Assert.isTrue(operation != null && monitor != null);

it's not acceptable to pass in null.  NullProgressMonitor works better.
Comment 1 Rüdiger Herrmann CLA 2008-01-20 12:44:10 EST 
Thanks for the patch, changes applied to CVS HEAD