You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by ch...@apache.org on 2002/11/24 13:54:15 UTC
cvs commit: jakarta-ant-antidote/src/java/org/apache/tools/ant/gui/modules/edit DependentTargetChooser.java
chrisw 2002/11/24 04:54:15
Modified: src/java/org/apache/tools/ant/gui/modules/edit
DependentTargetChooser.java
Log:
Fixed Bug in dependency checking.
Revision Changes Path
1.3 +3 -4 jakarta-ant-antidote/src/java/org/apache/tools/ant/gui/modules/edit/DependentTargetChooser.java
Index: DependentTargetChooser.java
===================================================================
RCS file: /home/cvs/jakarta-ant-antidote/src/java/org/apache/tools/ant/gui/modules/edit/DependentTargetChooser.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- DependentTargetChooser.java 4 May 2001 18:55:19 -0000 1.2
+++ DependentTargetChooser.java 24 Nov 2002 12:54:15 -0000 1.3
@@ -198,13 +198,12 @@
DOMNodeList allNodes = parentProject.getChildNodes();
ArrayList retVal = new ArrayList();
int length = allNodes.getLength();
- for (int i = 0; i < length; i++)
- {
+ for (int i = 0; i < length; i++) {
DOMNode node = allNodes.item(i);
if (node instanceof ACSTargetElement) {
ACSTargetElement currentElement = ((ACSTargetElement)node);
// ... leave out the current target TODO: avoid cyclic relations!
- if (currentElement != aTarget) retVal.add(currentElement);
+ if (!currentElement.getName().equals(aTarget.getName())) retVal.add(currentElement);
}
}
return retVal;
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>