You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ma...@apache.org on 2018/03/10 05:40:18 UTC
[11/14] logging-chainsaw git commit: Simplify boolean return
expressions
Simplify boolean return expressions
Project: http://git-wip-us.apache.org/repos/asf/logging-chainsaw/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-chainsaw/commit/0a52dc99
Tree: http://git-wip-us.apache.org/repos/asf/logging-chainsaw/tree/0a52dc99
Diff: http://git-wip-us.apache.org/repos/asf/logging-chainsaw/diff/0a52dc99
Branch: refs/heads/master
Commit: 0a52dc99070b5e1b3f99fe99bd2466a1418632a4
Parents: 27a76c0
Author: Matt Sicker <bo...@gmail.com>
Authored: Fri Mar 9 23:10:36 2018 -0600
Committer: Matt Sicker <bo...@gmail.com>
Committed: Fri Mar 9 23:10:36 2018 -0600
----------------------------------------------------------------------
.../ChainsawCyclicBufferTableModel.java | 11 +++----
.../java/org/apache/log4j/chainsaw/LogUI.java | 11 ++-----
.../log4j/chainsaw/LoggerNameTreePanel.java | 9 ++----
.../org/apache/log4j/chainsaw/RuleMediator.java | 19 +++++------
.../filter/EventTypeEntryContainer.java | 5 +--
.../receivers/PluginPropertyEditorPanel.java | 6 +---
.../java/org/apache/log4j/db/DBAppender.java | 34 +++++++++-----------
7 files changed, 33 insertions(+), 62 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/logging-chainsaw/blob/0a52dc99/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java b/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java
index c15a2e8..81887b6 100644
--- a/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java
+++ b/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java
@@ -766,15 +766,12 @@ fireTableRowsInserted(begin, end);
* @see javax.swing.table.TableModel#isCellEditable(int, int)
*/
public boolean isCellEditable(int rowIndex, int columnIndex) {
- if (getColumnName(columnIndex).equalsIgnoreCase(ChainsawConstants.LOG4J_MARKER_COL_NAME_LOWERCASE)) {
- return true;
- }
+ if (getColumnName(columnIndex).equalsIgnoreCase(ChainsawConstants.LOG4J_MARKER_COL_NAME_LOWERCASE)) {
+ return true;
+ }
- if (columnIndex >= columnNames.size()) {
- return false;
- }
+ return columnIndex < columnNames.size() && super.isCellEditable(rowIndex, columnIndex);
- return super.isCellEditable(rowIndex, columnIndex);
}
/* (non-Javadoc)
http://git-wip-us.apache.org/repos/asf/logging-chainsaw/blob/0a52dc99/src/main/java/org/apache/log4j/chainsaw/LogUI.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/log4j/chainsaw/LogUI.java b/src/main/java/org/apache/log4j/chainsaw/LogUI.java
index 98174ad..82aba92 100644
--- a/src/main/java/org/apache/log4j/chainsaw/LogUI.java
+++ b/src/main/java/org/apache/log4j/chainsaw/LogUI.java
@@ -1580,11 +1580,7 @@ e.printStackTrace();
Map.Entry entry = (Map.Entry) aPanelSet;
Object o = entry.getValue();
boolean valueToSend;
- if (o instanceof LogPanel) {
- valueToSend = ((DockablePanel) entry.getValue()).isDocked();
- } else {
- valueToSend = true;
- }
+ valueToSend = !(o instanceof LogPanel) || ((DockablePanel) entry.getValue()).isDocked();
m.put(entry.getKey(), valueToSend);
}
@@ -1780,11 +1776,8 @@ e.printStackTrace();
* @return log tree panel visible flag
*/
public boolean isLogTreePanelVisible() {
- if (getCurrentLogPanel() == null) {
- return false;
- }
+ return getCurrentLogPanel() != null && getCurrentLogPanel().isLogTreeVisible();
- return getCurrentLogPanel().isLogTreeVisible();
}
/**
http://git-wip-us.apache.org/repos/asf/logging-chainsaw/blob/0a52dc99/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java b/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java
index 84a53f9..79ab5c0 100644
--- a/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java
+++ b/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java
@@ -195,19 +195,14 @@ final class LoggerNameTreePanel extends JPanel implements LoggerNameListener
colorRuleDelegate =
new AbstractRule()
{
- public boolean evaluate(LoggingEvent e, Map matches)
- {
+ public boolean evaluate(LoggingEvent e, Map matches) {
boolean hiddenLogger = e.getLoggerName() != null && isHiddenLogger(e.getLoggerName());
boolean hiddenExpression = (ignoreExpressionRule != null && ignoreExpressionRule.evaluate(e, null));
boolean alwaysDisplayExpression = (alwaysDisplayExpressionRule != null && alwaysDisplayExpressionRule.evaluate(e, null));
boolean hidden = (!alwaysDisplayExpression) && (hiddenLogger || hiddenExpression);
String currentlySelectedLoggerName = getCurrentlySelectedLoggerName();
- if (!isFocusOnSelected() && !hidden && currentlySelectedLoggerName != null && !"".equals(currentlySelectedLoggerName))
- {
- return (e.getLoggerName().startsWith(currentlySelectedLoggerName+".") || e.getLoggerName().endsWith(currentlySelectedLoggerName)) ;
- }
- return false;
+ return !isFocusOnSelected() && !hidden && currentlySelectedLoggerName != null && !"".equals(currentlySelectedLoggerName) && (e.getLoggerName().startsWith(currentlySelectedLoggerName + ".") || e.getLoggerName().endsWith(currentlySelectedLoggerName));
}
};
http://git-wip-us.apache.org/repos/asf/logging-chainsaw/blob/0a52dc99/src/main/java/org/apache/log4j/chainsaw/RuleMediator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/log4j/chainsaw/RuleMediator.java b/src/main/java/org/apache/log4j/chainsaw/RuleMediator.java
index b8f6270..0e26fb9 100644
--- a/src/main/java/org/apache/log4j/chainsaw/RuleMediator.java
+++ b/src/main/java/org/apache/log4j/chainsaw/RuleMediator.java
@@ -50,20 +50,17 @@ public class RuleMediator extends AbstractRule {
* @see org.apache.log4j.chainsaw.rule.Rule#evaluate(org.apache.log4j.spi.LoggingEvent)
*/
public boolean evaluate(LoggingEvent e, Map matches) {
- if (findRuleRequired) {
- if (findRule == null) {
- return false;
+ if (findRuleRequired) {
+ if (findRule == null) {
+ return false;
+ }
+ if (!findRule.evaluate(e, null)) {
+ return false;
+ }
}
- if (!findRule.evaluate(e, null)) {
- return false;
- }
- }
- if (loggerRule != null && !loggerRule.evaluate(e, null)) {
- return false;
- }
+ return (loggerRule == null || loggerRule.evaluate(e, null)) && (filterRule == null || filterRule.evaluate(e, null));
- return filterRule == null || filterRule.evaluate(e, null);
}
public boolean isFindRuleRequired() {
http://git-wip-us.apache.org/repos/asf/logging-chainsaw/blob/0a52dc99/src/main/java/org/apache/log4j/chainsaw/filter/EventTypeEntryContainer.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/log4j/chainsaw/filter/EventTypeEntryContainer.java b/src/main/java/org/apache/log4j/chainsaw/filter/EventTypeEntryContainer.java
index 505fb15..459356d 100644
--- a/src/main/java/org/apache/log4j/chainsaw/filter/EventTypeEntryContainer.java
+++ b/src/main/java/org/apache/log4j/chainsaw/filter/EventTypeEntryContainer.java
@@ -75,10 +75,7 @@ public class EventTypeEntryContainer {
}
public boolean modelExists(String fieldName) {
- if (fieldName != null) {
- return modelMap.keySet().contains(fieldName.toUpperCase());
- }
- return false;
+ return fieldName != null && modelMap.keySet().contains(fieldName.toUpperCase());
}
public ListModel getModel(String fieldName) {
http://git-wip-us.apache.org/repos/asf/logging-chainsaw/blob/0a52dc99/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java b/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java
index 7479931..9cd329c 100644
--- a/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java
+++ b/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java
@@ -318,12 +318,8 @@ public class PluginPropertyEditorPanel extends JPanel {
public boolean isCellEditable(int rowIndex, int columnIndex) {
// TODO Determine if the property is one of the ones a User could edit
- if (columnIndex == 1) {
-
- return descriptors[rowIndex].getWriteMethod() != null;
- }
+ return columnIndex == 1 && descriptors[rowIndex].getWriteMethod() != null;
- return false;
}
/* (non-Javadoc)
http://git-wip-us.apache.org/repos/asf/logging-chainsaw/blob/0a52dc99/src/main/java/org/apache/log4j/db/DBAppender.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/log4j/db/DBAppender.java b/src/main/java/org/apache/log4j/db/DBAppender.java
index f1efbea..799d5b5 100644
--- a/src/main/java/org/apache/log4j/db/DBAppender.java
+++ b/src/main/java/org/apache/log4j/db/DBAppender.java
@@ -167,27 +167,23 @@ public class DBAppender extends AppenderSkeleton implements UnrecognizedElementH
}
public void activateOptions() {
- LogLog.debug("DBAppender.activateOptions called");
+ LogLog.debug("DBAppender.activateOptions called");
- if (connectionSource == null) {
- throw new IllegalStateException(
- "DBAppender cannot function without a connection source");
- }
+ if (connectionSource == null) {
+ throw new IllegalStateException(
+ "DBAppender cannot function without a connection source");
+ }
- sqlDialect = Util.getDialectFromCode(connectionSource.getSQLDialectCode());
- if (GET_GENERATED_KEYS_METHOD != null) {
- cnxSupportsGetGeneratedKeys = connectionSource.supportsGetGeneratedKeys();
- } else {
- cnxSupportsGetGeneratedKeys = false;
- }
- cnxSupportsBatchUpdates = connectionSource.supportsBatchUpdates();
- if (!cnxSupportsGetGeneratedKeys && (sqlDialect == null)) {
- throw new IllegalStateException(
- "DBAppender cannot function if the JDBC driver does not support getGeneratedKeys method *and* without a specific SQL dialect");
- }
-
- // all nice and dandy on the eastern front
- super.activateOptions();
+ sqlDialect = Util.getDialectFromCode(connectionSource.getSQLDialectCode());
+ cnxSupportsGetGeneratedKeys = GET_GENERATED_KEYS_METHOD != null && connectionSource.supportsGetGeneratedKeys();
+ cnxSupportsBatchUpdates = connectionSource.supportsBatchUpdates();
+ if (!cnxSupportsGetGeneratedKeys && (sqlDialect == null)) {
+ throw new IllegalStateException(
+ "DBAppender cannot function if the JDBC driver does not support getGeneratedKeys method *and* without a specific SQL dialect");
+ }
+
+ // all nice and dandy on the eastern front
+ super.activateOptions();
}
/**