CMakeLists.txt: Drop superfluous "include(FindPkgConfig)".

This causes the check for the pkg-config tool to be performed twice
since it is followed by another "find_package(PkgConfig)".

This can cause issues in some situation, see e.g.
https://github.com/mxe/mxe/issues/1023#issuecomment-161255930

This fixes bug #663.

DecodeTrace: Don't dereference iterator pointing at end of container.

LogicSignal: Only draw the trigger marker for enabled channels.

This makes the behaviour consistent across LogicSignal's three
painting functions (paint_back(), paint_mid() and paint_fore()).

LogicSignal: Put all trigger matches into a single trigger stage.

Previously each trigger match was assigned its own trigger stage,
which is problematic because devices may not support more than one
stage.

Ideally we would want the user to decide which matches go into
which stages, but that's a long way off and this change makes
things work just a little bit better for more devices.

Fixes bug #683.

MainBar: Only show "total sampling time" tooltip if supported by device.

If the device doesn't support SR_CONF_LIMIT_SAMPLES, the tooltip would
always show a sampling time of 0s.

Session: document the purpose of sampling_mutex_.

Session: don't hold sampling_mutex_ during signal emission.

We were holding the lock while emitting capture_state_changed.

View: support multiple trigger markers in one acquisition.

Remove the previous acquisition's markers when a new acquisition
is started, and add a new one everytime we receive the SR_DF_TRIGGER
packet.

Minor (Doxygen) comment cosmetics.

Use the TriggerMarker class to visualize the time of SR_DF_TRIGGER.

This allows us to remove the name-based immovable hack from Flag.
Note that this also means that the trigger marker no longer has
a name at all.

Add TriggerMarker class.

This will be used in a follow-up changeset for the markers that
visualize the time of SR_DF_TRIGGER.

Note that this copies a little code from the TimeMarker class,
but that seemed a better idea than deriving from it and disabling
the popup and label-drawing code.

When hovering a ViewItem only change the mouse cursor for movable items.

Add a mechanism to make ViewItem classes immovable.

This adds a new method ViewItem::is_draggable(), which returns
true iff the view item is supposed to be draggable/movable.

Also, ViewItem::drag() is turned into a nop if the item is
not draggable.

StoreSession: Observe proper range order

When I wrote the original solution I wanted min() and abs()
to take care of this issue. However, I forgot that abs()
works on a difference of two uint64_t, so this subtraction
will underflow if the range is in the wrong order.
Converting them to int64_t isn't a good solution, so I'm now
using an ordinary condition instead.

Fix #685 by adding a special T marker when SR_DT_TRIGGER arrives

Minor whitespace cosmetics.

MainWindow: Merge save_selection_to_file() back into export_file()

MainWindow: Add selection_only parameter to export_file()

Add "save selection range as..." to the export menu on the toolbar

Make ExportMenu accept a vector of actions

Refactoring: move samplerate algo from View to Session

Introduce "save selection range as..." feature

CMakeLists.txt: Enable unit tests by default.

This will help to avoid missing some breakage scenarios.

Suppress warnings from glibmm about deprecated auto_ptr

This fixes bug #654.

build: Limit auto-versioning to git revision hash

When building from a git checkout and HEAD does not exactly match
a tag, append the shortened revision hash to the hardcoded version
string. Do not attempt to provide any information beyond that.

This brings the PulseView versioning in line with the other modules
in the sigrok stack.

avoid using deprecated std::autoptr

This fixes the following g++-5 warning:
pv/dialogs/about.hpp:51:7: warning: ‘template<class> class std::auto_ptr’ is deprecated [-Wdeprecated-declarations]
  std::auto_ptr<QTextDocument> supportedDoc;

win32: Re-fix the Windows build (namespace pollution via windows.h).

This brings back / extends fixes for bug #517.

Minor update related to the libsigrok analog format changes.

This is only a temporary fix, and assumes only SR_DF_ANALOG packets with
float data ever arrive. Support for other types will be needed later.

test/CMakeLists.txt: Add missing files.

Viewport: Anti-alias time items, but not row items

View: Place non-grouped logic channels in a TraceGroup

ViewWidget: Moved update() call into ViewItem

AnalogSignal: Apply scale handle dragging

LogicSignal: Apply scale handle dragging

ScaleSignalHandle: Initial implementation

ViewWidget: Added drag-handling support for non-TraceTreeItems

Header: Added drag-handling support for non-TraceTreeItems

ViewItem: Make hit_box_rect take ViewItemPaintParams

LogicSignal: Replaced SignalMargin with half font-height

Header: Hit-test items in reverse order

ViewItem: Added a default label_rect implementation

Signal: Added scale_handle_offset, scale_handle_dragged and scale_handle_released

Viewport: Render all types of RowItem

TraceTreeItem: Update when selection state changes

ViewItem: Make select virtual

ViewItem: Make drag_release virtual

ViewItem: Make paint_label optional

Signal: Derive from ViewItemOwner

TraceTreeItemOwner: Removed non-const item_list accessor

Added ViewItemOwner

RowItemIterator: Removed usage of RowItem::owner()

Session: Removed signals_mutex(), and made signals() return a copy not a reference

LogicSignal: Call TriggerStage::begin with matches.begin()

TraceTreeItem: Added valid owner assert to drag_by

Use list_by_type instead of iterator

Reduce include bloat by including boost/thread/{locks,shared_mutex}.hpp directly

View: Moved in list_row_item_owners

View: Use list_by_type to get TraceTreeItems

Header: Use list_by_type in on_ungroup

TraceTreeItemOwner: Make list_by_type return a vector instead of a set

TraceTreeItemOwner: Moved list_by_type template definition into header

RowItem: Moved hover_point_changed in

ViewItem: Make a create_popup optional

TraceTreeItem: Separated from RowItem

ViewItemPaintParams: Added missing include

AnalogSignal: Removed set_scale

Nothing uses it now

ViewItemPaintParams: Fixed header guard

RowItem: Corrected header guard

windows: Install data files into share subdirectory

This allows these files to be found on standard paths relative
to the location of the executable.

android: Include firmwares in the APK

android: Fix copylibs task to include the right libs

Since more and more libs are losing their versioned soname, the
heuristic of including libs with versioned sonames is an increasingly
poor one.  Instead, simply blacklist the libs we do not want.

android: Don't include qtaccessiblewidgets plugin

Since Qt 5.4.0, accessibility is no longer a plugin.

android: Check for firmware in standard locations, too

android: Read firmware resources from assets

Set up a resource reader to load firmware resources as assets
bundled with the APK.

SAMPLERATE is a guint64

This fixes bug #621.

Ruler: Don't use a function that is only available in recent boost versions

Ruler: Prevent ticks from disappearing at high zoom levels

Build: Substitute build configuration into NSIS script

This enables out-of-source builds of the Windows installer,
and no longer hardcodes the MXE/mingw prefix.

Build: Fix build with Qt 4 moc

ViewItem: Adjust the threshold between dark and light colors

Previously, the function returned 'Qt::black' for all used colors except
when black itself was passed in, making the text on brown, blue or violet
backgrounds very hard to read.

Viewport: Don't draw the cursors and markers antialiased

After adding them or zooming around, the cursors and markers could
appear washed out.

Ruler: Don't draw the tick marks antialiased

Next to the sharp font of the timestamp labels the washed out markers
didn't look that good.

Untangle the time formatting functions a bit

The formatting depending on the distance between two timestamps is
reintroduced with the responding function residing in the 'Ruler' class
(the prime user of that function).

Fixes a rounding bug for the least significant digit in the
'format_time_minutes()' function.

The 'Cursor' and 'CursorPair' classes now use the same precision when
formatting timestamps as the rest of the program.

View: Do not emit superfluous signals in a loop

View: Change the type of tick_period to pv::util::Timestamp

This makes the tick position/label calculation in the ruler more precise,
avoiding rounded values for zero which would yield to "+0" or "-0" labels.

Ruler: Factor out the calculation of the tick spacing

This avoids useless recalculations of the tick spacings on every mouse
pointer move and makes the function easier to test.

Ruler: Fix tick mark calculation

Prior to this change, due to floating point errors the loop that
calculates the tick mark positions could loop thousands of iterations
while zooming in at high time values, blocking the whole UI and leading
to distorted text in the end.

Use typesafe enum classes in pv::util

Add a spin box widget for timestamp values

format_time(): Use the timestamp type in the calculation

format_si_value(): Use the timestamp type in the calculation

Use a type with a greater resolution to represent time values

Fixes #627.

Remove unused arguments from 'format_time()'

This function is already complicated enough (it tries to guess in what
context it is currently used and changes its behaviour accordingly), so
remove the unused parameters to make it somewhat comprehensible.

Fix #605 by closing current device when another is selected

As the device manager class holds a pointer to all devices
they will never be destroyed until PV exits. Devices are
opened with the first call to create() but only closed in
the destructor. Together, this results in devices never
being closed.

This patch fixes this by renaming create() to open()
and introducing a matching close() method that the
session class calls when a different device is to be
selected.

Fix #592 completely by showing the actual error message

Before, the supplied infoMessage was ignored (unnamed parameter)
and instead the local variable infoMessage was used, which was
empty. For this reason, the cause of the error never showed.

Don't try to store a reference to a shared pointer.

Fixes #596.

Ruler scale calculation optimization

Fix wrong ruler scale after loading a file

Fix #595 by correctly handling UINT64_MAX and INT_MAX boundaries

By using an int64_t to hold UINT64_MAX, it was made to overflow
to -1. This lead to range_min=0 and range_max=-1, resulting in
a disabled spin button.

Fix #597 by disabling device-related toolbar buttons as necessary