You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by db...@apache.org on 2014/05/01 00:16:38 UTC

git commit: trivial StringBuilder usage fixes

Repository: cassandra
Updated Branches:
  refs/heads/trunk 5a9ca7765 -> 43f19eb57


trivial StringBuilder usage fixes


Project: http://git-wip-us.apache.org/repos/asf/cassandra/repo
Commit: http://git-wip-us.apache.org/repos/asf/cassandra/commit/43f19eb5
Tree: http://git-wip-us.apache.org/repos/asf/cassandra/tree/43f19eb5
Diff: http://git-wip-us.apache.org/repos/asf/cassandra/diff/43f19eb5

Branch: refs/heads/trunk
Commit: 43f19eb576004ac7694e76425d37cf876c01673c
Parents: 5a9ca77
Author: Dave Brosius <db...@mebigfatguy.com>
Authored: Wed Apr 30 18:15:43 2014 -0400
Committer: Dave Brosius <db...@mebigfatguy.com>
Committed: Wed Apr 30 18:15:43 2014 -0400

----------------------------------------------------------------------
 .../src/org/apache/cassandra/stress/settings/GroupedOptions.java | 2 +-
 .../stress/src/org/apache/cassandra/stress/settings/Legacy.java  | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cassandra/blob/43f19eb5/tools/stress/src/org/apache/cassandra/stress/settings/GroupedOptions.java
----------------------------------------------------------------------
diff --git a/tools/stress/src/org/apache/cassandra/stress/settings/GroupedOptions.java b/tools/stress/src/org/apache/cassandra/stress/settings/GroupedOptions.java
index d0e63b0..e351521 100644
--- a/tools/stress/src/org/apache/cassandra/stress/settings/GroupedOptions.java
+++ b/tools/stress/src/org/apache/cassandra/stress/settings/GroupedOptions.java
@@ -85,7 +85,7 @@ public abstract class GroupedOptions
             }
             firstRow = false;
 
-            StringBuilder sb = new StringBuilder("Usage: " + command);
+            StringBuilder sb = new StringBuilder("Usage: ").append(command);
             for (Option option : grouping.options())
             {
                 sb.append(" ");

http://git-wip-us.apache.org/repos/asf/cassandra/blob/43f19eb5/tools/stress/src/org/apache/cassandra/stress/settings/Legacy.java
----------------------------------------------------------------------
diff --git a/tools/stress/src/org/apache/cassandra/stress/settings/Legacy.java b/tools/stress/src/org/apache/cassandra/stress/settings/Legacy.java
index 5ddb346..eb04290 100644
--- a/tools/stress/src/org/apache/cassandra/stress/settings/Legacy.java
+++ b/tools/stress/src/org/apache/cassandra/stress/settings/Legacy.java
@@ -216,12 +216,12 @@ public class Legacy implements Serializable
             {
                 StringBuilder rep = new StringBuilder();
                 if (cmd.hasOption("R"))
-                    rep.append("strategy=" + cmd.getOptionValue("R"));
+                    rep.append("strategy=").append(cmd.getOptionValue("R"));
                 if (cmd.hasOption("l"))
                 {
                     if (rep.length() > 0)
                         rep.append(",");
-                    rep.append("factor=" + cmd.getOptionValue("l"));
+                    rep.append("factor=").append(cmd.getOptionValue("l"));
                 }
                 if (cmd.hasOption("O"))
                 {