fixed logging, removed unused variable

apache · Dec 17, 2023 · 9994899 · 9994899
1 parent 19056e9
commit 9994899
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java b/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java
@@ -352,7 +352,7 @@ public boolean isSortEnabled() {
 
     @Override
     public void sortColumn(int col, boolean ascending) {
-        logger.debug("request to sort col=" + col);
+        logger.debug("request to sort col={}", col);
         currentSortAscending = ascending;
         currentSortColumn = col;
         sortEnabled = true;
@@ -856,7 +856,7 @@ public void propertyChange(PropertyChangeEvent arg0) {
                                 monitor.setMillisToDecideToPopup(250);
                                 monitor.setMillisToPopup(100);
                                 logger.debug(
-                                    "Changing Model, isCyclic is now " + cyclic);
+                                    "Changing Model, isCyclic is now {}", cyclic);
 
                                 List newUnfilteredList;
                                 List newFilteredList;
@@ -869,8 +869,6 @@ public void propertyChange(PropertyChangeEvent arg0) {
                                     newFilteredList = new ArrayList(cyclicBufferSize);
                                 }
 
-                                int increment = 0;
-
                                 for (Object anUnfilteredList : unfilteredList) {
                                     LoggingEventWrapper loggingEventWrapper = (LoggingEventWrapper) anUnfilteredList;
                                     newUnfilteredList.add(loggingEventWrapper);