analysis: Update unit formatter for segment store

Change-Id: I5bd654504e4fb91f9c5b66ffac9737dd69e1ad5a
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/64302
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

diff --git a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/SubSecondTimeWithUnitFormat.java b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/SubSecondTimeWithUnitFormat.java
index a04a3ea3ffc5dfc673080122ddc5d8077619b45f..84f064e958992763de2c4fe0df789d5c1ae23c82 100644 (file)
--- a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/SubSecondTimeWithUnitFormat.java
+++ b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/SubSecondTimeWithUnitFormat.java
@@ -58,16 +58,19 @@ public final class SubSecondTimeWithUnitFormat extends Format {
                 return appender == null ? new StringBuffer() : NonNullUtils.checkNotNull(appender.append("---")); //$NON-NLS-1$
             }
             String unit = NANOSECONDS;
-            if (formattedTime > NANOS_PER_SEC) {
+            if (formattedTime >= NANOS_PER_SEC) {
                 unit = SECONDS;
                 formattedTime /= NANOS_PER_SEC;
-            } else if (formattedTime > NANOS_PER_MILLI) {
+            } else if (formattedTime >= NANOS_PER_MILLI) {
                 unit = MILLISECONDS;
                 formattedTime /= NANOS_PER_MILLI;
-            } else if (formattedTime > NANOS_PER_MICRO) {
+            } else if (formattedTime >= NANOS_PER_MICRO) {
                 unit = MICROSECONDS;
                 formattedTime /= NANOS_PER_MICRO;
             }
+            if (formattedTime == 0) {
+                return appender == null ? new StringBuffer() : NonNullUtils.checkNotNull(appender.append(0));
+            }
             String timeString = unit.equals(NANOSECONDS) ? Long.toString((long) formattedTime) : fDecimalFormat.format(formattedTime);
             return appender == null ? new StringBuffer() : NonNullUtils.checkNotNull(appender.append(timeString).append(' ').append(unit));
         }