oburn 2003/03/10 21:22:50
Modified: src/java/org/apache/log4j/chainsaw Main.java
EventDetailSink.java ChainsawAppender.java
Log:
Fix checkstyle errors
Revision Changes Path
1.7 +10 -10 jakarta-log4j/src/java/org/apache/log4j/chainsaw/Main.java
Index: Main.java
===================================================================
RCS file: /home/cvs/jakarta-log4j/src/java/org/apache/log4j/chainsaw/Main.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- Main.java 11 Mar 2003 03:59:21 -0000 1.6
+++ Main.java 11 Mar 2003 05:22:49 -0000 1.7
@@ -97,7 +97,7 @@
public static final String DETAILS_SEPARATOR_PROPERTY =
Preferences.PROP_PREFIX + ".details.separator";
private static final Preferences PREFS = Preferences.getInstance();
- private JSplitPane aDetailsDivider;
+ private JSplitPane mDetailsDivider;
private final MyTableColumnModel mColumnModel;
/**
@@ -127,11 +127,11 @@
}
/**
- * Constructs the JTable used for displaying the Events logs
- * @param tableModel
- * @param tableColumnModel
- * @return
- */
+ * Constructs the JTable used for displaying the Events logs
+ * @param tableModel
+ * @param tableColumnModel
+ * @return
+ */
private JTable buildTable(
TableModel tableModel, TableColumnModel tableColumnModel) {
final JTable table = new JTable(tableModel, mColumnModel);
@@ -162,9 +162,9 @@
details.setPreferredSize(new Dimension(900, 100));
// Add the table and stack trace into a splitter
- aDetailsDivider =
+ mDetailsDivider =
new JSplitPane(JSplitPane.VERTICAL_SPLIT, scrollPane, details);
- getContentPane().add(aDetailsDivider, BorderLayout.CENTER);
+ getContentPane().add(mDetailsDivider, BorderLayout.CENTER);
}
/**
@@ -229,7 +229,7 @@
final int divider = PREFS.getInteger(DETAILS_SEPARATOR_PROPERTY, -1);
if (divider > 0) {
- aDetailsDivider.setDividerLocation(divider);
+ mDetailsDivider.setDividerLocation(divider);
}
final int x = PREFS.getInteger(X_POSITION_PROPERTY, 0);
@@ -250,7 +250,7 @@
mColumnModel.savePrefs();
PREFS.setInteger(
- DETAILS_SEPARATOR_PROPERTY, aDetailsDivider.getDividerLocation());
+ DETAILS_SEPARATOR_PROPERTY, mDetailsDivider.getDividerLocation());
PREFS.setInteger(X_POSITION_PROPERTY, getX());
PREFS.setInteger(Y_POSITION_PROPERTY, getY());
PREFS.setInteger(WIDTH_PROPERTY, getWidth());
1.3 +1 -1
jakarta-log4j/src/java/org/apache/log4j/chainsaw/EventDetailSink.java
Index: EventDetailSink.java
===================================================================
RCS file:
/home/cvs/jakarta-log4j/src/java/org/apache/log4j/chainsaw/EventDetailSink.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- EventDetailSink.java 11 Mar 2003 03:59:22 -0000 1.2
+++ EventDetailSink.java 11 Mar 2003 05:22:49 -0000 1.3
@@ -57,5 +57,5 @@
* @version 1.0
*/
public interface EventDetailSink {
- public void addEvent(EventDetails aDetail);
+ void addEvent(EventDetails aDetail);
}
1.3 +12 -3
jakarta-log4j/src/java/org/apache/log4j/chainsaw/ChainsawAppender.java
Index: ChainsawAppender.java
===================================================================
RCS file:
/home/cvs/jakarta-log4j/src/java/org/apache/log4j/chainsaw/ChainsawAppender.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- ChainsawAppender.java 11 Mar 2003 03:56:57 -0000 1.2
+++ ChainsawAppender.java 11 Mar 2003 05:22:49 -0000 1.3
@@ -96,7 +96,7 @@
/**
* Return the singleton instance of the ChainsawAppender, it should only
* be initialised once.
- * @return
+ * @return the instance
*/
static ChainsawAppender getInstance() {
return sSharedAppender;
@@ -141,45 +141,54 @@
* Close does nothing
*/
public void close() {
- // TODO: perhaps it should clear the internal TableModel
+ // TODO: perhaps it should clear the internal TableModel
}
// ==========================================================================
// All methods here are from TableModel, and simply forward on to the
// internal wrappedTableModel instance
- // ==========================================================================
+ // =========================================================================
+ /** @see TableModel */
public int getRowCount() {
return wrappedTableModel.getRowCount();
}
+ /** @see TableModel */
public int getColumnCount() {
return wrappedTableModel.getColumnCount();
}
+ /** @see TableModel */
public String getColumnName(int aColumnIndex) {
return wrappedTableModel.getColumnName(aColumnIndex);
}
+ /** @see TableModel */
public Class getColumnClass(int columnIndex) {
return wrappedTableModel.getColumnClass(columnIndex);
}
+ /** @see TableModel */
public boolean isCellEditable(int rowIndex, int columnIndex) {
return wrappedTableModel.isCellEditable(rowIndex, columnIndex);
}
+ /** @see TableModel */
public Object getValueAt(int rowIndex, int columnIndex) {
return wrappedTableModel.getValueAt(rowIndex, columnIndex);
}
+ /** @see TableModel */
public void setValueAt(Object aValue, int rowIndex, int columnIndex) {
wrappedTableModel.setValueAt(aValue, rowIndex, columnIndex);
}
+ /** @see TableModel */
public void addTableModelListener(TableModelListener l) {
wrappedTableModel.addTableModelListener(l);
}
+ /** @see TableModel */
public void removeTableModelListener(TableModelListener l) {
wrappedTableModel.removeTableModelListener(l);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
