[jboss-cvs] jbosside/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/views/server/providers ...
Robert Stryker
rawblem at gmail.com
Mon Dec 11 10:53:03 EST 2006
User: rawb
Date: 06/12/11 10:53:03
Modified: as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/views/server/providers
EventLogViewProvider.java
Log:
Added null checks. They came up in testing but not consistantly, so really I'm not sure they need to be there. They might have only shown up in testing due to some crazy breakpoint locations. I don't know.
Revision Changes Path
1.5 +2 -1 jbosside/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/views/server/providers/EventLogViewProvider.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: EventLogViewProvider.java
===================================================================
RCS file: /cvsroot/jboss/jbosside/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/views/server/providers/EventLogViewProvider.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -b -r1.4 -r1.5
--- EventLogViewProvider.java 30 Nov 2006 16:13:52 -0000 1.4
+++ EventLogViewProvider.java 11 Dec 2006 15:53:03 -0000 1.5
@@ -131,7 +131,7 @@
for( int i = 0; i < children.length; i++ ) {
if( children[i] instanceof EventLogTreeItem ) {
String type = ((EventLogTreeItem)children[i]).getEventClass();
- if( type.equals(parentElement))
+ if( type != null && type.equals(parentElement))
items.add(children[i]);
}
}
@@ -208,6 +208,7 @@
}
public String getText(Object element) {
+ if( element == null ) return "NULL, ERROR";
String suffix = getShowTimestamp() ? createTimestamp(element) : "";
if( labelProviderDelegates == null )
loadLabelProviderDelegates();
More information about the jboss-cvs-commits
mailing list