Check for existence of packs and messages before checking if they should be shown

Doing these cheaper checks against null first gives better short
circuiting behavior, avoiding the more expensive calls to check if
they should be shown when they don't exist.

Author: rbuch

src/projections/Tools/Timeline/EntryMethodObject.java

@@ -911,7 +911,7 @@ public boolean paintMe(Graphics2D g2d, final int actualDisplayWidth, final int t
 
 		 */
 
-		if(data.showPacks() && packs != null)
+		if(packs != null && data.showPacks())
 		{
 			g2d.setColor(Color.pink);
 			for(PackTime pt : packs){
@@ -938,7 +938,7 @@ public boolean paintMe(Graphics2D g2d, final int actualDisplayWidth, final int t
 
 		// Show the message sends. See note above for the message packing areas
 		// Don't change this without changing MainPanel's paintComponent which draws message send lines
-		if(data.showMsgs() == true && messages != null)
+		if(messages != null && data.showMsgs())
 		{
 			g2d.setColor(data.getForegroundColor());