[Exmmt-commit] SF.net SVN: exmmt: [52] net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder

[<lgr...@us...>]

Revision: 52
Author:   lgrammel
Date:     2006-06-05 02:37:43 -0700 (Mon, 05 Jun 2006)
ViewCVS:  http://svn.sourceforge.net/exmmt/?rev=52&view=rev

Log Message:
-----------
#1500774 scenario "shopping list"

Modified Paths:
--------------
    net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder/Composite.java
    net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder/TreeBuilder.java
Modified: net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder/Composite.java
===================================================================
--- net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder/Composite.java	2006-06-05 09:34:28 UTC (rev 51)
+++ net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder/Composite.java	2006-06-05 09:37:43 UTC (rev 52)
@@ -41,9 +41,16 @@
 		return parent;
 	}
 
-	public void setParent(Composite parent) {
+	private void setParent(Composite parent) {
 		this.parent = parent;
 	}
+	
+	public void add(Composite child) {
+		assert child != null;
+		
+		child.setParent(this);
+		children.add(child);
+	}
 
 	public boolean isRoot() {
 		return parent == null;

Modified: net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder/TreeBuilder.java
===================================================================
--- net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder/TreeBuilder.java	2006-06-05 09:34:28 UTC (rev 51)
+++ net.sourceforge.exmmt/trunk/src/java/net/sourceforge/exmmt/builder/TreeBuilder.java	2006-06-05 09:37:43 UTC (rev 52)
@@ -24,20 +24,16 @@
 
 		List<Element> childElements = element.getChildren();
 		for (Element childElement : childElements) {
-			// TODO getChildren should return immutable, create add method in
-			// Composite
+			// TODO getChildren should return immutable
 			Composite child = build(childElement);
-			child.setParent(composite);
-			composite.getChildren().add(child);
+			composite.add(child);
 		}
 
 		List<Attribute> attributes = element.getAttributes();
 		for (Attribute attribute : attributes) {
-			// TODO getChildren should return immutable, create add method in
-			// Composite
+			// TODO getChildren should return immutable
 			Composite child = build(attribute);
-			child.setParent(composite);
-			composite.getChildren().add(child);
+			composite.add(child);
 		}
 
 		return composite;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.