[GitHub] tinkerpop pull request #919: String loop to String builder [master]

[otaviojava]

Github user otaviojava commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/919#discussion_r211950542
  
    --- Diff: 
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/DefaultTraversalMetrics.java
 ---
    @@ -208,14 +208,14 @@ private void handleNestedTraversals(final 
Traversal.Admin traversal, final Mutab
         private void appendMetrics(final Collection<? extends Metrics> 
metrics, final StringBuilder sb, final int indent) {
             // Append each StepMetric's row. indexToLabelMap values are 
ordered by index.
             for (Metrics m : metrics) {
    -            String rowName = m.getName();
    +            final StringBuilder metricName = new 
StringBuilder(m.getName());
     
                 // Handle indentation
                 for (int ii = 0; ii < indent; ii++) {
    -                rowName = "  " + rowName;
    +                metricName.insert(0, "  ");
                 }
    --- End diff --
    
    Do you mean this way?
    ```java
      private static String padLeft(final String text, final int amountToPad) {
            // not sure why this method needed to exist. stupid string format 
stuff and commons utilities wouldn't
            // work for some reason in the context this method was used above.
            final StringBuilder newText = new StringBuilder();
            for (int ix = 0; ix < amountToPad; ix++) {
                newText.append(" ");
            }
            newText.append(text);
            return newText.toString();
        }
    
    ```


---