More fixes of static analysis warnings for UML2SD
[deliverable/tracecompass.git] / org.eclipse.linuxtools.tmf.ui / src / org / eclipse / linuxtools / tmf / ui / views / uml2sd / core / BasicFrame.java
index 3bff886a371573b659e677d37c2b88e7eb298039..5b43af7f2f326692c30b0b04007fe295c0f0c9d6 100755 (executable)
@@ -1,10 +1,11 @@
 /**********************************************************************
- * Copyright (c) 2005, 2008, 2011 IBM Corporation and others.
+ * Copyright (c) 2005, 2008 IBM Corporation and others.
+ * Copyright (c) 2011, 2012 Ericsson.
+ * 
  * All rights reserved.   This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
  * http://www.eclipse.org/legal/epl-v10.html
- * $Id: BasicFrame.java,v 1.2 2008/01/24 02:28:49 apnan Exp $
  * 
  * Contributors: 
  * IBM - Initial API and implementation
@@ -19,7 +20,7 @@ import java.util.List;
 import org.eclipse.linuxtools.tmf.core.event.ITmfTimestamp;
 import org.eclipse.linuxtools.tmf.core.event.TmfTimestamp;
 import org.eclipse.linuxtools.tmf.ui.views.uml2sd.drawings.IGC;
-import org.eclipse.linuxtools.tmf.ui.views.uml2sd.drawings.ISDPreferences;
+import org.eclipse.linuxtools.tmf.ui.views.uml2sd.preferences.ISDPreferences;
 import org.eclipse.linuxtools.tmf.ui.views.uml2sd.preferences.SDViewPref;
 
 /**
@@ -37,63 +38,91 @@ import org.eclipse.linuxtools.tmf.ui.views.uml2sd.preferences.SDViewPref;
  */
 public class BasicFrame extends GraphNode {
 
+    // ------------------------------------------------------------------------
+    // Attributes
+    // ------------------------------------------------------------------------
+
     /**
      * Contains the max elapsed time between two consecutive messages in the whole frame
      */
-    protected ITmfTimestamp maxTime = new TmfTimestamp(0);
+    protected ITmfTimestamp fMaxTime = new TmfTimestamp(0);
     /**
      * Contains the min elapsed time between two consecutive messages in the whole frame
      */
-    protected ITmfTimestamp minTime = new TmfTimestamp(0);
-
+    protected ITmfTimestamp fMinTime = new TmfTimestamp(0);
     /**
      * Indicate if the min and max elapsed time between two consecutive messages in the whole frame need to be computed
      */
-    protected boolean computeMinMax = true;
-
+    protected boolean fComputeMinMax = true;
     /**
      * Store the preference set by the user regarding the external time. This flag is used determine if the min and max
      * need to be recomputed in case this preference is changed.
      */
-    protected boolean lastExternalTimePref = SDViewPref.getInstance().excludeExternalTime();
-
+    protected boolean fLastExternalTimePref = SDViewPref.getInstance().excludeExternalTime();
     /**
      * The greater event occurrence created on graph nodes drawn in this Frame This directly impact the Frame height
      */
-    protected int verticalIndex = 0;
-
+    protected int fVerticalIndex = 0;
     /**
      * The index along the x axis where the next lifeline will is drawn This directly impact the Frame width
      */
-    protected int horizontalIndex = 0;
-
-    protected boolean timeInfo = false;
-
+    protected int fHorizontalIndex = 0;
     /**
-     * The current Frame visible area
+     * The time information flag.
      */
-    protected int visibleAreaX;
-    protected int visibleAreaY;
-    protected int visibleAreaWidth;
-    protected int visibleAreaHeight;
-
-    static ISDPreferences userPref = null;
-
-    protected int forceEventOccurrenceSpacing = -1;
-
-    protected boolean customMinMax = false;
-
-    protected ITmfTimestamp minSDTime = new TmfTimestamp();
-    protected ITmfTimestamp maxSDTime = new TmfTimestamp();
-    protected boolean initSDMin = true;
+    protected boolean fHasTimeInfo = false;
+    /**
+     * The current Frame visible area - x coordinates
+     */
+    protected int fVisibleAreaX;
+    /**
+     * The current Frame visible area - y coordinates
+     */
+    protected int fVisibleAreaY;
+    /**
+     * The current Frame visible area - width
+     */
+    protected int fVisibleAreaWidth;
+    /**
+     * The current Frame visible area - height
+     */
+    protected int fVisibleAreaHeight;
+    /**
+     * The event occurrence spacing (-1 for none)
+     */
+    protected int fForceEventOccurrenceSpacing = -1;
+    /**
+     * Flag to indicate customized minumum and maximum.
+     */
+    protected boolean fCustomMinMax = false;
+    /**
+     * The minimum time between messages of the sequence diagram frame.
+     */
+    protected ITmfTimestamp fMinSDTime = new TmfTimestamp();
+    /**
+     * The maximum time between messages of the sequence diagram frame.
+     */
+    protected ITmfTimestamp fMaxSDTime = new TmfTimestamp();
+    /**
+     * Flag to indicate that initial minimum has to be computed.
+     */
+    protected boolean fInitSDMin = true;
 
+    // ------------------------------------------------------------------------
+    // Constructors
+    // ------------------------------------------------------------------------
+    
     /**
      * Creates an empty frame.
      */
     public BasicFrame() {
-        Metrics.setForcedEventSpacing(forceEventOccurrenceSpacing);
+        Metrics.setForcedEventSpacing(fForceEventOccurrenceSpacing);
     }
 
+    // ------------------------------------------------------------------------
+    // Methods
+    // ------------------------------------------------------------------------
+
     /**
      * 
      * Returns the greater event occurence known by the Frame
@@ -101,7 +130,7 @@ public class BasicFrame extends GraphNode {
      * @return the greater event occurrence
      */
     protected int getMaxEventOccurrence() {
-        return verticalIndex;
+        return fVerticalIndex;
     }
 
     /**
@@ -110,7 +139,7 @@ public class BasicFrame extends GraphNode {
      * @param eventOccurrence the new greater event occurrence
      */
     protected void setMaxEventOccurrence(int eventOccurrence) {
-        verticalIndex = eventOccurrence;
+        fVerticalIndex = eventOccurrence;
     }
 
     /**
@@ -122,7 +151,7 @@ public class BasicFrame extends GraphNode {
      * @return a new lifeline index
      */
     protected int getNewHorizontalIndex() {
-        return ++horizontalIndex;
+        return ++fHorizontalIndex;
     }
 
     /**
@@ -132,22 +161,21 @@ public class BasicFrame extends GraphNode {
      * @see Frame#getNewHorizontalIndex() for horizontal index description
      */
     protected int getHorizontalIndex() {
-        return horizontalIndex;
+        return fHorizontalIndex;
     }
 
-    /**
-     * Add a GraphNode into the frame
-     * 
-     * @param nodeToAdd the node to add
+    /*
+     * (non-Javadoc)
+     * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#addNode(org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode)
      */
     @Override
     public void addNode(GraphNode nodeToAdd) {
-        computeMinMax = true;
+        fComputeMinMax = true;
         super.addNode(nodeToAdd);
     }
 
-    /**
-     * @return the frame x axis value in the containing view
+    /*
+     * (non-Javadoc)
      * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#getX()
      */
     @Override
@@ -155,44 +183,44 @@ public class BasicFrame extends GraphNode {
         return Metrics.FRAME_H_MARGIN;
     }
 
-    /**
-     * @return the frame y axis value in the containing view
-     * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#getX()
+
+    /*
+     * (non-Javadoc)
+     * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#getY()
      */
     @Override
     public int getY() {
         return Metrics.FRAME_V_MARGIN;
     }
 
-    /**
-     * The frame width depends on the number of lifeline added in the frame
-     * 
-     * @return the frame width
+    /*
+     * (non-Javadoc)
      * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#getWidth()
      */
     @Override
     public int getWidth() {
-        if (horizontalIndex == 0)
+        if (fHorizontalIndex == 0) {
             return 3 * Metrics.swimmingLaneWidth() + Metrics.LIFELINE_H_MAGIN * 2 - Metrics.FRAME_H_MARGIN - Metrics.LIFELINE_SPACING / 2;
-        else
-            return horizontalIndex * Metrics.swimmingLaneWidth() + Metrics.LIFELINE_H_MAGIN * 2 + 1 - Metrics.LIFELINE_SPACING;
+        } else {
+            return fHorizontalIndex * Metrics.swimmingLaneWidth() + Metrics.LIFELINE_H_MAGIN * 2 + 1 - Metrics.LIFELINE_SPACING;
+        }
     }
 
-    /**
-     * The Frame height depends on the maximum number of messages added to a lifeline( Taking all lifelines into
-     * account)
-     * 
-     * @return the frame height
+    /*
+     * (non-Javadoc)
      * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#getHeight()
      */
     @Override
     public int getHeight() {
-        if (verticalIndex == 0)
+        // The Frame height depends on the maximum number of messages added to a lifeline
+        if (fVerticalIndex == 0) {
             return 5 * (Metrics.getMessagesSpacing() + Metrics.getMessageFontHeigth()) + Metrics.LIFELINE_NAME_H_MARGIN + Metrics.FRAME_NAME_H_MARGIN + Metrics.getFrameFontHeigth() + Metrics.LIFELINE_VT_MAGIN + Metrics.LIFELINE_VB_MAGIN
                     + Metrics.LIFELINE_NAME_H_MARGIN + Metrics.FRAME_NAME_H_MARGIN + Metrics.getLifelineFontHeigth() * 2;
-        if (forceEventOccurrenceSpacing >= 0)
-            Metrics.setForcedEventSpacing(forceEventOccurrenceSpacing);
-        return verticalIndex * (Metrics.getMessagesSpacing() + Metrics.getMessageFontHeigth()) + Metrics.LIFELINE_NAME_H_MARGIN + Metrics.FRAME_NAME_H_MARGIN + Metrics.getFrameFontHeigth() + Metrics.LIFELINE_VT_MAGIN + Metrics.LIFELINE_VB_MAGIN
+        }
+        if (fForceEventOccurrenceSpacing >= 0) {
+            Metrics.setForcedEventSpacing(fForceEventOccurrenceSpacing);
+        }
+        return fVerticalIndex * (Metrics.getMessagesSpacing() + Metrics.getMessageFontHeigth()) + Metrics.LIFELINE_NAME_H_MARGIN + Metrics.FRAME_NAME_H_MARGIN + Metrics.getFrameFontHeigth() + Metrics.LIFELINE_VT_MAGIN + Metrics.LIFELINE_VB_MAGIN
                 + Metrics.LIFELINE_NAME_H_MARGIN + Metrics.FRAME_NAME_H_MARGIN + Metrics.getLifelineFontHeigth() * 2;
     }
 
@@ -206,27 +234,34 @@ public class BasicFrame extends GraphNode {
      * @param list the list to search in
      * @param fromIndex list browsing starting point
      * @return the graph node containing the point given in parameter, null otherwise
+     *
+     * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#getNodeFromListAt(int, int, java.util.List, int)
      */
     @Override
     protected GraphNode getNodeFromListAt(int x, int y, List<GraphNode> list, int fromIndex) {
-        if (list == null)
+        if (list == null) {
             return null;
+        }
         for (int i = fromIndex; i < list.size(); i++) {
             GraphNode node = (GraphNode) list.get(i);
             // only lifeline list is x ordered
             // Stop browsing the list if the node is outside the visible area
             // all others nodes will be not visible
-            if ((node instanceof Lifeline) && (node.getX() > visibleAreaX + visibleAreaWidth))
+            if ((node instanceof Lifeline) && (node.getX() > fVisibleAreaX + fVisibleAreaWidth)) {
                 break;
+            }
             if (node.getHeight() < 0) {
-                if (node.getY() + node.getHeight() > visibleAreaY + visibleAreaHeight)
+                if (node.getY() + node.getHeight() > fVisibleAreaY + fVisibleAreaHeight) {
                     break;
+                }
             } else {
-                if (node.getY() > visibleAreaY + visibleAreaHeight)
+                if (node.getY() > fVisibleAreaY + fVisibleAreaHeight) {
                     break;
+                }
             }
-            if (node.contains(x, y))
+            if (node.contains(x, y)) {
                 return node;
+            }
         }
         return null;
     }
@@ -237,8 +272,11 @@ public class BasicFrame extends GraphNode {
      * @param context the context to draw to
      */
     protected void drawFrame(IGC context) {
-        context.setBackground(Frame.getUserPref().getBackGroundColor(ISDPreferences.PREF_FRAME));
-        context.setForeground(Frame.getUserPref().getForeGroundColor(ISDPreferences.PREF_FRAME));
+        
+        ISDPreferences pref = SDViewPref.getInstance();
+        
+        context.setBackground(pref.getBackGroundColor(ISDPreferences.PREF_FRAME));
+        context.setForeground(pref.getForeGroundColor(ISDPreferences.PREF_FRAME));
 
         int x = getX();
         int y = getY();
@@ -249,36 +287,33 @@ public class BasicFrame extends GraphNode {
         context.fillRectangle(x, y, w, h);
         context.drawRectangle(x, y, w, h);
 
-        context.setBackground(Frame.getUserPref().getBackGroundColor(ISDPreferences.PREF_FRAME_NAME));
-        context.setForeground(Frame.getUserPref().getForeGroundColor(ISDPreferences.PREF_FRAME_NAME));
-        context.setFont(Frame.getUserPref().getFont(ISDPreferences.PREF_FRAME_NAME));
+        context.setBackground(pref.getBackGroundColor(ISDPreferences.PREF_FRAME_NAME));
+        context.setForeground(pref.getForeGroundColor(ISDPreferences.PREF_FRAME_NAME));
+        context.setFont(pref.getFont(ISDPreferences.PREF_FRAME_NAME));
 
         int nameWidth = context.textExtent(getName()) + 2 * Metrics.FRAME_NAME_V_MARGIN;
         int nameHeight = Metrics.getFrameFontHeigth() + +Metrics.FRAME_NAME_H_MARGIN * 2;
 
         // Draw the frame name area
-        if (nameWidth > w)
+        if (nameWidth > w) {
             nameWidth = w;
+        }
 
         int[] points = { x, y, x + nameWidth, y, x + nameWidth, y - 11 + nameHeight, x - 11 + nameWidth, y + nameHeight, x, y + nameHeight, x, y + nameHeight };
         context.fillPolygon(points);
         context.drawPolygon(points);
         context.drawLine(x, y, x, y + nameHeight);
 
-        context.setForeground(Frame.getUserPref().getFontColor(ISDPreferences.PREF_FRAME_NAME));
+        context.setForeground(pref.getFontColor(ISDPreferences.PREF_FRAME_NAME));
         context.drawTextTruncatedCentred(getName(), x, y, nameWidth - 11, nameHeight, false);
 
-        context.setBackground(Frame.getUserPref().getBackGroundColor(ISDPreferences.PREF_FRAME));
-        context.setForeground(Frame.getUserPref().getForeGroundColor(ISDPreferences.PREF_FRAME));
+        context.setBackground(pref.getBackGroundColor(ISDPreferences.PREF_FRAME));
+        context.setForeground(pref.getForeGroundColor(ISDPreferences.PREF_FRAME));
     }
 
-    /**
-     * Draws the Frame on the given context.<br>
-     * This method start width GraphNodes ordering if needed.<br>
-     * After, depending on the visible area, only visible GraphNodes are drawn.<br>
-     * 
-     * @param context the context to draw to
-     * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#draw(IGC)
+    /*
+     * (non-Javadoc)
+     * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#draw(org.eclipse.linuxtools.tmf.ui.views.uml2sd.drawings.IGC)
      */
     @Override
     public void draw(IGC context) {
@@ -295,30 +330,27 @@ public class BasicFrame extends GraphNode {
      * @see org.eclipse.linuxtools.tmf.ui.views.uml2sd.core.GraphNode#draw(IGC)
      */
     protected void draw(IGC context, boolean drawFrame) {
-        visibleAreaHeight = context.getVisibleHeight();
-        visibleAreaWidth = context.getVisibleWidth();
-        visibleAreaX = context.getContentsX();
-        visibleAreaY = context.getContentsY();
-
-        if (forceEventOccurrenceSpacing >= 0)
-            Metrics.setForcedEventSpacing(forceEventOccurrenceSpacing);
-        else
+        fVisibleAreaHeight = context.getVisibleHeight();
+        fVisibleAreaWidth = context.getVisibleWidth();
+        fVisibleAreaX = context.getContentsX();
+        fVisibleAreaY = context.getContentsY();
+
+        if (fForceEventOccurrenceSpacing >= 0) {
+            Metrics.setForcedEventSpacing(fForceEventOccurrenceSpacing);
+        } else {
             Metrics.setForcedEventSpacing(-1);
-        if (userPref == null)
-            return;
-        super.drawChildenNodes(context);
-    }
-
-    public static void setUserPref(ISDPreferences pref) {
-        userPref = pref;
-    }
+        }
 
-    public static ISDPreferences getUserPref() {
-        return userPref;
+        super.drawChildenNodes(context);
     }
 
+    /**
+     * Sets the event occurrence spacing (-1 for none)
+     * 
+     * @param space A spacing to set.
+     */
     public void forceEventOccurrenceSpacing(int space) {
-        forceEventOccurrenceSpacing = space;
+        fForceEventOccurrenceSpacing = space;
     }
 
     /**
@@ -327,7 +359,7 @@ public class BasicFrame extends GraphNode {
      * @return the X coordinates of the frame visible area
      */
     public int getVisibleAreaX() {
-        return visibleAreaX;
+        return fVisibleAreaX;
     }
 
     /**
@@ -336,7 +368,7 @@ public class BasicFrame extends GraphNode {
      * @return the frame visible area width
      */
     public int getVisibleAreaWidth() {
-        return visibleAreaWidth;
+        return fVisibleAreaWidth;
     }
 
     /**
@@ -345,7 +377,7 @@ public class BasicFrame extends GraphNode {
      * @return the frame visible area height
      */
     public int getVisibleAreaHeight() {
-        return visibleAreaHeight;
+        return fVisibleAreaHeight;
     }
 
     /**
@@ -354,7 +386,7 @@ public class BasicFrame extends GraphNode {
      * @return the X coordinates of the frame visible area
      */
     public int getVisibleAreaY() {
-        return visibleAreaY;
+        return fVisibleAreaY;
     }
 
     /**
@@ -363,30 +395,30 @@ public class BasicFrame extends GraphNode {
      * @return the minimum GraphNode time
      */
     public ITmfTimestamp getMinTime() {
-        if (lastExternalTimePref != SDViewPref.getInstance().excludeExternalTime()) {
-            lastExternalTimePref = SDViewPref.getInstance().excludeExternalTime();
-            computeMinMax = true;
+        if (fLastExternalTimePref != SDViewPref.getInstance().excludeExternalTime()) {
+            fLastExternalTimePref = SDViewPref.getInstance().excludeExternalTime();
+            fComputeMinMax = true;
         }
-        if ((computeMinMax) && (!customMinMax)) {
+        if ((fComputeMinMax) && (!fCustomMinMax)) {
             computeMinMax();
-            computeMinMax = false;
+            fComputeMinMax = false;
         }
-        return minTime;
+        return fMinTime;
     }
 
-    public void setMin(TmfTimestamp min) {
-        minTime = min;
-        customMinMax = true;
+    public void setMin(ITmfTimestamp min) {
+        fMinTime = min;
+        fCustomMinMax = true;
     }
 
-    public void setMax(TmfTimestamp max) {
-        maxTime = max;
-        customMinMax = true;
+    public void setMax(ITmfTimestamp max) {
+        fMaxTime = max;
+        fCustomMinMax = true;
     }
 
     public void resetCustomMinMax() {
-        customMinMax = false;
-        computeMinMax = true;
+        fCustomMinMax = false;
+        fComputeMinMax = true;
     }
 
     /**
@@ -395,46 +427,62 @@ public class BasicFrame extends GraphNode {
      * @return the maximum GraphNode time
      */
     public ITmfTimestamp getMaxTime() {
-        if (lastExternalTimePref != SDViewPref.getInstance().excludeExternalTime()) {
-            lastExternalTimePref = SDViewPref.getInstance().excludeExternalTime();
-            computeMinMax = true;
+        if (fLastExternalTimePref != SDViewPref.getInstance().excludeExternalTime()) {
+            fLastExternalTimePref = SDViewPref.getInstance().excludeExternalTime();
+            fComputeMinMax = true;
         }
-        if (computeMinMax) {
+        if (fComputeMinMax) {
             computeMinMax();
-            computeMinMax = false;
+            fComputeMinMax = false;
         }
-        return maxTime;
+        return fMaxTime;
     }
 
+    /**
+     * Computes the minimum and maximum time between consecutive messages within the frame. 
+     */
     protected void computeMaxMinTime() {
-        if (!initSDMin)
+        if (!fInitSDMin) {
             return;
+        }
 
         List<SDTimeEvent> timeArray = buildTimeArray();
-        if (timeArray == null)
+
+        if ((timeArray == null) || timeArray.isEmpty()) {
             return;
+        }
         for (int i = 0; i < timeArray.size(); i++) {
             SDTimeEvent m = (SDTimeEvent) timeArray.get(i);
 
-            if (m.getTime().compareTo(maxSDTime, true) > 0) {
-                maxSDTime = m.getTime();
+            if (m.getTime().compareTo(fMaxSDTime, true) > 0) {
+                fMaxSDTime = m.getTime();
             }
 
-            if ((m.getTime().compareTo(minSDTime, true) < 0) || (initSDMin == true)) {
-                minSDTime = m.getTime();
-                initSDMin = false;
+            if ((m.getTime().compareTo(fMinSDTime, true) < 0) || fInitSDMin) {
+                fMinSDTime = m.getTime();
+                fInitSDMin = false;
             }
         }
     }
 
+    /**
+     * Returns the minimum time between consecutive messages.
+     * 
+     * @return the minimum time between consecutive messages
+     */
     public ITmfTimestamp getSDMinTime() {
         computeMaxMinTime();
-        return minSDTime;
+        return fMinSDTime;
     }
 
+    /**
+     * Returns the maximum time between consecutive messages.
+     * 
+     * @return the maximum time between consecutive messages
+     */
     public ITmfTimestamp getSDMaxTime() {
         computeMaxMinTime();
-        return maxSDTime;
+        return fMaxSDTime;
     }
 
     /**
@@ -442,46 +490,59 @@ public class BasicFrame extends GraphNode {
      */
     protected void computeMinMax() {
         List<SDTimeEvent> timeArray = buildTimeArray();
-        if (timeArray == null)
+
+        if ((timeArray == null) || timeArray.isEmpty()) {
             return;
+        }
         for (int i = 0; i < timeArray.size() - 1; i++) {
             SDTimeEvent m1 = (SDTimeEvent) timeArray.get(i);
             SDTimeEvent m2 = (SDTimeEvent) timeArray.get(i + 1);
           
             updateMinMax(m1, m2);
-            
         }
     }
 
+    /**
+     * Updates the minimum and maximum time between consecutive message within the frame based on the given values.
+     * 
+     * @param m1 A first SD time event.
+     * @param m2 A second SD time event.
+     */
     protected void updateMinMax(SDTimeEvent m1, SDTimeEvent m2) {
         ITmfTimestamp delta = m2.getTime().getDelta(m1.getTime());
-        if (computeMinMax) {
-            minTime = delta.clone();
-            if (minTime.compareTo(TmfTimestamp.Zero, false) < 0) {
-                minTime = new TmfTimestamp(0, m1.getTime().getScale(), m1.getTime().getPrecision());
+        if (fComputeMinMax) {
+            fMinTime = delta.clone();
+            if (fMinTime.compareTo(TmfTimestamp.ZERO, false) < 0) {
+                fMinTime = new TmfTimestamp(0, m1.getTime().getScale(), m1.getTime().getPrecision());
             }
-            maxTime = minTime.clone();
-            computeMinMax = false;
+            fMaxTime = fMinTime.clone();
+            fComputeMinMax = false;
         }
 
-        if ((delta.compareTo(minTime, true) < 0) && (delta.compareTo(TmfTimestamp.Zero, false) > 0)) {
-            minTime = delta.clone();
+        if ((delta.compareTo(fMinTime, true) < 0) && (delta.compareTo(TmfTimestamp.ZERO, false) > 0)) {
+            fMinTime = delta.clone();
         }
 
-        if ((delta.compareTo(maxTime, true) > 0) && (delta.compareTo(TmfTimestamp.Zero, false) > 0)) {
-            maxTime = delta.clone();
+        if ((delta.compareTo(fMaxTime, true) > 0) && (delta.compareTo(TmfTimestamp.ZERO, false) > 0)) {
+            fMaxTime = delta.clone();
         }
     }
 
+    /**
+     * Builds the time array based on the list of graph nodes.
+     * 
+     * @return the time array else empty list.
+     */
     protected List<SDTimeEvent> buildTimeArray() {
-        if (!hasChilden)
-            return null;
+        if (!fHasChilden) {
+            return new ArrayList<SDTimeEvent>();
+        }
 
-        Iterator<String> it = fSort.keySet().iterator();
+        Iterator<String> it = fForwardSort.keySet().iterator();
         List<SDTimeEvent> timeArray = new ArrayList<SDTimeEvent>();
         while (it.hasNext()) {
             String nodeType = it.next();
-            List<GraphNode> list = (List<GraphNode>) nodes.get(nodeType);
+            List<GraphNode> list = (List<GraphNode>) fNodes.get(nodeType);
             for (int i = 0; i < list.size(); i++) {
                 Object timedNode = list.get(i);
                 if ((timedNode instanceof ITimeRange) && ((ITimeRange) timedNode).hasTimeInfo()) {
This page took 0.034292 seconds and 5 git commands to generate.