ss: Bug 475300: Fix inconsistent TreeMapStore due to value comparators

Add Comparable to the ISegment interface for segments to specify
a natural ordering. This ordering will then be used by the tree map
to "break ties" when segments have the same start and end times.

Also removed the getElementAtIndex() method. It was not doing what
the Javadoc said it did, and it does not seem to be useful for the
analysis as it is.

Change-Id: Ic65f50b12f6e94c59d4a60f0e96d9c78ee04b741
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/54146
Reviewed-by: Hudson CI

tmf: Support resuming the trace index

This allows resuming the indexing job after a trace was closed while the
indexing was not finished.
This also allows to index new content that was added to a trace, even if the
content was added after closing the trace and the indexer was finished.

Change-Id: I58296c0114a9f495605836f8c6df6dfceb424654
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/53851
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Hudson CI

tmf: make piecharts not visible when no trace opened

Change-Id: Id59ec9bb8a6c861b0de98cb4a5072f075add85b0
Signed-off-by: Alexis Cabana-Loriaux <alex021994@gmail.com>
Reviewed-on: https://git.eclipse.org/r/54171
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Bug 473280: Internalize TMF Statistics UI API

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

ctf: Lose reference to StreamInputReader in EventDefinition

StreamInputReader is AutoCloseable, it is better to not have a
reference to it in the event.

Change the constructor to require the CPU number instead of the
whole StreamInputReader, as it is the only thing that the
EventDefinition uses.

While we are changing most calls, we can merge both versions of
the constructor into one, since they only differ by one @Nullable
parameter.

Change-Id: Ibeae8e7ff7e4f78ab1f11a24d61c739706bfc881
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/53327
Reviewed-by: Hudson CI

ctf: accelerate eventheaderdefinition timestamp parsing

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

ctf: update api to 2.0 baseline

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

tmf: extract UpdateJob class and introduce TmfPieChartViewer

Change-Id: I0371ed878bce9e3121eed98141248dac80dec95d
Signed-off-by: Alexis Cabana-Loriaux <alex021994@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53160
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

releng: update feature versions to 2.0

Change-Id: I891313eae9fb1428d6007f08b46939edf33bcdca
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/54091
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Add grouping separator in indexer job

This makes it easier to read big numbers and track the progress of the job.

Change-Id: I2fc4329b378e71c78a738200b5c0886e183118c6
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/53849
Reviewed-by: Hudson CI

ctf: Update CtfTmfContextTest

- Dispose the trace after the test
- Remove unused array
- Avoid extra expected value due to math rounding

Change-Id: I46e8fdff95dd3e39d12b786d5efc4560b46fefe5
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53357
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

[CTF] Events wrongly parsed as Lost Events

In CTFStreamInputPacketReader, setCurrentPacket can be called multiple times
without necessarily calling readNextEvent, which means that the cached value
for fHasLost can be wrong the next time readNextEvent is called.

Bug: 475007
Change-Id: I36c22c22c144cb8307403f93c76c996c60ff52aa
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/53811
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

Replace a few Boolean.valueOf(foo).booleanValue() with Boolean.parseBoolean

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

tmf: Add indexer tracing option and add some trace points

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

tmf: Remove unused filter methods in time graph control

The methods getTraceFilter() and getFilteredOut() were not used and
their implementation did not return a valid result.

Change-Id: Id1d330acab4e36a80259bfead3eccbeb3ffce82e
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53587
Reviewed-by: Hudson CI

releng: Add a Maven profile to run a custom test suite in alltests

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

tmf: Make StandardImportGzipTraceTest more robust

In a build, I got the failure:
"IllegalArgumentException: The row number (2) is more than the
number of rows(1) in the table"
http://eclip.se/5i

I think it's a case of the assertion hapenning before the table
was completely filled. Adding a few waitUntil should fix it.

Change-Id: I89b15a29bd0ae9f64f213fa0ffa715566127650a
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/53915
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Bug 474710: Time graph combo expanded state is not synchronized

When a time graph combo is refreshed, newly discovered items are
expanded to the auto-expand level in the time graph viewer, and the tree
viewer item expanded states are synchronized with those of the time
graph viewer. The expanded state of existing items is no longer affected
by the call to refresh().

Change-Id: I709681ef488e7d22991d5b7384c667532afd3410
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53558
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Add a timeout to AnalysisModuleTest

Change-Id: Ieb5c436b53760002ce3e35769bda2641ade92a4b
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51994
Reviewed-by: Hudson CI

build: add minimum version to guava references in tmf.remote.core

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

tmf: Use basic comparators in TmfSimpleTableViewer

- Make the method createColumn() public and remove createColumns()
  so that the class can be using without needing to be extended.

- Remove ViewerCompoundComparator. We can use basic
  java.util.Comparator's in the analysis and the view, all the
  way until we convert them to ViewerComparator's to pass to the
  TableViewer object.
  This will allow us to re-use comparators defined in the core
  plugin. Chaining of multiple comparators can be done using
  Guava's Ordering.compound() method.

- Support DeferredContentProvider which must be supplied with the
  sort order and ignores the sorter set on the table viewer.
  Remove the method sortOrderChanged().

Change-Id: I3acd9e79e5a2298707e67cd668c17cca1e428234
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53584
Reviewed-by: Hudson CI

analysis: Add graph ui plug-in skeleton

Creating the skeleton early in the change set avoids the plug-in project
directory to disappear when rebasing, which otherwise causes Eclipse to
bail.

Change-Id: I631c7768d085f1ea5daeaa80d79f70672b988da7
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53338
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

lttng: Split UST event names in a separate interface

Remove the definitions pertaining to event and field names in
UstMemoryStrings, and move them to a global interface.

These names can change from one version to the other of the
tracer, so this is the first step in abstracting away the actual
trace event names from the "concepts" the library and analysis
agree upon.

Change-Id: Ia8ccf528029cc8cd3e5de13e4e7041ad6e89eeae
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/50312
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

ctf: Fix leaking context in CtfTmfTrace.initTrace()

The context used to read the first event of the trace is never disposed.
Its trace reader is auto-closed after registering the event types.

This leaves the context with its closed trace reader in the iterator
manager. If the iterator manager becomes full, the trace reader can be
reused for another context but this will fail since it's been closed.

Change-Id: Iab224db1349d2ded5cb0f2d69b7b418a53e5eca0
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53332
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

Bug 473441: Fix concurrency issue in ThreadedHistoryTreeBackend

We add a blockingPeek() method to the BBQ, which allows blocking
on the queue without removing elements from it. This allows
iterating threads doing query to continue seeing the element
while it is being inserted.

The implementation is changed so that the output buffer remains in the
inner queue until it is empty. Only then is it removed from the inner
queue and the next head of the queue becomes the output buffer.

The iterator is fixed to be a true descending iterator without
duplications and that properly handles concurrent modification of the
inner queue.

The size() method is added.

Change-Id: I9e708824748a625010aded82a3198f096c1fc427
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52640
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Fix some sonar warnings (naming conventions)

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

ss: add unit tests for the segment store

Change-Id: Ic29ccc34190a4f09723dc259ce79da4ec213b84a
Signed-off-by: France Lapointe Nguyen <francelap@gmail.com>
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51279
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Fix generics warning in ViewerCompoundComparator

We extend a class that uses Java 1.4 syntax, but in the 1.5+
world, Comparator takes a generic type.

Change-Id: Ie40c8ce4a22d6c1611d53d16dbb30d1f01e6ca22
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/53430
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI

swtbot: Fix CopyToClipboardTest

The table sometimes did not react fast enough to the selection to remove
the Copy to Clipboard item from its context menu. The condition is
changed to wait until the item is no longer found and won't fail even if
the item is still found for a short time.

Also temporarily fixed to handle the consequences of SWTBot Bug 474063.

Change-Id: I6db683abbcec3ec86d4b70c67b6abcf588cd5783
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53426
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

tmf.ui: add Comparators and refresh content to Simpletable

Change-Id: Ib5fc283a0f78ca86bd7b9ce50764bc94739748ea
Signed-off-by: France Lapointe Nguyen <francelap@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52254
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>

ss: Provide a basic implementation of ISegment

Change-Id: Ic66504a2b136ac00c21a37ac526b09db4dba32db
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/52816
Reviewed-by: France Lapointe Nguyen <francelap@gmail.com>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ss: Also clear the size of the TreeMapStore in its dispose()

Change-Id: I9c69017fea1fae562b8e693300672bdff7a5ad7b
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/52813
Reviewed-by: France Lapointe Nguyen <francelap@gmail.com>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Temporarily change @since 1.1 tags to @since 2.0

These should be reverted back to @since 1.1 when that baseline is out.

Change-Id: Icb415848daedcc526a41ce203665b4cb14f9bdd2
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/53324
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ss: Fix wrong indexing in TreeMapStore

Change-Id: I348ba5af90aecd237ce057e077ca5f081631c6f1
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/52752
Reviewed-by: France Lapointe Nguyen <francelap@gmail.com>
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ctf: add test trace for dynamic scopes, timestamps in fields, and empty streams

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

tmf: Remove unused batch import trace wizard source code

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

analysis: Make ResourcesView extend AbstractStateSystemTimeGraphView

Change-Id: If12898c07fede5b9f8a53127fcc87a82280ec678
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52967
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

analysis: Make ControlFlowView extend AbstractStateSystemTimeGraphView

Change-Id: I2a721725c271f40092ae4e988c935e324efdfbfe
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52966
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Add an abstract state system time graph view

This subclass of the abstract time graph view can be used when the time
events are built using a state system. The full states of the state
system are first queried chronologically for the whole time range using
a time resolution, and the list of full states is kept in memory and
reused by every time graph entry to build its time event list. This
avoids repeatedly loading the state system nodes from disk for each time
graph entry, which can be very slow if the full time range queries
require more nodes than is available in the state system cache.

Change-Id: I7dcac5d546a84462dba0e5a0cf320ba18d3617f5
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52965
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Allow time graph zoomed event list to be built incrementally

Instead of being replaced with a whole new list in one operation, time
events can now be incrementally merged into the zoomed event list.

The time graph entry adds a method to add time events to the existing
zoomed event list, replacing the last event if necessary.

NullTimeEvent is updated to correctly split without losing its type.

Change-Id: Icf51199483244a61580592d3f193ef460eded655
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52964
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

ctf: fixes tsdl in verbose metadata

This patch does many things that are tightly coupled. Splitting up IOStructGen will help making more
atomic patches here.

* Make enums and floats update their endianness if a trace declares it.
* Make variants register children even if typedeffed
* Make enums retain children when endianness is swapped
* NullCheck event fields before iterating over them

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

ctf: support timestamps in event contexts

Someone may put timestamps in contexts, this is handled as a last resort lookup

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

TMF: Close sequence diagram view after sequence diagram tests

This will make sure that the sequence diagram view is not open in
other tests while running all JUnit tests.

This should fix failures when running all JUnit tests in a loop.

Change-Id: I13109a0edff4d08f228d0dc218b8a5564accc067
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/52983
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

lttng control: Fix number of connection nodes not zero after deletion.

It is possible that the connections are saved due to the auto-save
feature of the workbench which calls ControlView.saveState(IMemento).
This can happen at any time (e.g. when calling delay()).

When showing the view above ControlView.init(IMemento) is called which
restores saved connections.

The tests require that the ControlView is empty. So we remove all the
connection nodes from the root when initializing the tests.

Change-Id: If94dd42cd3ba4bc3de1ea7bec959cc9117b91582
Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/52970
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

ctf: Fix priority list containing closed streams

When a CTFTraceReader was closed, the priority list was not
emptied. The hasMoreEvents() method thus returned true in
cases where the CTFTraceReader was closed before the end of
the trace. This patch fixes that behavior.

Change-Id: I43f9ad60ddf67bfe0561834e32d6e5872fcf7b55
Signed-off-by: Raphaël Beamonte <raphael.beamonte@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52890
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

ctf: make metadata field names respect coding style.

Change-Id: I871b8ce03eb9adc7748f07e25bb18e6439a46492
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/50674
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI

releng: Upgrade to Tycho 0.23.1

See
https://wiki.eclipse.org/Tycho/Release_Notes/0.23#New_and_Noteworthy

Change-Id: I767ceca42c533a3300348309f346d49d631352b3
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51762
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>

ctf: Fix closed iterators still reattributed in CtfIteratorManager

CtfIterator extends CTFTraceReader which implements AutoCloseable,
making it closing its streams when used in a try-block.
CtfIteratorManager reattributes randomly the iterators when its
ArrayList contains the defined maximum number of elements. It thus
sometimes reattributes a CtfIterator with closed streams that
will never be reopened. This patch allows to verify if the
reattributed iterator has stream input readers before
reattributing it, and, if it's not the case, replace it with
a fresh new CtfIterator.

Change-Id: I67ff06280381529a01bac8dc0a596ca3e876754d
Signed-off-by: Raphaël Beamonte <raphael.beamonte@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52868
Reviewed-by: Hudson CI
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ctf: add support for empty streams in a trace

This can handle "touch"ed files in a trace folder, or 0 byte streams due to poor setting up
of lttng-tools.

Change-Id: Ib9916c92b8728098f9d844c88baab7a1c068d063
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/52548
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Fix FetchRemoteTracesTest on Mac

1) Opening the Preferences through the menu doesn't work...anymore?
I have gone back to Eclipse 4.4 and SWTBot 2.2.1 to see if it worked and it doesn't.
I suspect it might be a change in OSX but I don't have access to an older one.
I am also not 100% sure this ever worked but I thought it did.

2) We need to wait for the Preferences dialog to be active before looking
for the tree in it. That seems like a good fix for all platforms.

Change-Id: Ib9b3ed6edee75d368d3238451dc57a982f65ee0a
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51861
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Wrong menu manager passed to appendToRawPopupMenu

Change-Id: Iee28118fa0711412295bd9c1c892e8a70e4da7e9
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52748
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

ss: Fix a bug where history tree node utilization is inaccurate

When a node is read from a file, the size of interval section is left at 0,
so using that metric to monitor the performance of the state system was
misleading.

Change-Id: I885a41ae744ea7f38dd25f1adce42268fe2fbd6f
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/52681
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-by: Hudson CI

Update Oomph setup for 4.5 target and new 1.0.0 baseline

Change-Id: I3279d34c1b2aa592940bc0c816c8ce5bc2e83efd
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/52678
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Fix TmfEventField#equals()

Since the getter methods are not final, they can be extended
by sub-classes. This means the equals() and hashCode() method
should use these instead of the direct fields.

In the case of fFields however, since it is not exposed
directly, we can mark its separate getter methods final, and
use the class attribute in equals() safely.

We also need to handle the case where the Object value could be
implemented as a [] array of any type. This will require special
care, since standard Object.equals() will not check the contents.

Change-Id: I21f20ebee9a7c6485df9d16859f736b8ca961587
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51213
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>

Analysis: Add unit tests for the graph base classes

Change-Id: I2fa96a924d17e760191906fb75aa5a8e1a29876a
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/41478
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

Analysis: Add the dependency graph plugin and base classes

Adds a new plugin: org.eclipse.tracecompass.analysis.graph.core and the base
classes used to represent a dependency graph.

Change-Id: I8bd45263cbf02b42d62b44868409b50ac46125ac
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Signed-off-by: Francis Giraldeau <francis.giraldeau@gmail.com>
Reviewed-on: https://git.eclipse.org/r/41477
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

common: Add ObjectUtils with static methods deepEquals and deepHashCode

These methods allow equals() and hashCode() to consider arbitrary
objects which may be arrays or collections of arrays with equal elements
to be equal, recursively.

Change-Id: Ic33f6451f5b535989ad8bed1cc35095bff500b0a
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52187
Reviewed-by: Hudson CI
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: fix parser not parsing "thing := keyword typealias;" regression

This regression was introduced when dynamic scopes added stricter checks
to tsdl parsing. The checks are good, this fix will extract the pertinent
data before the check.

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

remote: add automatic archive extraction to remote import.

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

ctf: allow traces to read timestamps in fields [bug 470845]

This adds declaration looking up their sub-fields.

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

tmf: Bug 473195: Invalid thread access closing editors in non-UI thread

Change-Id: If216cc1032c9bca918a393142f0dca43ddebabbb
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/52316
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

tmf: Split ImportTraceWizardPage into multiple classes

This should make it easier to reuse some parts and also reduce the file
complexity.

Change-Id: I790f8c86a1e7e7011d9f8350d68de4bbc73f5e76
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51815
Reviewed-by: Hudson CI
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Make Filter and Colors tests close the views they opened

That way if the perspective checker runs again, it won't fail.

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

releng: Move the "performance" profile check inside the alltests plugin

Right now, the alltests/pom.xml is used solely to run the
performance" profile, which runs the performance tests.

We can move the "if" inside the alltests plugin itself, so
that this pom.xml can be used for other profiles.

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

tmf: Fix TestInvalidCtfTrace when running from RunAllSWTBotTests

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

releng: Update tycho-eclipserun to use Eclipse 4.5

As far as I know, there hasn't been any changes to help index format
(Apache Lucene) but it's just less dependencies to download since we are
building against Eclipse 4.5 already.

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

ctf: make dynamic scopes work with definitions [bug 470846]

Change-Id: I0838fb9f17ceab856f3009bdf340185b24e82f28
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51546
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI

ctf: make IOStructGen support relative and absolute scopes [Bug 470846]

Change-Id: I1c20fdd7842badcbece63e5461c156da160ffc60
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/50981
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

ctf: set IOStructGen's default alignment to 1 [Bug 470846]

Change-Id: I6a5a60501f2f77809df053c1ebf1f284e9f5ebef
signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/50979
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI

ctf: make declaration scopes support looking up concatenated scopes [Bug 470846]

Change-Id: I97dbc4cf51668f118b379c86db82504789e46522
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/50977
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI

ctf: Add scope strings [Bug 470846]

Change-Id: Ideefd263ec02b18f37ffca11848e5478ca92b38e
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/50976
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-by: Hudson CI

ctf.core add root constructor to DeclarationScope in CTFTrace [Bug 470846]

Change-Id: I7c241c64eaf254bf2ed47db97e8d650d48624831
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/49778
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

ctf: make DeclarationScopes named [Bug 470846]

This makes the Declaration Scopes be part of a tree. Each node is named,
and the root is the trace node.

Change-Id: I90c197364609917385498b828403100be234b8d3
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/49781
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

ctf: update copyright header date in grammars

Change-Id: I1631db8b78f0fc959570d9c3587240008a3216cc
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/50975
Reviewed-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Fix exception importing empty archive or only folders

Change-Id: Ia48a7a2b0abbffaa8eb09c21cd131f06c18341bf
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51500
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Annotate methods in ITmfEventField

Some methods in ITmfEventField can be annotated @NonNull to make
their usage easier.

getValue() and getField() should eventually be marked @Nullable,
but this will be a much larger undertaking, as most analysis
modules currently just get fields and values without even
checking them...

Change-Id: I564c6d067a62239cde5c67350ea709315bfe89f4
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51212
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Fix flaky ImportAndReadSmokeTest test

In this test, it was possible for SWTBotUtils.openEditor to timeout
because it would not find the right Tree and it would wait for a
TreeItem forever. Instead, we can use the SWTBot from the view to narrow
down where the Tree is searched and it returns the right one. I saw this
test being flaky on Windows.

Change-Id: I44e98b6e88d795fd8b0d38b75ad0506e849af4b5
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51535
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Fix StandardImportGzipTraceTest failing when running from RunAllSWTBotTests

StandardImportGzipTraceTest should delete its test project. Otherwise, other
tests using the same project name "Tracing" will fail to run. Also fix path
in TmfTestTrace for the test to work.

Change-Id: I908e9aaded2b47762ee07b36384257545d680300
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51534
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Bring back the API filter for UnsortedPropertySheetPage

PropertySheetPage is marked @noextend, but arguably should not be.
See bug 109617.

I'm pretty sure we had such filter at some point, but it seems to
have been lost, and I could not find any references to it in the
git tree.

Change-Id: If373e480267fd311992a94965635d57ea5100e5c
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/51532
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

pcap: Fix pcap tests skipped when running from RunAllCoreTests

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

lttng: Add a timeout to ExperimentSyncTest

Under certain configurations, the testDjangoExperimentSync() can
deadlock and throw "GC overhead limit exceeded" errors. Add a
timeout to this test suite to catch this problem.

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

Increase RCP version number to 2.0

Change-Id: I8b6f2125cdfa0c3acd5114297b07b7cb703f8ad9
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51132
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Fix FunctionNameMapperTest failing when running from RunAllCoreTests

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

lttng: Fix SessionConfigGeneratorTest failing when running from RunAllCoreTests

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

ctf: bug 471966: Fix file handle leak in CTF

If something goes wrong in the CTFStreamInputReader constructor or one
of its calls, the file handles in this class are not closed. This is
because the object is not fully constructed yet therefore the caller of
the constructor cannot close it.

Change-Id: Ie03164888711dd49d7b30ce5d774b4e4cdef9228
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51426
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Make custom trace output column name @NonNull

Change-Id: Ibe9600a9b80833a860c361320d5d755d148e2711
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/51302
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Tested-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>

ctf: support devices that are stored as enumerations Bug 471685

Enums are actually more precise than ints, so let's keep that data.
A side effect is that this will support floating point devices, but
the information will be cropped to an integer level.

Change-Id: I61c1d98ed63bd18ff1d64ffea5b1f7a9da9c8d7a
Signed-off-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51306
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

Fix parent version number for alltests

Change-Id: I6a6d60c625ddcd2366fdaa5e37a451ec8e7e011e
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51417
Reviewed-by: Hudson CI

tmf: Fix TmfEventsTable leaked

Every time a trace was opened and closed, a TmfEventsTable is leaked (along
with others). Because its context menus are not disposed, the Display keeps a
reference to the menus which hold a reference to the table through menu
listeners.

Change-Id: I7e4bb1497282d33edb0844c85796e6e6ad145e0b
Signed-off-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/51359
Reviewed-by: Hudson CI
Reviewed-by: Patrick Tasse <patrick.tasse@gmail.com>
Tested-by: Patrick Tasse <patrick.tasse@gmail.com>

tmf: Fix context resource leak in raw event viewer

Change-Id: I20dfb3f921d260771efee4b69abaa8d90ec9c44f
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/49466
Reviewed-by: Hudson CI

tmf: Add support for blending sub-pixel events in time graph

This allows for many short time events mapping to the same pixel to be
blended in together instead of drawing only the first one and discarding
the remaining. This functionality is optional and off by default.

The Time Chart view is updated to use this scheme to allow the
highlighted ticks from a color settings filter to be noticeable even if
the time event is not the first event of that pixel.

Change-Id: I316f044157cef50f6744d128c3661964b97f09c1
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/51068
Reviewed-by: Hudson CI
Tested-by: Marc-Andre Laperle <marc-andre.laperle@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
(cherry picked from commit 37a507dca88f92e040df4d860191a88224fca382)
Reviewed-on: https://git.eclipse.org/r/51344

tmf: Dispose clipboard after Copy to Clipboard operation

Change-Id: Icc6ee4b8de1a936ef86c61f0e8b5b9e6afb0de71
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/50633
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Bug 470851: Trace time range should include lost event end time

Change-Id: I273cf79f93f0718a30226d65e1558a7696ed3253
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/50754
Reviewed-by: Hudson CI

tmf: Mark ITmfSourceLookup#getCallsite as @Nullable

Any event that implements ITmfSourceLookup may return null to
indicate there is no specific call site information for this
event.

Clarify this by marking the method @Nullable, which ensures this
case is handled gracefully.

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

tmf: Bug 470830: Incorrect lost event count in histogram

Change-Id: I19a8b3455b50c0ad17a013ce6b3e6fb0a9b870ce
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/50738
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

ctf: Bug 470754: Incorrect time range in CTF Lost Event

Change-Id: Ibdabcf68999cd4f801018c167364bfadf33d3edd
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/50683
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

lttng: Really make lttng kernel event matching package internal

Simply renaming the package with 'internal' in the name did not make it
internal.

Change-Id: I4cbaae0df9e4f691600294e895077d952cbdea74
Signed-off-by: Geneviève Bastien <gbastien+lttng@versatic.net>
Reviewed-on: https://git.eclipse.org/r/51083
Reviewed-by: Hudson CI

releng: Bump versions of parent modules

To make it clearer that this is targetting 2.0.0 release, bump the parent
modules to 2.0.0. We technically don't need to do that but it's less confusing
to see 2.0.0 when building from the top level.

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

Update usage of IAdaptable#getAdapter

Generics were added to the APIs of IAdaptable and related
classes. Update our usage of these methods, as suggested in
http://dev.eclipse.org/mhonarc/lists/cross-project-issues-dev/msg11590.html

Change-Id: I8fb374869567b5c2b7421b4773998c60e9fcc468
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/43472
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

tmf: Bug 470741: Missing lost events in scaled histogram buckets

Change-Id: I8cf2c1a977845c9a60026178940bb09e75206b4c
Signed-off-by: Patrick Tasse <patrick.tasse@gmail.com>
Reviewed-on: https://git.eclipse.org/r/50659
Reviewed-by: Hudson CI
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>