Changes

analysis: Initialize a new 'callstack' plugin core

Base the new minimal set of plugin files on sibling profiling.core ones.

This new plugin is to incrementally hold the callstack implementation to
bring in from Incubator. Goal being, to main-line the latter's callstack
in Trace Compass, as an improved implementation over the current, more
limited one.

Scaffold the incubated callstack alongside the 'profiling' one this way,
before potentially deprecating the latter once proven as a subset.

When run in Eclipse, ActivatorTest's getDefault and getAbsoluteFilePath
methods are not covered locally. Keep them still to match with
profiling.core's own lack of coverage for those, consistently, for this
change scope.

Change-Id: Iaad2fd837c3d58f4b6c6a0514dfdd7cfd0d5cee7
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/197869
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Bring incubated CallGraph, internal 1st

Bring CallGraph from Incubator for an eventual ICallGraphProvider as
next step. Make this CallGraph internal (non-API) only for now, or until
proven as API once/if that becomes the case.

Bring these other classes/interfaces as minimally required dependencies.

Except for .tree which is trivial enough, wait before introducing
sub-packages as Incubator's, to not prematurely create some that could
be misnamed or contain too few classes/interfaces.

CallGraph and its dependencies added herein are tested through upstream
classes in Incubator. Thus postpone their test coverage until the latter
can be ported in turn.

Change-Id: I614575fac6464158f85f0d1a578131f3a4a8014c
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/197758
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Bring incubated ICallGraphProvider in

Add ICallGraphProvider from Incubator in turn, as well as its necessary
dependencies.

Skip ICalledFunction thus reuse the sibling profiling.core one, which
only differs in contract; Incubator's has no default implementation that
may differ. Do so until more contract becomes required if ever. (Likely,
though.)

Change-Id: If6f29c696283355b83b061a9ce0f45a57b230f0d
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/197779
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Bring CallGraphAnalysis from Incubator

Do so along with the related dependencies. CallGraphAnalysis being
another building block for incubated callstack main-lining in TC core.

Also bring incubated ICalledFunction at this stage, now that its
slightly augmented interface is required by hereby added classes.

For now, bring all these in using the callstack.core package, as a flat
list. The follow-up commits shall refactor the packaging, based on this
whole added content and the relationships.

Have InstrumentedCallStackAnalysis import the sibling profiling.core's
CallStackStateProvider still after this excubation. This is to not
change Incubator's use of the latter in its current
InstrumentedCallStackAnalysis. This added dependency on profiling.core
is to likely be removed once ready to deprecate that plugin. I.e., after
callstack.core becomes ready enough as the new Callstack plugin in TC.

Change-Id: I58c11d0e4c34c1514e5e1e7bbb9fe0f8f53214d2
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199452
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Move these to new callstack.core.base

As a way to start reducing the size of callstack.core's WIP flat list.

Base this early packaging on the existing relationships among these
callstack objects from Incubator and the sibling analysis.profiling
plugin. Also base this on which objects use each, blending all of these
informal criteria together, balanced yet for an eventual clean plugin.

Move IDataPalette from core.tree to core.base, as it became used by
objects beyond core.tree centric ones.

Change-Id: I4b0f08f76fc070ae7eaaca74c55aa6629b70f444
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199464
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Move these to new callstack.core.model

As a way to keep reducing the size of callstack.core's WIP flat list.

Base this early packaging on the current relationships among these
objects, as started with the base commit.

Change-Id: Ie6f1fd82f32581909135dd501eeea51cc2e39708
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199465
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Move these to callstack.core.instrumented

As a way to keep reducing the size of callstack.core's WIP flat list.

Base this latest packaging on the current relationships among these
objects, as started with the base commits.

Move CallStackGroupDescriptor and EdgeStateValue from core.base to
core.instrumented, now that they are solely used by the latter.

Change-Id: Ie5fbcc438a122caa4a5faf322873db50e7c224b5
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199466
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Move these to callstack.core.callgraph

As a way to finish reducing the size of callstack.core's objects list.

Base this now refactored packaging on the current relationships among
these objects, as incremented through the base commits.

Change-Id: Icd6217e7a9a23561e78fc5a776f433e0a799c753
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199488
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Init ui folders for FlameChartView next

Prepare the ground for the following FlameChartView addition from
Incubator. This will also include the required dependencies such as
FlameChartDataProvider and company. Goal remaining, to main-line that UI
(whole new Callstack) off Incubator.

Include a settings-only callstack.ui.tests folder structure, to be
consistent with the sibling profiling.ui.tests one solely. Base this
whole commit on the sibling profiling.ui packages, for the time being
that is. This may change as more gets extracted from Incubator.

Skip callstack.ui.swtbot.tests in analysis/pom.xml for now, as there are
no tests yet, which failed the build otherwise.

Change-Id: Ia642026a1dc5a7d390ddfea78be4beda5257a5da
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199568
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Bring FlameChartView UI from Incubator

Along with its minimally related dependencies and likely users. This is
based on the current understanding of the latter, which may change next.

FlameChartView requires FlameChartDataProvider, which in turn requires
FlameChartArrowProvider. Both require FlameChartEntryModel. Include all
of these, brought from Incubator, consistently. Also include the few
currently necessary message and icon resources.

Strictly make callstack.ui an x-friend of callstack.core.instrumented,
so the former bundle's new classes may use the latter's, herein. For
now, keep such new API exposure as restricted as possible. This access
may open up as eventually required.

CallStackAnalysisListener is a plugin.xml element, along with
FlameChartView which gets used by the former. Access to this new view
from the UI is to be proven next, fixing that access as necessary then.

Keep this early Java packaging as minimal as possible. Refactoring some
current package decisions remains likely through follow-up changes as
they become relevant.

[Added] o.e.t.i.a.callstack.ui.FlameChartView

Change-Id: I4d70d12dc9b83bdcfde948a963b6397a1ea4297d
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199583
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Prepare FlameChartDataProviderFactory

Bring FlameChartDataProviderFactory from Incubator in turn. This should
start being exercised next as LttngUstCallStackAnalysis is also brought
in. Keep this for another focused change series.

[Added] o.e.t.i.a.callstack.core.instrumented.FlameChartDataProviderFactory

Change-Id: I2ad1b8829c54287739f354ed9d69381a1fc449a2
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199591
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add incubated LttngUstCallStackAnalysis

Bring Incubator's LTTng-UST CallStack analysis as the new mainlined one.

Add com.google.common.base to the UI dependencies, which was reported as
missing at runtime otherwise, upon exiting.

[Added] o.e.t.i.a.callstack.core.LttngUstCallStackAnalysis

Change-Id: I6e190aa1e85906e99c307bf61fc5062d9a69d9f7
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199641
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Fix CompositeHostModel exit de-register

Before this change, upon Trace Compass Eclipse application exit, there
was a "Resource leak" error in Console, about CompositeHostModel not
having been deregistered.

Dispose then for every InstrumentedCallStackAnalysis subclass, this way.

Add missing blank line at beginning of InstrumentedCallStackAnalysis for
consistency with sibling files, while there.

Change-Id: I5fbc1364d4e504417654fa796af356dad8ca42a8
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199659
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Slightly refactor getDependentAnalyses

Do so in order to potentially help troubleshoot an intermittent IDE
warning in Eclipse, about nullability around the related
getRequiredModules call locally. The said warning doesn't currently show
while writing these lines, but did intermittently prior.

Should the warning be permanently gone somehow, keep this change anyway
as it makes these lines slightly more readable.

Change-Id: I32aa97b602a77df0f57b8b9a5b2857efac55e615
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199661
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add initial test classes from Incubator

Start test-covering this new callstack plugin being brought from
Incubator. More tests should come then, likely including SWTBot ones.

The included CallStackStateSystemTest happens to cover CallStackSeries.

Next to cover are LttngUstCallStackAnalysis{Requirement}, followed by
sub-packages classes, still based on Incubator coverage first.

As in Incubator, skip package-info.java for this root package, as it
introduces many Eclipse build errors with the current implementation.

Have CallStackStateSystemTest import o.e.t.a.profiling.core.callstack's
CallStackStateProvider instead of Incubator's, unlike its originating
Incubator counterpart. This is because all is needed for now is this
link from its javadoc.

Change-Id: Ifba4de6f660f6546420063b42d427f912ff79ea9
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199681
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add incubated LTTngUstCallStack tests

To fully pass these tests locally, [1] has to be extracted under [2].

[1] https://archive.eclipse.org/tracecompass/test-traces/qmlscene.tgz
[2] ctf/org.eclipse.tracecompass.ctf.core.tests/traces/

Also, these tests require more than the default -Xmx512m of heap in the
Eclipse launcher. Required is at least -Xmx1024m of maximal heap.

Change-Id: I647cc228257646413957474130ea83c1835d36e1
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199766
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add incubated CallGraph-related tests

As in Incubator, skip package-info.java for this package, as it
introduces many Eclipse build errors with the current implementation.

No longer @Ignore testCallGraphWithCpuTime in
CallGraphWithCallStackAnalysisTest, as done in the Incubator version.

Change-Id: I0f36a1427b021461758f5de3ad41abb8972364f6
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199769
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add incubated FlameChartDataProviderTest

Compared to the originating implementation [1], skip the XML callstack
case in testGetDescriptors. Do so until that specific callstack type
gets added to TC core (mainlined) in turn.

[1]o.e.t.incubator.callstack.core.tests.flamechart.FlameChartDataProviderTest

Change-Id: I8f68e606d45b9f9da6b6f2e1fb5fe408a18725f7
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199783
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add FlameGraphDataProvider|Factory|Test

-From incubator. Prepare for adding FlameGraphTest and
FlameGraph{Sel}View(s) next, in turn.

[Added] o.e.t.i.a.callstack.core.flamegraph.FlameGraphDataProviderFactory

Change-Id: Ie4c34438286364c99815c23bfaca5218928e2f69
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199828
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add FlameGraphTest, FlameGraph{Sel}View

-From incubator. Include the related dependencies.

Just like the Incubator version, skip the otherwise usual
package-info.java for FlameGraphTest's package, as its originating
implementation excluded @NonNullByDefault that way too.

Also keep FlameGraphView's own, class-scope disabling of
@NonNullByDefault, set (through package-info.java) at package level.

Reuse o.e.t.i.a.profiling.ui.flamegraph.SortOption in FlameGraphView,
rather than adding Incubator's SortOption which has the same
implementation. Slightly adapt profiling.ui.flamegraph.SortOption so it
can be reused this way. Fix file formatting while editing it in Eclipse.

Beside other amends in o.e.t.a.c.ui/META-INF/MANIFEST.MF, import
com.google.common.base at runtime so its Predicate class and company are
found, upon manually opening these new Flame Graph views.

[Added] o.e.t.i.a.callstack.ui.flamegraph.FlameGraphView
[Added] o.e.t.i.a.callstack.ui.flamegraph.FlameGraphSelView

Change-Id: Ie364180b55f856cbeb9cc5111d3397f583df6f91
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199841
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add incubated FunctionDensityView &co.

[Added] o.e.t.i.a.callstack.ui.functiondensity.FunctionDensityView

Change-Id: I8b79d2c5313f0796a621fae441366c2b53989d6b
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199894
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add incubated WeightedTreeView &co.

Add o.e.t.i.a.callstack.core.model.ModelListener to core's plugin.xml
now, in order for Active CPU times to get populated herein.

[Added] o.e.t.i.a.callstack.ui.weightedtree.WeightedTreeView

Change-Id: I05d6efb836ce5935d2504935e7c0e9c251186016
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199908
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add Descriptive Stats under new analysis

Add the Descriptive Statistics view under the LTTng-UST CallStack (new)
analysis, alongside the others. Do so as Incubator also lists it there.

Change-Id: I786367c61d6e278461e71ba03713d7af745f50ac
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/199921
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Add the doc.user files from Incubator's

Preserve the original Incubator files, and supporting files as they are
elsewhere in the code base. Simply main-line the incubated callstack
documentation as is, for this initial scope.

Use 8.3.0.qualifier as Bundle-Version to pass the Maven build locally.

Change-Id: Ic96253819a3489fb1952f91040e5fe6105c03877
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/200145
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

Ust: Fixed SonarCloud Regex HotSpots flag

Fixed SonarCloud Regex related to [1]'s HotSpots issue.

[1]https://sonarcloud.io/organizations/eclipse/rules?open=java%3AS5852&rule_key=java%3AS5852

Change-Id: I2e8a507fcecd805ae12ae2408af3b5c77307940e
Signed-off-by: Estelle Foisy <estelle.foisy@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/200851
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Marco Miller <marco.miller@ericsson.com>

RhinoScriptEngine: Remove redundant // for NOSONAR

Fix the Eclipse file formatting while editing it.

Change-Id: Iba6499538f34df22ed590b56bb7e8c82b0706742
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/200886
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>

callstack: Fix the "i.e." abbreviation syntax used

Change-Id: I8f92e4299bb5a7553d20c133d44d0079bbc1c416
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/200928
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

callstack: Fix nullness annotation warnings across

Fix the Eclipse warning [1] which seems like having appeared since the
IDE version 2023-03. This warning wasn't noticed with 2012-12, that is.

[1] The nullness annotation is redundant with a default that applies to
this location

Change-Id: Ie7b505666194776c80e1e4f4f9bb972d1e887bb3
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/200944
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Bernd Hufmann <bernd.hufmann@ericsson.com>
Reviewed-by: Bernd Hufmann <bernd.hufmann@ericsson.com>

README files: Fix the markdownlint IDE warnings

Fix the markdownlint issues reported through VS Code/Codium for the
README files:

- MD009: Trailing spaces
- MD012: Multiple consecutive blank lines
- MD041: First line in a file should be a top-level heading
- MD045: Images should have alternate text
- MD047: Files should end with a single newline character

Change-Id: Id492f68fcf2dde1a18c5c24c667f19b753206409
Signed-off-by: Marco Miller <marco.miller@ericsson.com>
Reviewed-on: https://git.eclipse.org/r/c/tracecompass/org.eclipse.tracecompass/+/200960
Tested-by: Trace Compass Bot <tracecompass-bot@eclipse.org>
Tested-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Reviewed-by: Matthew Khouzam <matthew.khouzam@ericsson.com>