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

Implement LogicSegment::get_surrounding_edges() and use it

Manual: Describe sticky scrolling

Logging: avoid "cb was NULL" warning message during logging shutdown

Commit 4b9234088f56 introduced the fetch and restore of the libraries'
default log routines in addition to the registration of a Pulseview
application's log routine. Some code paths in main() could result in
unexpected "cb was NULL" messages on stderr, when logging::init() did
not execute but the destructor tried to restore a handler which was not
fetched before. Silence the error message.

DecodeTrace: Rename get_sample_range() to get_view_sample_range()

Include suggested changes by clang-tidy

Fix #1270 by replacing insignificant characters before comparing

manual/generate: Always generate a PDF manual as well by default.

device.cpp: Add SR_CONF_AVG_SAMPLES.

device.cpp: Add SR_CONF_AVERAGING.

Logging: Write messages from qDebug() and friends also to stdout

Binding: Add help icons for entries with descriptions

Manual: Initial revision

Settings: Fix description text

When %q is used, we don't just encapsulate annotation texts in
quotation marks, we encapsulate all fields.

DecodeSignal: Unpause reliably

manpage: Drop obsolete -s|--log-to-stdout docs.

Android: implement chained log callbacks (buffer and terminal output)

Query the libraries' default log handlers before registering the
application's own handler. Queue log messages in the application's
internal buffer _and_ hand them to the default handlers which send
the text to stdout as well.

Logging: obsolete -s, log to internal buffer as well as stdout

Query the sr and srd libraries' current log routines before registering
PV's own log callback which queues messages in an internal buffer. This
allows duplication of messages to the internal buffer _and_ stdout, and
thus obsoletes the -s option.

pulseview.qrc: Add missing files.

Implement decode pausing/resuming

Fix two warnings

Fix item dragging

Replace (double)0 by 0.0

AnalogSignal: Set initial value for the "value at cursor" variable

Without this, the analog traces show random values after startup
until the mouse cursor enters the application window and the cursor
position updates.

Fix #1259 by not silently ignoring file read errors

Enum: Use slider instead of combo box for contiguous ranges