Author: vyemialyanchyk Date: 2009-04-10 10:42:58 -0400 (Fri, 10 Apr 2009) New Revision: 14687 Modified: trunk/hibernatetools/plugins/org.hibernate.eclipse.console/src/org/hibernate/eclipse/console/wizards/NewReverseEngineeringFileWizard.java Log: JBIDE-4139 - change assignment order to assign new value to class member instead of function argument Modified: trunk/hibernatetools/plugins/org.hibernate.eclipse.console/src/org/hibernate/eclipse/console/wizards/NewReverseEngineeringFileWizard.java =================================================================== --- trunk/hibernatetools/plugins/org.hibernate.eclipse.console/src/org/hibernate/eclipse/console/wizards/NewReverseEngineeringFileWizard.java 2009-04-10 14:42:03 UTC (rev 14686) +++ trunk/hibernatetools/plugins/org.hibernate.eclipse.console/src/org/hibernate/eclipse/console/wizards/NewReverseEngineeringFileWizard.java 2009-04-10 14:42:58 UTC (rev 14687) @@ -93,7 +93,9 @@ */ public void addPages() { - if (selection == null) selection = new StructuredSelection(); + if (selection == null) { + selection = new StructuredSelection(); + } cPage = new ExtendedWizardNewFileCreationPage( "Ccfgxml", selection ); //$NON-NLS-1$ cPage.setTitle( HibernateConsoleMessages.NewReverseEngineeringFileWizard_create_hibernate_reverse_engineering_file ); cPage.setDescription( HibernateConsoleMessages.NewReverseEngineeringFileWizard_create_new_hibernate_reveng_xml ); @@ -226,8 +228,10 @@ * @see IWorkbenchWizard#init(IWorkbench, IStructuredSelection) */ public void init(IWorkbench workbench, IStructuredSelection selection) { + if (selection == null) { + selection = new StructuredSelection(); + } this.selection = selection; - if (selection == null) selection = new StructuredSelection(); } public IPath getCreatedFilePath() {