Tests: Add tests for Segment class

Switch segment storage from single vector to vector of arrays

Previously, PV would run out of storage space for the data
segments because data was stored in a vector. As a vector allows
contiguous access to the underlying data (much like an array),
it needs a contiguous section of memory. With incoming data and
constant resizing of the vector, the OS at some point can no
longer supply such a section of memory, causing PV to abort
acquisition.

This change fixes this by using several chunks that are never
grown in size. Instead, new chunks are allocated and added to
the vector as needed. This way, the OS will be able to provide
memory until it runs out of system memory.

Unify get_samples() semantics for AnalogSegment and LogicSegment

pulseview_cross.nsi.in: Bump Python version to 3.6.

Build fixes for Qt5 Windows/mingw/MXE support.

We currently need to (ab)use pkg-config to get all the required
Qt5 static linking dependencies right, since this doesn't yet
work properly in MXE's cmake.

We use ${PKGDEPS_STATIC_LDFLAGS} instead of ${PKGDEPS_STATIC_LIBRARIES}
to avoid some linker issues related to libbz2.

We need to add Qt5::QSvgPlugin, Qt5::QWindowsIntegrationPlugin,
Qt5PlatformSupport and all the Qt5 libs and their dependencies to
the link libraries list (for both PulseView and the unit tests).

In one of the source code files we need to explicitly list all
static Qt plugins via Q_IMPORT_PLUGIN to make static builds work,
which is currently QWindowsIntegrationPlugin and QSvgPlugin.

We're only focusing on having a working Qt5 build for Windows,
as we no longer need to or want to support Qt4 there.

Details:
https://github.com/mxe/mxe/issues/1642

Thanks to Tony Theodore for the help!

This should also fix bug #871, since we're now building with Qt >= 5.6
which has high-DPI support in general.

Tested via manual specification (might need further changes, though):

  set QT_SCALE_FACTOR=2
  pulseview.exe

Confirm with user also when trying to close the session's main dock

Confirm with user when trying to close sessions with unsaved data

MainWindow: Improve view removal

TraceView: Center traces more than once

This is to make sure the traces are centered when the view is
first drawn but also properly centered when the final size is
known. This fixes a "jumping" display when opening a new trace
view in addition to an existing one.

TraceView: Center traces vertically after view creation

TraceView: Fix variable name

TraceView: Restore vertical offset

license: remove FSF postal address from boiler plate license text

Remove the FSF postal address as it might change (it did in the past).
Reference the gnu.org website instead which is more stable.

MainWindow, View::View: Fix two memory errors reported by valgrind

(use of freed data, use of uninitialized variable)

Add tooltips

Fix unit tests

DecoderStack: Fix "data may be unitialized" error

MainWindow: Fix "main_window may be uninitialized" error

MainBar: Correct event handler name

Fix "always zoom to fit" feature

Implement views::trace::StandardBar and derive MainBar from it

Move file loading from MainBar to Session

Do not change session name when performing only a selective save

Fix #849 by making sure no references to the DecodeTrace instance remain

MainWindow: Update tab text when session name changes

DecoderStack: Remove unneeded shared_ptr use

DecodeTrace: Remove unneeded pointer to the signalbase instance

MainWindow: Add settings button

MainWindow: Add separator

MainWindow: Don't use get_active_view() to determine active session

While it would be neat if it worked, it unfortunately doesn't as
the currently focused item may not be related to any view - e.g.
when one of the tabs of the QTabWidget was clicked.
For this reason, we store the last session the user interacted
with and treat it as the currently focused session.

MainWindow: Fix session error slot declaration

Move run/stop button from the menu bar to the tab widget

MainWindow: Select new session after creation

MainWindow: Prevent the QTabWidget from collapsing when empty

MainWindow: Update session tab when focusing a different view

Move the "new session" button to the main window's tab area

MainWindow: Fix dock nesting

The nesting flag was applied to the wrong QMainWindow
after the introduction of the tab widget.

MainWindow: Allow tabs to be closed

MainWindow: Use regular pointer for QDockWidgets

Using shared_ptrs conflicts with the Qt memory management,
so we can't use them if we don't have control over when
those objects are deleted by Qt. In this case, we need to
handle the dock widgets properly.

MainWindow: Use a QTabWidget to display the QDockWidgets

Change namespace for the trace view and implement ViewBase

MainWindow: Make session naming consistent

CMakeLists.txt: Avoid -fext-numeric-literals, (not supported by clang).

This (re-)fixes bug #863.

Workaround for a gcc 6.2.1 bug causing boost-related build issues.

Patch provided by Aurélien Jacobs, thanks a lot!

This fixes bug #863.

cmake, INSTALL: Bump the Boost requirement to >= 1.55.

Support for serialization of multiprecision data (which is used for view
timestamps) was introduced in boost version 1.55. Since commit 3a21afa6
builds with versions up to and including 1.54 are reported to fail.

Bump the required version from 1.53 to 1.55. Rephrase the CMake logic
which searches for the Boost dependency. Create a list of libraries from
fixed and optional parts, which as a byproduct avoids long lines. Pass
that list of required libraries to one common find_package() call.

Signed-off-by: Gerhard Sittig <redacted>

INSTALL: Add new libboost-serialization dependency.

MainWindow: Fix crash on closing last session

pulseview.qrc: Add missing entries for new icons.

MainBar: Remove empty menu button

MainWindow: Don't return bogus view

Update main window and dock widget titles as session name changes

MainWindow: Don't save sessions that have no device

Session: Fix signal mappings for the views

Allow dock windows to be closed and handle this properly

Add "new session" and "new view" toolbar buttons

Those are temporary and will eventually replaced by
a proper UI.

MainWindow: Enable dock nesting

Save/restore view states and signal settings

Session: Implement .sr file save/restore

Fix crash when saving session with open files

Let SignalBase store/restore its internal data

Session: Save/restore channel info and top-level decoder

SignalBase: Add internal name

MainWindow/MainBar: Fix signals

Before, QMetaObject::connectSlotsByName(this) was used to connect
the signals to the slots automagically. This is no longer feasible
for the MainBar as there are slots that can't be auto-assigned and
the MainWindow doesn't have any signals at the moment.

Session/MainBar: Fix device change handling

Session: Fix include order

Implement multi-session handling

This includes letting sessions restore their internal state on
their own and that there are unique names associated with each
session that the GUI can use when there is no data.

Make the first view own the toolbar instead of the main window

Handling multiple sessions requires that every session can be
controlled individually. This means that either

a) there could be one toolbar that adjusts dynamically to the
session whose view currently has focus

or

b) every session has its own toolbar.

I opted for b) because it's more intuitive and more straightforward
to implement. So now every session has a main view (the first one
created) and a main bar that sits in the same dock widget.

MainWindow: Remove menu bar

MainWindow: Prevent Qt from restoring the dock widgets

Remove context menu for central MainWindow widget

Fix signal connection for view::Signal and SignalBase enabled state

Move view-independent data from view::DecodeTrace to SignalBase

Move signals to views and make Session handle multiple views

Use SignalBase instead of LogicSignal for decoders

Remove data_ also for LogicSignal and use SignalBase data instead

Move signal data to SignalBase and adjust view::AnalogSignal

The signal data belongs to the signal base (M) and not
the view's AnalogSignal (V). Hence we need to move it.

Rename Trace::channel_ to Trace::base_, including dependencies

Introduce pv::data::SignalBase

Fix #814 by using PNG application logo instead of SVG

Make view windows non-closeable

Implement MainWindow::add_view()

ViewWidget: Define ViewTypes

Introduce MainWindow::get_active_view()

Use a QDockWidget to contain the view

View: Move assertion after assignment

Device: add support for list of discrete values for probe factor

Connect dialog: Fix up UI and allow for VXI

Connect dialog: Fix constructor initialization order

Connect dialog: Add support for connecting to raw-tcp devices

This fixes bug #769.

Revert "cmake: use pkg-config ldflags directly instead of spliting libs and dirs"

This reverts commit bb4dede4c8bd711628566d3eb2136442f7205a41.

The -Wl,--whole-archive in libsigrok that lead to this change no longer
exists, so this change can be reverted (since it also causes issues with
"make test").

This fixes bug #808.

Fix #805 by resetting selected device on failure

It can happen that devices can be selected but not used
(permissions problems, connection issues, driver issues, etc.),
so in those cases we want to fail gracefully instead of
segfaulting.
The reason for the segfault is the device selector button
isn't reset in case the device couldn't be opened, causing
the rest of the application to try and work with a device
instance that is actually invalid.

Resetting the device selector when the device failed to
open not only fixes this but also makes the UI more
consistent with the internal state.

Fix warning for unused Session::sample_thread_proc() parameter

Session: Improve session robustness

Don't try to start an acquisition if the device is undefined.

cmake: add check for explicit linking against libatomic

To use atomics functions, some toolchains requires to explicitly add
-latomic to the linker flags (because they are not provided by libc,
but libatomic).

This change adds a helper function trying to build/link a test program
using atomics, then calls it to:
* first check if atomics are directly available in the libc;
* if not and libatomic has been found, then run the same test with
  "-latomic" added to the linker flags.
The pulseview link library list is updated according to the results of
these tests.

This issue was triggered by the Buildroot farms:
  http://autobuild.buildroot.org/results/1e3/1e3101261252d5f30fdf842cc99604e4f4c25eef/build-end.log

Notes:
1- CMAKE_REQUIRED_* variables are only used in check functions. They
   are not automatically forwarded to/handled by the target commands
   (such as target_link_library), because the check functions are
   implemented as macro in CMake code, whereas many target commands
   are native.
2- Because of note #1, CMAKE_REQUIRED_LIBRARIES (or its value) must be
   explicitly passed to the target_link_library command when this is
   needed.
3- In this implementation, LIBATOMIC_LIBRARY is only set when it is
   needed; so, unconditionally appending it to PULSEVIEW_LINK_LIBS
   will produce the expected behavior.

This fixes bug #810.

Signed-off-by: Samuel Martin <redacted>

cmake: use pkg-config ldflags directly instead of spliting libs and dirs

This allows picking some necessary linker flags such as
-Wl,--whole-archive

Trace: Make popup forms compatible with OSX

Qt behaves differently on OSX regarding widget lifetime it seems,
causing the addition of a stacked decoder to crash PV.

According to Elias, the object popup_form_ gets destroyed, yet
it still has the parent popup_ which in turn will destroy it
again at the end of its life. Either removing the parent, or the
preliminary destruction, fixes the crash.

Session: Keep track of signal data locally

Currently, the Session class doesn't hold pointers to the signal
data itself but instead fetches the data on demand using the list
of signals it has. As the Session is providing the model in the
MVC paradigm, it owns the data and thus should keep track of it
directly. The signals are (in an ideal world) mere views on that
data, so querying them for the signal data isn't good style.

InputFile: Use new reset() function to allow re-reading file

Commit 519d0ccbe67d005a9c442795ce3b8255e78ca46d tried to solve
the problem of being unable to re-read an opened file by clicking
the "run" button. However, the solution is insufficient.
PV expects the session device to be updated after the open()
call. If it is not, the device settings popup will show the
settings of the previously used device.

This patch fixes this and also makes use of the new reset()
method to allow re-reading of the file contents. The method
is necessary because otherwise we have to destroy the input
module and create a new instance. This however also means
setting a new session device, which is highly undesirable.
So reset() it is.

DecoderStack: Force a viewport update more frequently

Updating the viewport every 65kib of input data causes
the decode trace to make really big leaps, leaving the
user with the feeling of a not-so-snappy user interface.
We can easily change this by updating the trace every 1kib
since the decoder and the UI thread are separate and the
decode trace drawing is much faster since when the limit
was originally set.

Fix #792 by making sure we don't request samples that don't exist