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

[otaviojava]

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

    https://github.com/apache/tinkerpop/pull/919#discussion_r211940616
  
    --- 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 --
    
    That is the same previous behavior, but using StringBuilder this time.
    
https://docs.oracle.com/javase/8/docs/api/java/lang/StringBuilder.html#insert-int-java.lang.String-


---