diff --git a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/internal/templateengine/wizard/TemplateCNewWizard.java b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/internal/templateengine/wizard/TemplateCNewWizard.java index e5d78a4f748..d312b4c6c0c 100644 --- a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/internal/templateengine/wizard/TemplateCNewWizard.java +++ b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/internal/templateengine/wizard/TemplateCNewWizard.java @@ -31,7 +31,7 @@ public class TemplateCNewWizard extends CNewWizard { */ public EntryDescriptor[] createItems(boolean supportedOnly, IWizard wizard) { Template[] templates = TemplateEngineUI.getDefault().getTemplates(); - ArrayList items = new ArrayList(); + ArrayList items = new ArrayList(); for (int k=0; k < templates.length; k++) { TemplateInfo templateInfo = templates[k].getTemplateInfo(); @@ -43,7 +43,7 @@ public class TemplateCNewWizard extends CNewWizard { null, null)); } - return (EntryDescriptor[])items.toArray(new EntryDescriptor[items.size()]); + return items.toArray(new EntryDescriptor[items.size()]); } public void setDependentControl(Composite parent, diff --git a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/Template.java b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/Template.java index ddb3bda07d5..35c30fa5a25 100644 --- a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/Template.java +++ b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/Template.java @@ -53,7 +53,7 @@ public class Template extends TemplateCore { private TemplateDescriptor templateDescriptor; private UIElementTreeBuilderManager uiElementTreeBuilderManager; private UIPagesProvider uiPagesProvider; - private Map/**/ pageMap; + private Map pageMap; public Template(TemplateInfo templateInfo) throws TemplateInitializationException { super(templateInfo); @@ -69,9 +69,9 @@ public class Template extends TemplateCore { * 4. Request the UIPagesProvider to generate UIPages for the Tree. * 5. return the HashMap of UIPages. */ - public Map/**/ getUIPages() { + public Map getUIPages() { if (pageMap == null) { - pageMap = new HashMap/**/(); + pageMap = new HashMap(); List rootPropertyGrouplist = templateDescriptor.getPropertyGroupList(); uiPagesProvider.clearOrderVector(); @@ -91,7 +91,7 @@ public class Template extends TemplateCore { public IWizardPage[] getTemplateWizardPages(IWizardPage predatingPage, IWizardPage followingPage, IWizard wizard) { - List pages= new ArrayList(); + List pages= new ArrayList(); // if (predatingPage != null) { // pages.add(predatingPage); // } @@ -142,7 +142,7 @@ public class Template extends TemplateCore { followingPage.setPreviousPage(predatingPage); - return (IWizardPage[]) pages.toArray(new IWizardPage[pages.size()]); + return pages.toArray(new IWizardPage[pages.size()]); } IWizardDataPage[] getExtraCreatedPages(IWorkbenchWizard wizard, IWorkbench workbench, IStructuredSelection selection) { @@ -160,7 +160,7 @@ public class Template extends TemplateCore { * @return List,which contains Page display order */ - public List/**/ getPagesOrderVector() { + public List getPagesOrderVector() { return uiPagesProvider.getOrderVector(); } @@ -197,7 +197,7 @@ public class Template extends TemplateCore { } }; try { - wmo.run(monitor); // TODO support progress monitors + wmo.run(monitor); } catch(InterruptedException ie) { throw new RuntimeException(ie); } catch(InvocationTargetException ite) { diff --git a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateClassWizard.java b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateClassWizard.java index 4428c6182cb..cf2c06dedf8 100644 --- a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateClassWizard.java +++ b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateClassWizard.java @@ -21,8 +21,10 @@ import org.eclipse.jface.viewers.Viewer; import org.eclipse.ui.INewWizard; import org.eclipse.ui.wizards.newresource.BasicNewProjectResourceWizard; +import org.eclipse.cdt.core.templateengine.TemplateCore; + public class TemplateClassWizard extends TemplatesChoiceWizard implements INewWizard, IExecutableExtension { - + public static final String WIZARD_ID = TemplateClassWizard.class.getName(); private IWizardDataPage[] pagesBeforeTemplatePages; @@ -32,12 +34,12 @@ public class TemplateClassWizard extends TemplatesChoiceWizard implements INewWi private ProjectSelectionPage projectSelectionPage; private IConfigurationElement configElement; - + public TemplateClassWizard() { super(); setWindowTitle(Messages.getString("TemplateClassWizard.0")); //$NON-NLS-1$ //TODO: Fix the imagedescriptor later. -// setDefaultPageImageDescriptor(TemplateEnginePlugin.imageDescriptorFromPlugin(TemplateEnginePlugin.getDefault().getWizardIconPluginID(), TemplateEnginePlugin.getDefault().getWizardIconFile())); + //setDefaultPageImageDescriptor(TemplateEnginePlugin.imageDescriptorFromPlugin(TemplateEnginePlugin.getDefault().getWizardIconPluginID(), TemplateEnginePlugin.getDefault().getWizardIconFile())); } public String getListSelectionTitle() @@ -61,7 +63,7 @@ public class TemplateClassWizard extends TemplatesChoiceWizard implements INewWi projectSelectionPage.setTitle(Messages.getString("TemplateClassWizard.4")); //$NON-NLS-1$ projectSelectionPage.setDescription(Messages.getString("TemplateClassWizard.5")); //$NON-NLS-1$ projectSelectionPage.init(selection); - pagesBeforeTemplatePages = new IWizardDataPage[] {projectSelectionPage}; + pagesBeforeTemplatePages = new IWizardDataPage[] {projectSelectionPage}; } return pagesBeforeTemplatePages; } @@ -74,15 +76,13 @@ public class TemplateClassWizard extends TemplatesChoiceWizard implements INewWi } public Template[] getTemplates() { - SortedSet templateList = new TreeSet(Template.TEMPLATE_ID_CASE_INSENSITIVE_COMPARATOR); - + SortedSet templateList = new TreeSet(Template.TEMPLATE_ID_CASE_INSENSITIVE_COMPARATOR); templateList.addAll(Arrays.asList(TemplateEngineUI.getDefault().getTemplates())); - - return (Template[]) templateList.toArray(new Template[templateList.size()]); + return templateList.toArray(new Template[templateList.size()]); } public void setInitializationData(IConfigurationElement config, String propertyName, Object data) throws CoreException { - configElement = config; + configElement = config; } public boolean performFinish() { @@ -127,6 +127,6 @@ public class TemplateClassWizard extends TemplatesChoiceWizard implements INewWi } public void inputChanged(Viewer viewer, Object oldInput, Object newInput) { - + } } diff --git a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateDrivenWizard.java b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateDrivenWizard.java index b41197a2215..03d95736e55 100644 --- a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateDrivenWizard.java +++ b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateDrivenWizard.java @@ -28,6 +28,7 @@ import org.eclipse.ui.actions.WorkspaceModifyDelegatingOperation; import org.eclipse.cdt.core.templateengine.process.ProcessFailureException; import org.eclipse.cdt.ui.CUIPlugin; +import org.eclipse.cdt.ui.templateengine.pages.UIWizardPage; /** @@ -37,14 +38,14 @@ import org.eclipse.cdt.ui.CUIPlugin; * */ public abstract class TemplateDrivenWizard extends Wizard { - protected List/**/ pagesBeforeTemplatePages = new ArrayList/**/(); - protected List/**/ pagesAfterTemplatePages = new ArrayList/**/(); + protected List pagesBeforeTemplatePages = new ArrayList(); + protected List pagesAfterTemplatePages = new ArrayList(); protected Template template; protected int pageIndex; - protected Map/**/ templatePages; + protected Map templatePages; protected Composite pageContainer; - protected List/**/ templatePagesOrderVector; + protected List templatePagesOrderVector; public final void addPage(IWizardPage page) { page.setWizard(this); @@ -84,13 +85,13 @@ public abstract class TemplateDrivenWizard extends Wizard { public IWizardPage getPreviousPage(IWizardPage page) { if (pageIndex > pagesBeforeTemplatePages.size() + templatePagesOrderVector.size()) {//current is some page after template pages other than the first post-template page pageIndex--; - return (IWizardPage) pagesAfterTemplatePages.get(pageIndex - pagesBeforeTemplatePages.size() - templatePagesOrderVector.size()); + return pagesAfterTemplatePages.get(pageIndex - pagesBeforeTemplatePages.size() - templatePagesOrderVector.size()); } else if (pageIndex > pagesBeforeTemplatePages.size()) {//current is some template page other than the first pageIndex--; - return (IWizardPage) templatePages.get(templatePagesOrderVector.get(pageIndex - pagesBeforeTemplatePages.size())); + return templatePages.get(templatePagesOrderVector.get(pageIndex - pagesBeforeTemplatePages.size())); } else if (pageIndex > 0) { pageIndex--; - return (IWizardPage) pagesBeforeTemplatePages.get(pageIndex); + return pagesBeforeTemplatePages.get(pageIndex); } return null; } @@ -98,7 +99,7 @@ public abstract class TemplateDrivenWizard extends Wizard { public IWizardPage getNextPage(IWizardPage page) { if (pageIndex < pagesBeforeTemplatePages.size() - 1) {//current is a page before template pages that is not the final one pageIndex++; - return (IWizardPage) pagesBeforeTemplatePages.get(pageIndex); + return pagesBeforeTemplatePages.get(pageIndex); } else if (pageIndex < pagesBeforeTemplatePages.size() + templatePagesOrderVector.size() - 1) { if(pageIndex == pagesBeforeTemplatePages.size() - 1) {//current is final page before template pages Template template = getTemplate(); @@ -110,7 +111,7 @@ public abstract class TemplateDrivenWizard extends Wizard { } }//else current is some template page other than the final one pageIndex++; - IWizardPage nextPage = (IWizardPage) templatePages.get(templatePagesOrderVector.get(pageIndex - pagesBeforeTemplatePages.size())); + IWizardPage nextPage = templatePages.get(templatePagesOrderVector.get(pageIndex - pagesBeforeTemplatePages.size())); nextPage.setWizard(this); if (nextPage.getControl() == null) { nextPage.createControl(pageContainer); @@ -118,7 +119,7 @@ public abstract class TemplateDrivenWizard extends Wizard { return nextPage; } else if (pageIndex < pagesBeforeTemplatePages.size() + templatePagesOrderVector.size() + pagesAfterTemplatePages.size() - 1) {//current is final template page or a page after the final template page pageIndex++; - return (IWizardPage) pagesAfterTemplatePages.get(pageIndex - pagesBeforeTemplatePages.size() - templatePagesOrderVector.size()); + return pagesAfterTemplatePages.get(pageIndex - pagesBeforeTemplatePages.size() - templatePagesOrderVector.size()); } return null; } diff --git a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateEngineUI.java b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateEngineUI.java index 63560bf05d4..ff810b3d35e 100644 --- a/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateEngineUI.java +++ b/core/org.eclipse.cdt.ui/templateengine/org/eclipse/cdt/ui/templateengine/TemplateEngineUI.java @@ -41,7 +41,7 @@ public class TemplateEngineUI { public Template[] getTemplates(String projectType, String toolChain, String usageFilter) { TemplateInfo[] templateInfoArray = TemplateEngine.getDefault().getTemplateInfos(projectType, toolChain, usageFilter); - List/*