Calculate y-coordinate of PE line once per PE when painting EMOs and UserEvents

Previously every EMO and UserEvent was recalculating the y coordinate
for its PE when painting itself even though this value is the same for
every object on a given PE. Profiliing showed that this was a source
of great overhead which this change fixes.

Author: rbuch

src/projections/Tools/Timeline/EntryMethodObject.java

@@ -812,7 +812,7 @@ public boolean isDisplayed() {
 	}
 
 
-	public boolean paintMe(Graphics2D g2d, int actualDisplayWidth, MainPanel.MaxFilledX maxFilledX){
+	public boolean paintMe(Graphics2D g2d, final int actualDisplayWidth, final int topCoord, MainPanel.MaxFilledX maxFilledX){
 		boolean paintedEP = false;
 		// If it is hidden, we may not display it
 		if(!isDisplayed()){
@@ -829,9 +829,6 @@ public boolean paintMe(Graphics2D g2d, int actualDisplayWidth, MainPanel.MaxFill
 		if(beginTime < data.startTime())
 			leftCoord = data.timeToScreenPixelLeft(data.startTime(), actualDisplayWidth);
 
-		int topCoord = data.entryMethodLocationTop(pe);
-//		int height = data.entryMethodLocationHeight();
-
 		// Determine the coordinates and sizes of the components of the graphical representation of the object
 		int rectWidth = Math.max(1, rightCoord - leftCoord + 1);
 		int rectHeight = data.barheight();

src/projections/Tools/Timeline/MainPanel.java

@@ -249,10 +249,11 @@ public long paintAll(Graphics2D g){
 				// Saves the furthest right position EPs and msg lines have been drawn in
 				// order to optimize performance by only drawing when new pixels will be colored in
 				MaxFilledX maxFilledX = new MaxFilledX();
+				final int topCoord = data.entryMethodLocationTop(pe);
 
 				while(iter.hasNext()){
 					EntryMethodObject o = iter.next();
-					if(o.paintMe((Graphics2D) g, getWidth(), maxFilledX)) {
+					if(o.paintMe((Graphics2D) g, getWidth(), topCoord, maxFilledX)) {
 						count1++;
 					}
 				}
@@ -270,11 +271,13 @@ public long paintAll(Graphics2D g){
 			for(Integer pe : pesToRender){
 				Query1D<UserEventObject> l = data.allUserEventObjects.get(pe);
 
+				final int bottomCoord = data.userEventLocationBottom(pe);
+
 				if(l != null){ // FIXME: this shouldn't be here but is to fix a race condition with reloading ranges once something is displayed
 					Iterator<UserEventObject> iter = l.iterator(leftClipTime, rightClipTime);
 					while(iter.hasNext()){
 						UserEventObject o = iter.next();
-						o.paintMe((Graphics2D) g, getWidth(), data);
+						o.paintMe((Graphics2D) g, getWidth(), data, bottomCoord);
 						count2 ++;
 					}
 				}

src/projections/Tools/Timeline/UserEventObject.java

@@ -115,7 +115,7 @@ public int getNestedID() {
 		return nestedID;
 	}
 
-	protected void paintMe(Graphics2D g, int actualDisplayWidth, Data data) {
+	protected void paintMe(Graphics2D g, int actualDisplayWidth, Data data, final int baseBottomCoord) {
 
 		if(data.userEventIsHiddenID(userEventID)){
 			return;
@@ -135,9 +135,9 @@ protected void paintMe(Graphics2D g, int actualDisplayWidth, Data data) {
 
 		if(width < 1)
 			width = 1;
-		
-		int topCoord = data.userEventLocationBottom(pe) - (1+nestedRow) * data.singleUserEventRectHeight();
+
 		int height = data.singleUserEventRectHeight();
+		int topCoord = baseBottomCoord - (1+nestedRow) * height;
 		int bottomCoord = topCoord+height-1;
 
 		Color c = getColor(data);