[Releng] Make use of StringUtil.appendSeparator()

2 files changed, 7 insertions, 25 deletions

diff --git a/plugins/org.eclipse.emf.cdo.common/src/org/eclipse/emf/cdo/internal/common/AbstractRepositoryProperties.java b/plugins/org.eclipse.emf.cdo.common/src/org/eclipse/emf/cdo/internal/common/AbstractRepositoryProperties.java
index 2588993328..58010a7497 100644
--- a/plugins/org.eclipse.emf.cdo.common/src/org/eclipse/emf/cdo/internal/common/AbstractRepositoryProperties.java
+++ b/plugins/org.eclipse.emf.cdo.common/src/org/eclipse/emf/cdo/internal/common/AbstractRepositoryProperties.java
@@ -18,6 +18,7 @@ import org.eclipse.emf.cdo.common.util.CDOCommonUtil;
 import org.eclipse.emf.cdo.internal.common.messages.Messages;
 
 import org.eclipse.net4j.util.AdapterUtil;
+import org.eclipse.net4j.util.StringUtil;
 import org.eclipse.net4j.util.properties.Properties;
 import org.eclipse.net4j.util.properties.Property;
 
@@ -216,11 +217,7 @@ public abstract class AbstractRepositoryProperties<RECEIVER> extends Properties<
         CDOCommonRepository repository = getRepository(receiver);
         for (CDOID.ObjectType objectIDType : repository.getObjectIDTypes())
         {
-          if (builder.length() != 0)
-          {
-            builder.append(", "); //$NON-NLS-1$
-          }
-
+          StringUtil.appendSeparator(builder, ", "); //$NON-NLS-1$
           builder.append(objectIDType);
         }
 
diff --git a/plugins/org.eclipse.emf.cdo.common/src/org/eclipse/emf/cdo/spi/common/revision/BaseCDORevision.java b/plugins/org.eclipse.emf.cdo.common/src/org/eclipse/emf/cdo/spi/common/revision/BaseCDORevision.java
index 2109626600..bf87916213 100644
--- a/plugins/org.eclipse.emf.cdo.common/src/org/eclipse/emf/cdo/spi/common/revision/BaseCDORevision.java
+++ b/plugins/org.eclipse.emf.cdo.common/src/org/eclipse/emf/cdo/spi/common/revision/BaseCDORevision.java
@@ -46,6 +46,7 @@ import org.eclipse.emf.cdo.spi.common.branch.InternalCDOBranch;
 import org.eclipse.emf.cdo.spi.common.revision.InternalCDOList.ConfigurableEquality;
 
 import org.eclipse.net4j.util.ObjectUtil;
+import org.eclipse.net4j.util.StringUtil;
 import org.eclipse.net4j.util.om.trace.ContextTracer;
 import org.eclipse.net4j.util.om.trace.PerfTracer;
 
@@ -1297,41 +1298,25 @@ public abstract class BaseCDORevision extends AbstractCDORevision
 
     if ((flags & FROZEN_FLAG) != 0)
     {
-      if (builder.length() != 0)
-      {
-        builder.append("|");
-      }
-
+      StringUtil.appendSeparator(builder, '|');
       builder.append("FROZEN");
     }
 
     if ((flags & READ_PERMISSION_FLAG) != 0)
     {
-      if (builder.length() != 0)
-      {
-        builder.append("|");
-      }
-
+      StringUtil.appendSeparator(builder, '|');
       builder.append("READ");
     }
 
     if ((flags & WRITE_PERMISSION_FLAG) != 0)
     {
-      if (builder.length() != 0)
-      {
-        builder.append("|");
-      }
-
+      StringUtil.appendSeparator(builder, '|');
       builder.append("WRITE");
     }
 
     if ((flags & BYPASS_PERMISSION_CHECKS_FLAG) != 0)
     {
-      if (builder.length() != 0)
-      {
-        builder.append("|");
-      }
-
+      StringUtil.appendSeparator(builder, '|');
       builder.append("BYPASS_PERMISSION_CHECKS");
     }