commit 06260e30b99e92f09e9579f70d4ac0a6430d9f59
author Edwin Kempin <ekempin@google.com> Wed May 16 16:38:33 2018 +0200
committer Edwin Kempin <ekempin@google.com> Wed May 16 16:38:33 2018 +0200
tree 11fc8f1a501e63c1ee9ebf8f5f06b367b9e8ee38
parent 3a9e55643a9762cfad8ff2797f40c99fe3896517

PostGpgKeys: Remove unneeded use of Joiner

We can just rely on the toString formatting of the List. This makes the
code more readable.

Change-Id: I98f0da6b5180198b9fcf0e45271e12e9ab182a52
Signed-off-by: Edwin Kempin <ekempin@google.com>

java/com/google/gerrit/gpg/server/PostGpgKeys.java

diff --git a/java/com/google/gerrit/gpg/server/PostGpgKeys.java b/java/com/google/gerrit/gpg/server/PostGpgKeys.java
index 6e638dc..4b92ec3 100644
--- a/java/com/google/gerrit/gpg/server/PostGpgKeys.java
+++ b/java/com/google/gerrit/gpg/server/PostGpgKeys.java
@@ -259,9 +259,10 @@
 
     if (accountStates.size() > 1) {
       StringBuilder msg = new StringBuilder();
-      msg.append("GPG key ").append(extIdKey.get()).append(" associated with multiple accounts: ");
-      Joiner.on(", ")
-          .appendTo(msg, Lists.transform(accountStates, AccountState.ACCOUNT_ID_FUNCTION));
+      msg.append("GPG key ")
+          .append(extIdKey.get())
+          .append(" associated with multiple accounts: ")
+          .append(Lists.transform(accountStates, AccountState.ACCOUNT_ID_FUNCTION));
       throw new IllegalStateException(msg.toString());
     }