Index: trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/EventTask.java
===================================================================
--- trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/EventTask.java	(revision 1214)
+++ trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/EventTask.java	(revision 1215)
@@ -21,23 +21,41 @@
 
 /**
- * TODO comment
- * 
- * @version $Revision: $ $Date: $
- * @author 2011, last modified by $Author: $
+ * <p>
+ * this is the default implementation of the interface {@link IEventTask}. It does not do anything
+ * fancy except implementing the interface.
+ * </p> 
+ *
+ * @author Patrick Harms
  */
 class EventTask extends Task implements IEventTask {
     
-    /**  */
+    /**
+     * <p>
+     * default serial version UID
+     * </p>
+     */
     private static final long serialVersionUID = 1L;
 
-    /** */
+    /**
+     * <p>
+     * the type of the represented event
+     * </p> 
+     */
     private IEventType eventType;
 
-    /** */
+    /**
+     * <p>
+     * the target of the represented event
+     * </p> 
+     */
     private IEventTarget eventTarget;
 
     /**
-     * @param eventType
-     * @param eventTarget
+     * <p>
+     * simple constructor initializing this task with an event type and an event target
+     * </p>
+     * 
+     * @param eventType   the type of the represented event
+     * @param eventTarget the target of the represented event
      */
     EventTask(IEventType eventType, IEventTarget eventTarget) {
@@ -47,22 +65,22 @@
     }
 
-    /**
-     * @return Returns the interaction.
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.IEventTask#getEventType()
      */
+    @Override
     public IEventType getEventType() {
         return eventType;
     }
 
-    /**
-     * @return Returns the GUIElement.
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.IEventTask#getEventTarget()
      */
+    @Override
     public IEventTarget getEventTarget() {
         return eventTarget;
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see de.harms.tasktrees.TreeNode#clone()
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.Task#clone()
      */
     @Override
@@ -73,5 +91,5 @@
 
     /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.IEventTask#accept(de.ugoe.cs.autoquest.tasktrees.treeifc.TaskVisitor)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.Task#accept(ITaskVisitor)
      */
     @Override
Index: trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Iteration.java
===================================================================
--- trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Iteration.java	(revision 1214)
+++ trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Iteration.java	(revision 1215)
@@ -18,19 +18,27 @@
 import de.ugoe.cs.autoquest.tasktrees.treeifc.IOptional;
 import de.ugoe.cs.autoquest.tasktrees.treeifc.ITask;
-import de.ugoe.cs.autoquest.tasktrees.treeifc.ITaskVisitor;
 
 /**
- * TODO comment
- * 
- * @version $Revision: $ $Date: 19.02.2012$
- * @author 2012, last modified by $Author: patrick$
+ * <p>
+ * this is the default implementation of the interface {@link IIteration}. It does not do anything
+ * fancy except implementing the interface.
+ * </p> 
+ *
+ * @author Patrick Harms
  */
 class Iteration extends MarkingTemporalRelationship implements IIteration {
 
-    /**  */
+    /**
+     * <p>
+     * default serial version UID
+     * </p>
+     */
     private static final long serialVersionUID = 1L;
 
     /**
-     *
+     * <p>
+     * simple constructor providing the base class with a human readable name of the type of this
+     * task
+     * </p>
      */
     Iteration() {
@@ -39,5 +47,5 @@
 
     /* (non-Javadoc)
-     * @see MarkingTemporalRelationship#setMarkedTask(ITask)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.MarkingTemporalRelationship#setMarkedTask(ITask)
      */
     @Override
@@ -55,8 +63,6 @@
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see java.lang.Object#clone()
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.MarkingTemporalRelationship#clone()
      */
     @Override
@@ -65,11 +71,3 @@
     }
     
-    /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.TaskTreeNode#accept(de.ugoe.cs.autoquest.tasktrees.treeifc.NodeVisitor)
-     */
-    @Override
-    public void accept(ITaskVisitor visitor) {
-        visitor.visit(this);
-    }
-
 }
Index: trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/MarkingTemporalRelationship.java
===================================================================
--- trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/MarkingTemporalRelationship.java	(revision 1214)
+++ trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/MarkingTemporalRelationship.java	(revision 1215)
@@ -20,5 +20,6 @@
 /**
  * <p>
- * TODO comment
+ * this is the default implementation of the interface {@link IMarkingTemporalRelationship}. It
+ * does not do anything fancy except implementing the interface.
  * </p>
  * 
@@ -29,14 +30,22 @@
 {
 
-    /**  */
+    /**
+     * <p>
+     * default serial version UID
+     * </p>
+     */
     private static final long serialVersionUID = 1L;
 
     /**
-     * 
+     * <p>
+     * the task marked through this marking temporal relationship
+     * </p>
      */
     private ITask markedTask;
     
     /**
-     * 
+     * <p>
+     * a human readable name for this temporal relationship
+     * </p>
      */
     private String relationshipType;
@@ -44,8 +53,8 @@
     /**
      * <p>
-     * TODO: comment
+     * initializes this temporal relationship with a human readable name
      * </p>
      *
-     * @param description
+     * @param relationshipType the human readable name of this temporal relationship
      */
     MarkingTemporalRelationship(String relationshipType) {
@@ -83,5 +92,9 @@
 
     /**
-     * @param markedTask the markedTask to set
+     * <p>
+     * used to set the marked task
+     * </p>
+     * 
+     * @param markedTask the marked task to set
      */
     protected void setMarkedTask(ITask markedTask) {
Index: trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Optional.java
===================================================================
--- trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Optional.java	(revision 1214)
+++ trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Optional.java	(revision 1215)
@@ -17,19 +17,27 @@
 import de.ugoe.cs.autoquest.tasktrees.treeifc.IOptional;
 import de.ugoe.cs.autoquest.tasktrees.treeifc.ITask;
-import de.ugoe.cs.autoquest.tasktrees.treeifc.ITaskVisitor;
 
 /**
- * TODO comment
- * 
- * @version $Revision: $ $Date: 19.02.2012$
- * @author 2012, last modified by $Author: patrick$
+ * <p>
+ * this is the default implementation of the interface {@link IOptional}. It does not do anything
+ * fancy except implementing the interface.
+ * </p> 
+ *
+ * @author Patrick Harms
  */
 class Optional extends MarkingTemporalRelationship implements IOptional {
 
-    /**  */
+    /**
+     * <p>
+     * default serial version UID
+     * </p>
+     */
     private static final long serialVersionUID = 1L;
 
     /**
-     *
+     * <p>
+     * simple constructor providing the base class with a human readable name of the type of this
+     * task
+     * </p>
      */
     Optional() {
@@ -38,5 +46,5 @@
     
     /* (non-Javadoc)
-     * @see MarkingTemporalRelationship#setMarkedTask(ITask)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.MarkingTemporalRelationship#setMarkedTask(ITask)
      */
     @Override
@@ -50,8 +58,6 @@
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see java.lang.Object#clone()
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.MarkingTemporalRelationship#clone()
      */
     @Override
@@ -60,12 +66,3 @@
     }
 
-    /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITemporalRelationship#accept(de.ugoe.cs.autoquest.tasktrees.treeifc.TaskVisitor)
-     */
-    @Override
-    public void accept(ITaskVisitor visitor) {
-        // TODO Auto-generated method stub
-        System.out.println("TODO: implement ITemporalRelationship.accept ");
-        
-    }
 }
Index: trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Selection.java
===================================================================
--- trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Selection.java	(revision 1214)
+++ trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Selection.java	(revision 1215)
@@ -17,21 +17,27 @@
 import de.ugoe.cs.autoquest.tasktrees.treeifc.ISelection;
 import de.ugoe.cs.autoquest.tasktrees.treeifc.ITask;
-import de.ugoe.cs.autoquest.tasktrees.treeifc.ITaskVisitor;
 
 /**
- * TODO comment
- * 
- * @version $Revision: $ $Date: 19.02.2012$
- * @author 2012, last modified by $Author: patrick$
+ * <p>
+ * this is the default implementation of the interface {@link ISelection}. It does not do anything
+ * fancy except implementing the interface.
+ * </p> 
+ *
+ * @author Patrick Harms
  */
 class Selection extends StructuringTemporalRelationship implements ISelection {
 
-    /**  */
+    /**
+     * <p>
+     * default serial version UID
+     * </p>
+     */
     private static final long serialVersionUID = 1L;
 
     /**
-     * TODO: comment
-     * 
-     * @param name
+     * <p>
+     * simple constructor providing the base class with a human readable name of the type of this
+     * task
+     * </p>
      */
     Selection() {
@@ -39,8 +45,6 @@
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see java.lang.Object#clone()
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.StructuringTemporalRelationship#clone()
      */
     @Override
@@ -49,37 +53,21 @@
     }
 
-    /**
-     * <p>
-     * TODO: comment
-     * </p>
-     *
-     * @param i
-     * @param newChild
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.StructuringTemporalRelationship#addChild(ITask)
      */
-    public void addChild(ITask newChild) {
+    @Override
+    void addChild(ITask newChild) {
         super.addChild(newChild);
         super.setDescription("selection of " + newChild);
     }
 
-    /**
-     * <p>
-     * TODO: comment
-     * </p>
-     *
-     * @param i
-     * @param newChild
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.StructuringTemporalRelationship#addChild(int,ITask)
      */
-    public void addChild(int index, ITask newChild) {
+    @Override
+    void addChild(int index, ITask newChild) {
         super.addChild(index, newChild);
         super.setDescription("selection of " + newChild);
     }
 
-    /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITemporalRelationship#accept(de.ugoe.cs.autoquest.tasktrees.treeifc.TaskVisitor)
-     */
-    @Override
-    public void accept(ITaskVisitor visitor) {
-        visitor.visit(this);
-    }
-
 }
Index: trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Sequence.java
===================================================================
--- trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Sequence.java	(revision 1214)
+++ trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Sequence.java	(revision 1215)
@@ -16,21 +16,27 @@
 
 import de.ugoe.cs.autoquest.tasktrees.treeifc.ISequence;
-import de.ugoe.cs.autoquest.tasktrees.treeifc.ITaskVisitor;
 
 /**
- * TODO comment
- * 
- * @version $Revision: $ $Date: 19.02.2012$
- * @author 2012, last modified by $Author: patrick$
+ * <p>
+ * this is the default implementation of the interface {@link ISequence}. It does not do anything
+ * fancy except implementing the interface.
+ * </p> 
+ *
+ * @author Patrick Harms
  */
 class Sequence extends StructuringTemporalRelationship implements ISequence {
     
-    /**  */
+    /**
+     * <p>
+     * default serial version UID
+     * </p>
+     */
     private static final long serialVersionUID = 1L;
 
     /**
-     * TODO: comment
-     * 
-     * @param name
+     * <p>
+     * simple constructor providing the base class with a human readable name of the type of this
+     * task
+     * </p>
      */
     Sequence() {
@@ -38,8 +44,6 @@
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see de.ugoe.cs.tasktree.treeimpl.TaskTreeNodeImpl#clone()
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.StructuringTemporalRelationship#clone()
      */
     @Override
@@ -48,11 +52,3 @@
     }
     
-    /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.tasktrees.treeimpl.TaskTreeNode#accept(de.ugoe.cs.autoquest.tasktrees.treeifc.NodeVisitor)
-     */
-    @Override
-    public void accept(ITaskVisitor visitor) {
-        visitor.visit(this);
-    }
-
 }
Index: trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/StructuringTemporalRelationship.java
===================================================================
--- trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/StructuringTemporalRelationship.java	(revision 1214)
+++ trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/StructuringTemporalRelationship.java	(revision 1215)
@@ -23,5 +23,6 @@
 /**
  * <p>
- * TODO comment
+ * this is the default implementation of the interface {@link IStructuringTemporalRelationship}. It
+ * does not do anything fancy except implementing the interface.
  * </p>
  * 
@@ -32,9 +33,15 @@
 {
 
-    /**  */
+    /**
+     * <p>
+     * default serial version UID
+     * </p>
+     */
     private static final long serialVersionUID = 1L;
     
     /**
-     * 
+     * <p>
+     * the list of children of this temporal relationship
+     * </p>
      */
     private List<ITask> children = new LinkedList<ITask>();
@@ -42,8 +49,8 @@
     /**
      * <p>
-     * TODO: comment
+     * initializes this temporal relationship with a human readable name
      * </p>
      *
-     * @param description
+     * @param relationshipType the human readable name of this temporal relationship
      */
     StructuringTemporalRelationship(String relationshipType) {
@@ -83,11 +90,10 @@
     /**
      * <p>
-     * TODO: comment
+     * used to add a new child to this temporal relationship.
      * </p>
      *
-     * @param i
-     * @param newChild
+     * @param newChild the new child to be added
      */
-    public void addChild(ITask newChild) {
+    void addChild(ITask newChild) {
         children.add(newChild);
     }
@@ -95,11 +101,11 @@
     /**
      * <p>
-     * TODO: comment
+     * used to add a new child to this temporal relationship at a specific position.
      * </p>
      *
-     * @param i
-     * @param newChild
+     * @param index    the index of the new child to be added.
+     * @param newChild the new child to be added
      */
-    public void addChild(int index, ITask newChild) {
+    void addChild(int index, ITask newChild) {
         children.add(index, newChild);
     }
@@ -107,8 +113,8 @@
     /**
      * <p>
-     * TODO: comment
+     * removes a child from this temporal relationship at a specific position.
      * </p>
      *
-     * @param index
+     * @param index the index of the child to be removed.
      */
     void removeChild(int index) {
Index: trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Task.java
===================================================================
--- trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Task.java	(revision 1214)
+++ trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/Task.java	(revision 1215)
@@ -19,25 +19,47 @@
 
 /**
- * TODO comment
+ * <p>
+ * this is the default implementation of the interface {@link ITask}. It
+ * does not do anything fancy except implementing the interface.
+ * </p>
  * 
- * @version $Revision: $ $Date: $
- * @author 2011, last modified by $Author: $
+ * @author Patrick Harms
  */
 class Task implements ITask {
 
-    /**  */
+    /**
+     * <p>
+     * default serial version UID
+     * </p>
+     */
     private static final long serialVersionUID = 1L;
 
-    /** */
+    /**
+     * <p>
+     * used as a counter to generate new ids for each newly created task. May overflow.
+     * </p>
+     */
     private static int temporalId = 0;
 
-    /** */
+    /**
+     * <p>
+     * the id of the task (unique throughout the system as long as {@link #temporalId} does not
+     * overflow.
+     * </p>
+     */
     private int id;
 
-    /** */
+    /**
+     * <p>
+     * a human readable description of the task
+     * </p>
+     */
     private String description;
 
     /**
-     * 
+     * <p>
+     * constructs a new task with a new id. The id is generated using the {@link #getNewId()}
+     * methdod
+     * </p>
      */
     Task() {
@@ -46,7 +68,10 @@
 
     /**
-     * TODO: comment
+     * <p>
+     * creates a new id for a task using {@link #temporalId} by incrementing it an returning its
+     * current value. Resets the counter if {@link Integer.MAX_VALUE} is reached.
+     * </p>
      * 
-     * @return
+     * @return a new unique id for a task as long as {@link #temporalId} does not overflow
      */
     private static synchronized int getNewId() {
@@ -58,15 +83,14 @@
     }
 
-    /**
-     * @return Returns the name.
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITask#geId()
      */
+    @Override
     public int getId() {
         return id;
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see de.ugoe.cs.tasktree.treeifc.TaskTreeNode#getDescription()
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITask#getDescription()
      */
     @Override
@@ -75,8 +99,6 @@
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.TaskTreeNode#equals(TaskTreeNode)
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITask#equals(ITask)
      */
     @Override
@@ -87,8 +109,6 @@
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see java.lang.Object#hashCode()
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITask#hashCode()
      */
     @Override
@@ -97,7 +117,5 @@
     }
 
-    /*
-     * (non-Javadoc)
-     * 
+    /* (non-Javadoc)
      * @see java.lang.Object#toString()
      */
@@ -117,8 +135,6 @@
     }
 
-    /*
-     * (non-Javadoc)
-     * 
-     * @see java.lang.Object#clone()
+    /* (non-Javadoc)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITask#clone()
      */
     @Override
@@ -137,8 +153,9 @@
 
     /**
-     * TODO: comment
+     * <p>
+     * internally used to set the human readable description of the task
+     * </p>
      * 
-     * @param i
-     * @return
+     * @param description the new human readable description of the task
      */
     void setDescription(String description) {
@@ -147,5 +164,5 @@
 
     /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITask#accept(de.ugoe.cs.autoquest.tasktrees.treeifc.TaskVisitor)
+     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITask#accept(ITaskVisitor)
      */
     @Override
