This is an automated email from the ASF dual-hosted git repository.
vy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git
The following commit(s) were added to refs/heads/master by this push:
new 241dbcc Simplify conditional expression. (#531)
241dbcc is described below
commit 241dbcc360500bc4a83b693f5087de24a6c4e48b
Author: Arturo Bernal <[email protected]>
AuthorDate: Thu Jun 24 20:08:57 2021 +0200
Simplify conditional expression. (#531)
---
log4j-1.2-api/src/main/java/org/apache/log4j/Category.java | 2 +-
.../logging/log4j/core/appender/rolling/RollingFileManager.java | 2 +-
.../java/org/apache/logging/log4j/core/filter/BurstFilter.java | 6 +-----
.../org/apache/logging/log4j/core/tools/picocli/CommandLine.java | 8 ++++----
.../src/main/java/org/apache/logging/slf4j/Log4jMarker.java | 4 ++--
.../src/main/java/org/apache/logging/slf4j/Log4jMarker.java | 4 ++--
6 files changed, 11 insertions(+), 15 deletions(-)
diff --git a/log4j-1.2-api/src/main/java/org/apache/log4j/Category.java
b/log4j-1.2-api/src/main/java/org/apache/log4j/Category.java
index a17697a..c8a949f 100644
--- a/log4j-1.2-api/src/main/java/org/apache/log4j/Category.java
+++ b/log4j-1.2-api/src/main/java/org/apache/log4j/Category.java
@@ -404,7 +404,7 @@ public class Category {
}
public boolean getAdditivity() {
- return isCoreAvailable ? CategoryUtil.isAdditive(logger) : false;
+ return isCoreAvailable && CategoryUtil.isAdditive(logger);
}
public void setAdditivity(final boolean additivity) {
diff --git
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
index 204a9b5..7bcbc15 100644
---
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
+++
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/RollingFileManager.java
@@ -652,7 +652,7 @@ public class RollingFileManager extends FileManager {
try {
FileUtils.makeParentDirs(file);
- final boolean created = data.createOnDemand ? false :
file.createNewFile();
+ final boolean created = !data.createOnDemand &&
file.createNewFile();
LOGGER.trace("New file '{}' created = {}", name, created);
} catch (final IOException ioe) {
LOGGER.error("Unable to create file " + name, ioe);
diff --git
a/log4j-core/src/main/java/org/apache/logging/log4j/core/filter/BurstFilter.java
b/log4j-core/src/main/java/org/apache/logging/log4j/core/filter/BurstFilter.java
index b815ee4..254d967 100644
---
a/log4j-core/src/main/java/org/apache/logging/log4j/core/filter/BurstFilter.java
+++
b/log4j-core/src/main/java/org/apache/logging/log4j/core/filter/BurstFilter.java
@@ -272,11 +272,7 @@ public final class BurstFilter extends AbstractFilter {
final LogDelay logDelay = (LogDelay) o;
- if (expireTime != logDelay.expireTime) {
- return false;
- }
-
- return true;
+ return expireTime == logDelay.expireTime;
}
@Override
diff --git
a/log4j-core/src/main/java/org/apache/logging/log4j/core/tools/picocli/CommandLine.java
b/log4j-core/src/main/java/org/apache/logging/log4j/core/tools/picocli/CommandLine.java
index 06aa387..bce2107 100644
---
a/log4j-core/src/main/java/org/apache/logging/log4j/core/tools/picocli/CommandLine.java
+++
b/log4j-core/src/main/java/org/apache/logging/log4j/core/tools/picocli/CommandLine.java
@@ -2253,7 +2253,7 @@ public class CommandLine {
args.push(value); // we don't consume the value
}
final Boolean currentValue = (Boolean) field.get(command);
- value = String.valueOf(currentValue == null ? true :
!currentValue); // #147 toggle existing boolean value
+ value = String.valueOf(currentValue == null ||
!currentValue); // #147 toggle existing boolean value
}
}
if (noMoreValues && value == null) {
@@ -2967,10 +2967,10 @@ public class CommandLine {
}
cls = cls.getSuperclass();
}
- sortOptions = (sortOptions == null) ? true :
sortOptions;
- abbreviateSynopsis = (abbreviateSynopsis == null) ? false :
abbreviateSynopsis;
+ sortOptions = sortOptions == null || sortOptions;
+ abbreviateSynopsis = abbreviateSynopsis != null &&
abbreviateSynopsis;
requiredOptionMarker = (requiredOptionMarker == null) ? ' ' :
requiredOptionMarker;
- showDefaultValues = (showDefaultValues == null) ? false :
showDefaultValues;
+ showDefaultValues = showDefaultValues != null && showDefaultValues;
synopsisHeading = (synopsisHeading == null) ? "Usage: "
: synopsisHeading;
commandListHeading = (commandListHeading == null) ?
"Commands:%n" : commandListHeading;
separator = (separator == null) ?
DEFAULT_SEPARATOR : separator;
diff --git
a/log4j-slf4j-impl/src/main/java/org/apache/logging/slf4j/Log4jMarker.java
b/log4j-slf4j-impl/src/main/java/org/apache/logging/slf4j/Log4jMarker.java
index f0c5b6c..c8976f2 100644
--- a/log4j-slf4j-impl/src/main/java/org/apache/logging/slf4j/Log4jMarker.java
+++ b/log4j-slf4j-impl/src/main/java/org/apache/logging/slf4j/Log4jMarker.java
@@ -63,7 +63,7 @@ public class Log4jMarker implements Marker {
@Override
public boolean contains(final String s) {
- return s != null ? this.marker.isInstanceOf(s) : false;
+ return s != null && this.marker.isInstanceOf(s);
}
@Override
@@ -127,6 +127,6 @@ public class Log4jMarker implements Marker {
@Override
public boolean remove(final Marker marker) {
- return marker != null ?
this.marker.remove(MarkerManager.getMarker(marker.getName())) : false;
+ return marker != null &&
this.marker.remove(MarkerManager.getMarker(marker.getName()));
}
}
diff --git
a/log4j-slf4j18-impl/src/main/java/org/apache/logging/slf4j/Log4jMarker.java
b/log4j-slf4j18-impl/src/main/java/org/apache/logging/slf4j/Log4jMarker.java
index 00351f9..3093185 100644
--- a/log4j-slf4j18-impl/src/main/java/org/apache/logging/slf4j/Log4jMarker.java
+++ b/log4j-slf4j18-impl/src/main/java/org/apache/logging/slf4j/Log4jMarker.java
@@ -63,7 +63,7 @@ class Log4jMarker implements Marker {
@Override
public boolean contains(final String s) {
- return s != null ? this.marker.isInstanceOf(s) : false;
+ return s != null && this.marker.isInstanceOf(s);
}
@Override
@@ -127,6 +127,6 @@ class Log4jMarker implements Marker {
@Override
public boolean remove(final Marker marker) {
- return marker != null ?
this.marker.remove(MarkerManager.getMarker(marker.getName())) : false;
+ return marker != null &&
this.marker.remove(MarkerManager.getMarker(marker.getName()));
}
}
