Skip to content

Commit 01e8c47

Browse files
committed
Reuse empty array constants.
Refactor empty array constants.
1 parent 543c441 commit 01e8c47

File tree

3 files changed

+3
-4
lines changed

3 files changed

+3
-4
lines changed

log4j-core/src/test/java/org/apache/logging/log4j/core/pattern/PatternParserTest2.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,7 @@ private void parse(final String pattern, final boolean convert, final StringBuil
6262
final List<PatternConverter> converters = new ArrayList<>();
6363
final List<FormattingInfo> fields = new ArrayList<>();
6464
parser0.parse(pattern, converters, fields, false, false, convert);
65-
final FormattingInfo[] infoArray = new FormattingInfo[fields.size()];
66-
final FormattingInfo[] patternFields = fields.toArray(infoArray);
65+
final FormattingInfo[] patternFields = fields.toArray(FormattingInfo.EMPTY_ARRAY);
6766
final ArrayPatternConverter[] converterArray = new ArrayPatternConverter[converters.size()];
6867
final ArrayPatternConverter[] patternConverters = converters.toArray(converterArray);
6968
formatFileName(patternConverters, patternFields, buf, date, i);

log4j-slf4j-impl/src/test/java/org/apache/logging/slf4j/LoggerContextTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public void testCleanup() throws Exception {
3535
Log4jLoggerFactory factory = (Log4jLoggerFactory) StaticLoggerBinder.getSingleton().getLoggerFactory();
3636
factory.getLogger("test");
3737
Set<LoggerContext> set = factory.getLoggerContexts();
38-
LoggerContext ctx1 = set.toArray(new LoggerContext[0])[0];
38+
LoggerContext ctx1 = set.toArray(LoggerContext.EMPTY_ARRAY)[0];
3939
assertTrue("LoggerContext is not enabled for shutdown", ctx1 instanceof LifeCycle);
4040
((LifeCycle) ctx1).stop();
4141
set = factory.getLoggerContexts();

log4j-slf4j18-impl/src/test/java/org/apache/logging/slf4j/LoggerContextTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public void testCleanup() throws Exception {
3535
Log4jLoggerFactory factory = (Log4jLoggerFactory) LoggerFactory.getILoggerFactory();
3636
factory.getLogger("test");
3737
Set<LoggerContext> set = factory.getLoggerContexts();
38-
LoggerContext ctx1 = set.toArray(new LoggerContext[0])[0];
38+
LoggerContext ctx1 = set.toArray(LoggerContext.EMPTY_ARRAY)[0];
3939
assertTrue("LoggerContext is not enabled for shutdown", ctx1 instanceof LifeCycle);
4040
((LifeCycle) ctx1).stop();
4141
set = factory.getLoggerContexts();

0 commit comments

Comments
 (0)