Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 315237 - Make ImportExportWizardButtons public
Summary: Make ImportExportWizardButtons public
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-build (show other bugs)
Version: 7.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 7.0   Edit
Assignee: Vivian Kong CLA
QA Contact: Andrew Gvozdev CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-01 11:04 EDT by Vivian Kong CLA
Modified: 2010-06-03 09:23 EDT (History)
0 users

See Also:
vivkong: review+

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 Vivian Kong CLA 2010-06-01 11:04:38 EDT 
We want to make ImportExportWizardButtons public so it can be reuse outside of CDT.

Andrew, if you have no objection I'll check this in.  Thanks.

Here's a small patch:
### Eclipse Workspace Patch 1.0
#P org.eclipse.cdt.ui
Index: src/org/eclipse/cdt/ui/newui/ImportExportWizardButtons.java
===================================================================
RCS file: /cvsroot/tools/org.eclipse.cdt/all/org.eclipse.cdt.ui/src/org/eclipse/cdt/ui/newui/ImportExportWizardButtons.java,v
retrieving revision 1.1
diff -u -r1.1 ImportExportWizardButtons.java
--- src/org/eclipse/cdt/ui/newui/ImportExportWizardButtons.java	9 Dec 2009 17:28:36 -0000	1.1
+++ src/org/eclipse/cdt/ui/newui/ImportExportWizardButtons.java	1 Jun 2010 15:03:53 -0000
@@ -34,7 +34,7 @@
  * 
  * @since 5.2
  */
-class ImportExportWizardButtons {
+public class ImportExportWizardButtons {
 	
 	private ImportExportWizardButtons() {}
Comment 1 Andrew Gvozdev CLA 2010-06-02 10:32:53 EDT 
I don't have objections just curious how you want to reuse it.
Comment 2 Vivian Kong CLA 2010-06-03 09:05:01 EDT 
In RDT we have a similar Paths and Symbols properties page and we want to add the buttons for the import/export project settings wizard in there too.
Comment 3 Vivian Kong CLA 2010-06-03 09:15:15 EDT 
Fixed in HEAD.
Comment 4 CDT Genie CLA 2010-06-03 09:23:02 EDT 
*** cdt cvs genie on behalf of vkong ***
Bug 315237 Make ImportExportWizardButtons public

[*] ImportExportWizardButtons.java 1.2 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.ui/src/org/eclipse/cdt/ui/newui/ImportExportWizardButtons.java?root=Tools_Project&r1=1.1&r2=1.2