DecoderSelector: Rework GSList usage to remove unnecessary list copying

Manual: Add session save/restore, PD selector dialog

ViewBase: Add missing constructor call

DecoderSelector: Implement filtering

DecoderSelector: Use proxy model to permit sorting

DecodeSelector: Use custom treeview to hook into selection changes

Fix #1309 by adding a PD info box

DecodeSignal: Use name of last stacked PD unless the name was changed

Fix #1147 by implementing decoder selector subwindow

views: Increase precision on analog views

Closes #1335.

Auto-load session setups if they exist and auto-save them if desired

Fix #971: Allow saving/restoring of session setups

Fix #1338 ("show time zero at the trigger" option)

Before, successive runs were shifting the zero point for no valid reason
and markers created on the ruler/trace used the wrong time and thus
showed up at the wrong location.

CMakeLists.txt: create NSIS file in binary directory not source

The sigrok-util scripts (mingw and msys2) expect the pulseview_cross.nsi
file in the build directory, and it's good tradition to not write into
the source directory during build. Adjust the configure_file() call.

mainbar: fix build without libsigrokdecode

We must not include decodesignal.hpp if we're not using libsigrokdecode
or the build will fail due to libsigrokdecode.hpp not being present in
the system.

Signed-off-by: Bartosz Golaszewski <redacted>

Session: Enable logic data acquisition using gstreamer

For now, reading data from /tmp/dummy_binary

CMakeLists.txt: Set ENABLE_FLOW to FALSE by default.

flow: Call Srf::init() early in main().

Rename ENABLE_GSTREAMERMM to ENABLE_FLOW.

Add libsigrokflow build system integration.

Add ENABLE_GSTREAMERMM, make gstreamermm support optional.

Add new dependency: libgstreamermm >= 1.8.0.

Also, initialize gstreamermm in main().

Connect dialog: add baud rate combobox for serial ports

Don't use QMessageBox::setInformativeText()

When using it, the extended error message text is not reliably appended
to the dialog text.

Abort acquisition if there are more than 64 logic channels

logicsegment.cpp: Minor whitespace fix.

cmake: Don't do a QUIET search for libsigrokcxx

libsigrokcxx is the most important dependency. A QUIET search means we
can miss important information, such as a version mismatch, and the
"libsigrokcxx not found" message is not at all helpful in determining
the cause of the problem.

manual: Bump date.

INSTALL: Mention both asciidoctor and asciidoctor-pdf.

Use more specific libsigrokcxx/libsigrokdecode minimum versions.

We require libsigrokcxx >= 0.5.1 e.g. due to Analog::get_logic_via_threshold(),
and libsigrokdecode >= 0.5.2 due to e.g. srd_log_callback_get().

Ruler: Fix context menu for items on the ruler

Installer: Add links to manual

INSTALL: Mention optional libboost-stacktrace.

manual: allow manual conversion without source builds

Add a cmake_minimum_required() directive to the manual/ subdir's CMake
rules, such that the documentation can get built without involving the
application and its build dependencies.

SignalBase: Make sure an invalid color isn't shown as "white"

While the color technically is black and not white, an alpha
value of 0 makes it appear as white.

Manual: Adjust wording

INSTALL: Update instructions for building the manual.

manual: Rename pulseview_manual.* to manual.*.

manual: link from decoder troubleshooting to sigrok FAQ item

manual: expand the protocol decoder troubleshooting subsection

Expand the discussion that timing information is not required or
optional to some decoders, but essential to others (because of the very
protocol that gets interpreted). Mention that some oversampling is
typically required. Don't suggest that PD exceptions must be bugs,
incomplete configuration may be even more typical.

Separate the "you can ..." introduction from the first check list item,
to increase visibility of the entry. Existing text was not re-flown, to
reduce the diff size.

manual: remove the generate script, got obsoleted by CMake rules

Instead of running the manual/generate(1) script, "make manual" executes
the conversion tools.

manual: cosmetic nits, add comments to CMake rules

manual: also install images/*.png with the HTML output

Extend the CMake rules for the manual. Do install the images/ subdir but
omit the *.xcf files. Prepare the CSS file although this seems to be not
strictly necessary (gets inlined, but we are prepared if it's external).
Only install these additional files when the asciidoctor(1) tool is
available, and thus the .html file can get generated (to not install
some questionable combination of a .pdf and unrelated .png files, or to
not install the resources when none of the manual files got created).

This approach was tested with the following command sequence:
  $ make manual
  $ make install
  $ xdg-open ~/share/doc/pulseview/*.html

manual: accept absence of the asciidoctor-pdf exectuable

The asciidoctor(1) executable is considered mandatory when building the
Pulseview manual. The asciidoctor-pdf(1) executable is not universally
available (is missing in Debian), accept its absence, avoid execution
failure in that case.

This implementation replaces the actual .txt to .pdf conversion with a
mere echo(1) message, which may go unnoticed in verbose build output.
"make --no-print-directories manual" may be required to remain aware.

manual: install manual conversion result files generated by cmake

Install the (optionally generated) HTML and PDF manual files. Absence of
the files is non-fatal.

manual: introduce CMake logic for asciidoctor execution

Introduce new "manual", "manual-html" and "manual-pdf" make(1) targets
(the former depending on the latter). None of these targets are part of
"make all" by design, users decide whether to convert the manual text.
Execution will fail (fatally) in the absence of dependencies or tools.

Settings::get_view_settings_form(): Fix an incorrect string.

SignalBase: Work around QColor serialization bug on OSX

AnalogSignal/LogicSignal: Internally cache settings used for painting

Implement customizable cursor fill color

Move meta type declaration out of MainWindow

Ask user about adjusting UI colors when choosing a theme

Themes: Fix icon shift bug when opening QToolButton menu

Settings: Allow user to choose the Qt UI style

DeviceManager: Prevent PV crashes when scanning for devices

Manual: Add PD troubleshooting section

Themes: Attempt at working around wrong widget backgrounds on Windows

GlobalSettings: Always use Fusion style on Windows for dark themes

Ruler: Add "Set as zero point" context menu entry

More clazy fixes

Typo fix

Make change performed in 79b53a1ae actually work

ViewWidget::mouse_down_point_ indeed should contain the position
of the last mouse click but since we ignore right clicks, it
doesn't.

Fix random clazy warnings

Some of them are invalid, though, so we suppress them for
future clazy runs.

Fix clazy warnings regarding range-for references

The warnings that are fixed are of this type:
warning: Missing reference in range-for with non trivial type (std::__cxx11::string) [-Wclazy-range-loop]

Fix #1035 by checking for exceptions when accessing config

Also fixes #651 because it makes Device::read_config()
generic.

Fix for "fill logic signal high areas" feature

At certain zoom levels, the algorithm wrongly fills areas where
the signal is low. This patch fixes this.

Fix #1222 by adding a tooltip for when there isn't enough space

Also fixes #870 by always using 12 digits for precision

Add generic "Create marker here" context menu entry

README: List icon/theme licenses explicitly.

Logging: Adjust UI colors to suit both dark and bright themes

Channels: Make channel enable/disable options symmetric

Speed up MipMap downsampling in logicsegment

Make use of the segment chunk size to process samples
in contiguous blocks when possible.

Provide routines for downsampling with sample sizes
of 1, 2, 4 & 8 bytes which the compiler can optimize.

These changes halve the time taken to display the
23MB ws2812b_neopixel24_4mhz_verylong.sr from
40 seconds to 18 seconds.

Expose length of valid data in segment chunk

Fix memory leak in DecoderMenu

==214004== 1,536 (16 direct, 1,520 indirect) bytes in 1 blocks are definitely lost in loss record 6,288 of 6,442
==214004==    at 0x4C2EE0B: malloc (vg_replace_malloc.c:299)
==214004==    by 0x67D8435: g_malloc (in /usr/lib64/libglib-2.0.so.0.5600.3)
==214004==    by 0x67F0056: g_slice_alloc (in /usr/lib64/libglib-2.0.so.0.5600.3)
==214004==    by 0x67F14FA: g_slist_copy_deep (in /usr/lib64/libglib-2.0.so.0.5600.3)
==214004==    by 0x5584FC: pv::widgets::DecoderMenu::DecoderMenu(QWidget*, bool) (decodermenu.cpp:33)
==214004==

Implement fixes proposed by clang-tidy

Add themes and implement theme support

Make "fill logic signal high areas" feature more robust

device.cpp: SR_CONF_AVG_SAMPLES: Add support for list of values.

Trace: Force trace replaint when a setting changes

Use QMenu->popup() instead of QMenu->exec() everywhere

GlobalSettings: Enable mouse hover marker by default

Implement "fill logic signal high areas" feature

Segment: Implement temporary workaround for #1284

Ruler: Add "toggle mouse hover marker" to context menu

Ruler: Add context menu with "Create marker here" entry

Segments: Fix iterator access to underlying value

Before this change, it->value was accessed even when the
iterator went past the end of the underlying buffer.

LogicSegment: Check upper bound when performing edge search

manual: Reword and update some stuff

View: Refine snapping algorithm

We now also choose the best match depending on edge density.

View: Fix snap-to-edge when using all signals

Rename --no-scan to --dont-scan

Fix #1292/1294 by snapping to any edge when not hovering over a signal

1292: TimeItem instances now snap to any edge when not hovering over a
      signal, including the ruler itself. Hence, there is always an area
      where one can hover without selecting a signal, even if the entire
      view is full of signals.
      Cursors also show the desired behavior: the left edge snaps while
      the right does not. Hence, the width is kept constant. If the user
      wants to snap the right edge to a signal edge, this can be achieved
      by dragging the right edge separately.

1294: TimeItem instances now snap to any edge - before, they were snapping
      only to edges when hovering over a signal.

View: Make TimeItem derivatives snap to edges when ruler-dragged

Without this change, the snap-to-edge feature doesn't work
when TimeItem derivatives are dragged in the ruler.

manual: Add a short Mac OS X install text.

manual: Various minor fixes.

Fix unit tests

Application::print_version_info(): Minor output whitespace fixes.

Fix #1213 by improving the version information handling

Add setting for the snap-to-edge distance

Fix #684 by implementing snap-to-edge for TimeItem-based classes