Author: ggregory
Date: Tue May 13 15:36:30 2014
New Revision: 1594251
URL: http://svn.apache.org/r1594251
Log:
Refactor magic strings into a constant.
Modified:
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java
Modified:
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java
URL:
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java?rev=1594251&r1=1594250&r2=1594251&view=diff
==============================================================================
---
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java
(original)
+++
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/lookup/StrSubstitutor.java
Tue May 13 15:36:30 2014
@@ -25,6 +25,7 @@ import java.util.Map;
import java.util.Properties;
import org.apache.logging.log4j.core.LogEvent;
+import org.apache.logging.log4j.util.Strings;
/**
* Substitutes variables within a string by values.
@@ -1116,7 +1117,7 @@ public class StrSubstitutor {
*/
public void appendWithSeparators(final StringBuilder sb, final Iterable<?>
iterable, String separator) {
if (iterable != null) {
- separator = (separator == null ? "" : separator);
+ separator = (separator == null ? Strings.EMPTY : separator);
final Iterator<?> it = iterable.iterator();
while (it.hasNext()) {
sb.append(it.next());