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 9c89f0a7b3e7a2ac28d7d6f6e795b9a7f45f5a8e..e5e675127d83fd2eacffbe01a7e7ba77b0388f29 100644 (file)
--- 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 8bfb1117f2f254f2a30a84bc5bdd039ea0c0aa26..8e60c855a5760d746c35dfac0e2f65cb2617323e 100644 (file)
--- 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 c7ca11f8da0b376048b56039c3e3cdcbf9a4e23c..30e6eb809ebf158438d22d67c0eec96abe2294e0 100644 (file)
--- 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 05fa3ac55bbffbca5a04dd219bdd754b8a4ef462..b3279219fdebee0b5d5b0d908bf44f67a8ebc506 100644 (file)
--- 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 56e06288f6394db59cac78a739bcf678d07c6ba4..be4e50e878bcb52dc0e444e6c6f31fc3961b0985 100644 (file)
--- 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;