From: Genevieve Bastien <gbastien+lttng@versatic.net>
Date: Sat, 15 Oct 2016 13:48:55 +0000 (-0400)
Subject: timing: avoid a division by zero with scatter viewer
X-Git-Url: http://git.efficios.com/?p=deliverable%2Ftracecompass.git;a=commitdiff_plain;h=61d830e8d0cc9a743ffa861134454bb9d0b56392

timing: avoid a division by zero with scatter viewer

Change-Id: Iee5bce80f07ae0231801b3be17d5cbad2d665e48
Signed-off-by: Genevieve Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/83354
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
---

diff --git a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/scatter/AbstractSegmentStoreScatterGraphViewer.java b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/scatter/AbstractSegmentStoreScatterGraphViewer.java
index 1017c9f581..fe2731c4da 100644
--- a/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/scatter/AbstractSegmentStoreScatterGraphViewer.java
+++ b/analysis/org.eclipse.tracecompass.analysis.timing.ui/src/org/eclipse/tracecompass/analysis/timing/ui/views/segmentstore/scatter/AbstractSegmentStoreScatterGraphViewer.java
@@ -108,7 +108,7 @@ public abstract class AbstractSegmentStoreScatterGraphViewer extends TmfCommonXL
             final long startTime = fCurrentRange.getStartTime().getValue();
             final long endTime = fCurrentRange.getEndTime().getValue();
             fPixelStart = startTime;
-            fPixelSize = (endTime - startTime) / MAX_POINTS;
+            fPixelSize = Math.max(1, (endTime - startTime) / MAX_POINTS);
             final Iterable<ISegment> intersectingElements = segStore.getIntersectingElements(startTime, endTime);
 
             final List<ISegment> list = convertIterableToList(intersectingElements, statusMonitor);