psmith      2003/09/02 22:42:02

  Modified:    src/java/org/apache/log4j/chainsaw
                        TableColorizingRenderer.java
  Log:
  Colour renderer was relying on Vectors to represent
  the events.  Underlying model is LoggingEvent now.
  
  This class however does not colour anything anymore.
  
  Revision  Changes    Path
  1.2       +6 -5      
jakarta-log4j/src/java/org/apache/log4j/chainsaw/TableColorizingRenderer.java
  
  Index: TableColorizingRenderer.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-log4j/src/java/org/apache/log4j/chainsaw/TableColorizingRenderer.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- TableColorizingRenderer.java      25 Jun 2003 04:05:21 -0000      1.1
  +++ TableColorizingRenderer.java      3 Sep 2003 05:42:02 -0000       1.2
  @@ -54,6 +54,7 @@
   import org.apache.log4j.chainsaw.prefs.SaveSettingsEvent;
   import org.apache.log4j.chainsaw.prefs.SettingsListener;
   import org.apache.log4j.helpers.ISO8601DateFormat;
  +import org.apache.log4j.spi.LoggingEvent;
   
   import java.awt.Color;
   import java.awt.Component;
  @@ -83,6 +84,7 @@
    *
    * @author Claude Duguay
    * @author Scott Deboy <[EMAIL PROTECTED]>
  + * @author Paul Smith <[EMAIL PROTECTED]>
    *
    */
   public class TableColorizingRenderer extends DefaultTableCellRenderer
  @@ -93,7 +95,6 @@
     private ColorFilter colorFilter;
     private JTable table;
     private Color background = new Color(255, 255, 254);
  -  private Vector v = new Vector();
     private final Color COLOR_ODD = new Color(230, 230, 230);
     private final JLabel idComponent = new JLabel();
     private final JLabel levelComponent = new JLabel();
  @@ -224,12 +225,12 @@
   
       if ((color == null) && (colorFilter != null)) {
         TableModel model = table.getModel();
  -
  +      LoggingEvent event = null;
         if (model instanceof EventContainer) {
           EventContainer model2 = (EventContainer) model;
  -        v = model2.getRow(row);
  +        event = model2.getRow(row);
   
  -        if (v == null) {
  +        if (event == null) {
             //ignore...probably changed displayed cols
             return c;
           }
  @@ -238,7 +239,7 @@
             getClass() + " can only support an EventContainer TableModel");
         }
   
  -      color = colorFilter.getColor(ChainsawColumns.getColumnsNames(), v);
  +      color = colorFilter.getColor(ChainsawColumns.getColumnsNames(), event);
       }
   
       if ((color != null)) {
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to