From: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Date: Thu, 5 Nov 2015 21:07:22 +0000 (-0500)
Subject: statesystem: Remove ITmfStateInterval.getViewerEndTime
X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=cdaa40088b8345b7e78f064a7761ffffd0ca2269;p=deliverable%2Ftracecompass.git

statesystem: Remove ITmfStateInterval.getViewerEndTime

Nothing calls it and its semantics are not well defined.
Also, it makes little sense that the state system would have knowledge
about what views want.

Change-Id: Idff4d0bf24c4173c8cd8a67ac18753dcd6fb4ff4
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/59785
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI
---

diff --git a/statesystem/org.eclipse.tracecompass.statesystem.core/META-INF/MANIFEST.MF b/statesystem/org.eclipse.tracecompass.statesystem.core/META-INF/MANIFEST.MF
index 9c89f0a7b3..e5e675127d 100644
--- a/statesystem/org.eclipse.tracecompass.statesystem.core/META-INF/MANIFEST.MF
+++ b/statesystem/org.eclipse.tracecompass.statesystem.core/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: %Bundle-Name
 Bundle-Vendor: %Bundle-Vendor
-Bundle-Version: 1.0.0.qualifier
+Bundle-Version: 2.0.0.qualifier
 Bundle-Localization: plugin
 Bundle-SymbolicName: org.eclipse.tracecompass.statesystem.core;singleton:=true
 Bundle-Activator: org.eclipse.tracecompass.internal.statesystem.core.Activator
diff --git a/statesystem/org.eclipse.tracecompass.statesystem.core/pom.xml b/statesystem/org.eclipse.tracecompass.statesystem.core/pom.xml
index 8bfb1117f2..8e60c855a5 100644
--- a/statesystem/org.eclipse.tracecompass.statesystem.core/pom.xml
+++ b/statesystem/org.eclipse.tracecompass.statesystem.core/pom.xml
@@ -18,7 +18,6 @@
   </parent>
 
   <artifactId>org.eclipse.tracecompass.statesystem.core</artifactId>
-  <version>1.0.0-SNAPSHOT</version>
   <packaging>eclipse-plugin</packaging>
 
   <name>Trace Compass State System Core Plug-in</name>
diff --git a/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/internal/statesystem/core/backend/historytree/HTInterval.java b/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/internal/statesystem/core/backend/historytree/HTInterval.java
index c7ca11f8da..30e6eb809e 100644
--- a/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/internal/statesystem/core/backend/historytree/HTInterval.java
+++ b/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/internal/statesystem/core/backend/historytree/HTInterval.java
@@ -348,11 +348,6 @@ public final class HTInterval implements ITmfStateInterval, Comparable<HTInterva
         return end;
     }
 
-    @Override
-    public long getViewerEndTime() {
-        return end + 1;
-    }
-
     @Override
     public int getAttribute() {
         return attribute;
diff --git a/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/statesystem/core/interval/ITmfStateInterval.java b/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/statesystem/core/interval/ITmfStateInterval.java
index 05fa3ac55b..b3279219fd 100644
--- a/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/statesystem/core/interval/ITmfStateInterval.java
+++ b/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/statesystem/core/interval/ITmfStateInterval.java
@@ -39,17 +39,6 @@ public interface ITmfStateInterval {
      */
     long getEndTime();
 
-    /**
-     * In case the "real" end time of the interval is not exactly the same as
-     * the end time you want to show in views, you can implement this method to
-     * assign a different value that the viewer can use.
-     *
-     * If not, you can simply have it return the same as getEndTime().
-     *
-     * @return The end time that views should use
-     */
-    long getViewerEndTime();
-
     /**
      * Retrieve the quark of the attribute this state interval refers to
      *
diff --git a/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/statesystem/core/interval/TmfStateInterval.java b/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/statesystem/core/interval/TmfStateInterval.java
index 56e06288f6..be4e50e878 100644
--- a/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/statesystem/core/interval/TmfStateInterval.java
+++ b/statesystem/org.eclipse.tracecompass.statesystem.core/src/org/eclipse/tracecompass/statesystem/core/interval/TmfStateInterval.java
@@ -58,11 +58,6 @@ public final class TmfStateInterval implements ITmfStateInterval {
         return end;
     }
 
-    @Override
-    public long getViewerEndTime() {
-        return end + 1;
-    }
-
     @Override
     public int getAttribute() {
         return attribute;