opentracing: avoid making a string several times in getSpanName

Just seek the third '/' in getSpanName

Change-Id: I239ef01d7f4c6c789559b3f953ecd46edc422171
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/138201
Tested-by: CI Bot
Reviewed-by: Simon Delisle <simon.delisle@ericsson.com>
Tested-by: Simon Delisle <simon.delisle@ericsson.com>

1 files changed, 4 insertions, 3 deletions

diff --git a/tracetypes/org.eclipse.tracecompass.incubator.opentracing.core/src/org/eclipse/tracecompass/incubator/internal/opentracing/core/analysis/spanlife/SpanLifeDataProvider.java b/tracetypes/org.eclipse.tracecompass.incubator.opentracing.core/src/org/eclipse/tracecompass/incubator/internal/opentracing/core/analysis/spanlife/SpanLifeDataProvider.java
index 75b4d345a..7ba5b0154 100644
--- a/tracetypes/org.eclipse.tracecompass.incubator.opentracing.core/src/org/eclipse/tracecompass/incubator/internal/opentracing/core/analysis/spanlife/SpanLifeDataProvider.java
+++ b/tracetypes/org.eclipse.tracecompass.incubator.opentracing.core/src/org/eclipse/tracecompass/incubator/internal/opentracing/core/analysis/spanlife/SpanLifeDataProvider.java
@@ -286,9 +286,10 @@ public class SpanLifeDataProvider extends AbstractTimeGraphDataProvider<@NonNull
     }
 
     private static String getSpanName(String attributeName) {
-        String spanNameAndId = attributeName.substring(0, attributeName.lastIndexOf('/'));
-        spanNameAndId = attributeName.substring(0, spanNameAndId.lastIndexOf('/'));
-        return spanNameAndId.substring(0, spanNameAndId.lastIndexOf('/'));
+        int slashPos = attributeName.lastIndexOf('/');
+        slashPos = attributeName.lastIndexOf('/', slashPos);
+        slashPos = attributeName.lastIndexOf('/', slashPos);
+        return attributeName.substring(0, Math.max(0, slashPos));
     }
 
     private static String getSpanId(String attributeName) {