[jboss-cvs] jbosside/core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/project/build ...

Robert Stryker rob.stryker at jboss.com
Mon Mar 19 21:44:58 EDT 2007


  User: rawb    
  Date: 07/03/19 21:44:58

  Modified:    core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/project/build 
                        PackageBuildDelegate.java
  Log:
  referenced packages using arraylist... needed to check for contains first
  
  Revision  Changes    Path
  1.9       +3 -2      jbosside/core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/project/build/PackageBuildDelegate.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: PackageBuildDelegate.java
  ===================================================================
  RCS file: /cvsroot/jboss/jbosside/core/plugins/org.jboss.ide.eclipse.packages.core/src/main/org/jboss/ide/eclipse/packages/core/project/build/PackageBuildDelegate.java,v
  retrieving revision 1.8
  retrieving revision 1.9
  diff -u -b -r1.8 -r1.9
  --- PackageBuildDelegate.java	7 Mar 2007 23:53:21 -0000	1.8
  +++ PackageBuildDelegate.java	20 Mar 2007 01:44:58 -0000	1.9
  @@ -404,7 +404,7 @@
   			
   			fileOperations.updateFileInFilesets(file, filesets, false);
   		}
  -		
  +		System.out.println("delegate filesToRemove: " + filesToRemove.size());
   		for (Iterator iter = filesToRemove.keySet().iterator(); iter.hasNext(); )
   		{
   			IFile file = (IFile)iter.next();
  @@ -452,6 +452,7 @@
   			// Is this right?? Is the parent guarenteed to be a package?
   			for( int i = 0; i < filesets.length; i++ ) {
   				IPackage parentPackage = PackagesCore.getParentPackage(filesets[i]);
  +				if( !packagesBeingChanged.contains(parentPackage)) 
   				packagesBeingChanged.add(parentPackage);
   			}
   			
  
  
  



More information about the jboss-cvs-commits mailing list