Reworked SamplingBar device selection logic

  * Split on_device_selected into update_device_config_widgets. This allows
    the widgets to be updated with calling SigSession::set_device.

  * Don't rely on the on_device_selected event handler to update the device
    config widgets. If MainWindow selects a device, that device may be the
    0th drop-down element, and already selected in which case there will be
    no device_selected signal, because there was no change in the selection.
    We now surpress on_device_selected with _updating_device_selector, then
    set the selected device in the drop-down, then call
    update_device_config_widgets to manually update the widgets.

  * Combined set_selected_device into set_device_list, so that the two can
    be set in one operation.

  * This fixes bug #334

Set PopupToolButton::_popup to NULL at construction

Revert back to the default device if a session file failed to load

This fixes #331

Don't show decoders with probes in the Stack Decoder menu

This fixes #333

Don't allow disabled probes to be selected

This fixes #313

Do not attempt to draw text for small annotations

Removed undefined Annotation::set_row and set_pd_index

Enum: Reference gvariants in the values list

Enum: Use an iterator instead of iterating by indexes

Added decoder options binding for double values

Addded support for decoder options with a values list

Moved print_gvariant into DeviceOptions

nsis: Fix the "working dir" path for PulseView.

This was previously incorrectly set to the firmware/ subdir which
resulted in PDs not showing up in PulseView's Decoders->Add menu.

Thanks to Peter Newman for bringing up this issue.

Fix typo in header guard causing warning

Added missing include for boost::hash_combine

Only show first-level decoders in the menu

Removed assert from DecoderStack

Fixed non-painting decode

Only update the decode state when decoding

Check required probes before starting the decode thread

Added an error message there are some unspecified required probes

Removed unused parameter in DecoderStack::decode_proc

Change-Id: I6ac5ef33b616f8e15628eb8888148140b19ad8e1

Notify repaint after decode_data instead of inside annotation_callback

Enable decode support by default

Implemented threaded decode

Renamed DecoderStack::_mutex to _output_mutex

Renamed SigSession::data_updated signal to data_received, and added the frame_ended signal

Moved inner decode loop into DecoderStack::decode_data

Begin a new decode session when a new frame begins

Added a reference to the SigSession in DecoderStack

Added LogicSignal::get_data

SamplingBar: Only use DefaultSampleCount if the _sample_count widget is unset.

This fixes bug #324

Use boost::filesystem::path to get the filename from a path

Don't attempt to set SR_CONF_LIMIT_SAMPLES when it's not supported

Show the file-device in the selector when a file is loaded

SamplingBar: Surpressed a spurious update loop

Moved default device functionality into SigSession

Moved session creation into DevInst objects

Removed a redundant call to SigSession::set_device

Moved SigSession::is_trigger_enabled into DevInst

Moved DevInst::_sdi down into Device

Moved DeviceManager::use_device and release_device into DevInst

Made DeviceManager only handle Device objects

Added pv::device::Device

Moved DevInst into the pv::device namespace

In View::zoom_one_to_one use View::get_visible_data

LogicSignal: Un-break trigger configuration. (bug #318)

Don't join with non-threads.  Fixes 323

Pass new unit_size argument to srd_inst_probe_set_all()

decode: Fix mixup of bytes vs samples

(DecoderStack::decode_proc): The final argument to srd_session_send()
is a byte count, not a sample count.
(LogicSnapshot::get_samples): Multiply start_sample by the unit size
to get the byte offset into the buffer.

memaccess.cmake: Use STATUS instead of WARNING for output.

Using STATUS looks nicer here (and it's an informational message only).

Old:
CMake Warning at CMake/memaccess.cmake:39 (MESSAGE):
  Cross compiling - using portable code for memory access
Call Stack (most recent call first):
  CMakeLists.txt:93 (memaccess_check_unaligned_le)

New:
-- Cross compiling - using portable code for memory access

memaccess.cmake: Cosmetics, whitespace, consistency fixes.

Match the rest of PulseView's CMake file conventions more closely.

Skip check for unportable sample pack/unpack when cross-compiling

Check if unportable sample pack/unpack works and in that case use it.

Implement portable LA sample packing and unpacking.

Added row heading text

Fixed unit test CMakeLists.txt

Change-Id: Ib6f431ecc26e771ad18fb16118fe047eec8b734f

Preallocate memory before samples are received.

Preallocating memory for samples when a capture is started improves performance during the capture, as realloc() is called only once.

Notify that config change when probes are enabled/disabled

This fixes 296

SweepTimingWidget: When the exact value isn't available try to set a close match

Moved all sr_probe modification into pv::DevInst

Removed SigSession from Trace

Removed _session.get_device calls from LogicSignal

Added DevInst pointer to Signal

Updated the sample rate selector when the config is changed

SamplingBar: Improved event loop protection

Added DevInst::config_changed signal

Moved config getting/setting into DevInst

Fix some disappearing annotations

Wrapped sr_dev_inst in a class: pv::DevInst

DecodeTrace: Draw the error over the unresolved period

Do not attempt to decode when required probes have not been specified

This fixes 204

Added some missing fromUtf8s

This fixes the issues mentioned in #307

RowData: Removed _ann_start_index and _ann_end_index

This fixes #308

Improved decode annotation colouring

This fixes bug #311

Fixed decoder probes auto-select logic

This fixes bug #310

Add bindings for new device options

Add device option bindings for the SR_CONF_TRIGGER_SLOPE,
SR_CONF_EXTERNAL_CLOCK and SR_CONF_CLOCK_EDGE settings.  For
now, the slope and edge settings use the generic combo box
widget.

Added show/hide decoder button

Added decoder show/hide icons

Improved row annotation colouring

Split annotation data set into rows, and improved painting.

This has allowed the removal of Annotation::_row and Annotation::_pd_index

Fix bug #235. Add shortcut keys for View/Zoom In (Ctrl++) and View/Zoom Out (Ctrl+-)

These are the shortcut keys for zoom in and out as suggested by
QT. Works with the normal keyboard and numpad keys.

Seems that the preferred simple + and - keys have an issue.
While the - key is working and displayed in the menu as expected,
the + key is zooming but not displayed at View/Zoom In
in the menu. I've tried several attempts.

Extract only the subset of annotations that are in view.

To achieve this lists of annotations have been added sorted by
start sample and end sample

Draw stacked decoder traces below the other PDs (avoids overlap).

Every stacked decoder is drawn below the PDs it stacks upon, so that all
of the annotations of all PDs in the stack can be seen, and no overlap
happens.

pv::data::decode::Annotation: Add pd_index() and set_pd_index().

pv::view::DecodeTrace: Draw annotation rows according to PD metadata.

pv::data::DecoderStack: Set _row for each annotation.

For each annotation, keep track of the "annotation row" it belongs to.

pv::data::decode::Annotation: Add row() and set_row().

Keep track of the "annotation row" this annotation is supposed to be
displayed on.

decoderstack.cpp: Improve two error messages a bit.

decode: Clear error message upon every new session run.

When a decoder throws an exception, i.e. srd_session_send() returns an error,
PulseView shows a red "Failed to initialise decoder" banner in the
decode trace at the moment.

This can happen when e.g. invalid option values are set in the PD popup,
or invalid probe combinations, and so on. However, after the user resolved
the error condition (e.g. selected a proper set of probes / options) the
red error banner would still remain visible and no re-decoding would happen.

This patch fixes that.

Fixed layout of labeled widgets

nsis: contrib/pulseview.nsi: Drop obsolete file.

We're only using the cross-compile version pulseview_cross.nsi these
days since it is much more convenient, and the old native build file
was not working and/or up-to-date anyway.

CMakeLists.txt, test/CMakeLists.txt: MinGW linking fix.

test/CMakeLists.txt: Fix MinGW build of the tests.

Fixed off-by-one in DecoderStack::decode_proc

pv::prop::Int: Removed assert from Int::get_widget

pv::prop::String: Do not create a widget if the initial value could not be retrieved

pv::prop::Double: Do not create a widget if the initial value could not be retrieved

pv::prop::Bool: Do not create a widget if the initial value could not be retrieved

Set default build type to RelWithDebInfo