Connect dialog: Fix scan for certain USB devices.

Before the fix, selecting "USB" (it's selected by default) in the
"Connect to device" dialog would incorrectly set a conn parameter to
some serial or HID devices, even for scans for pure USB devices which are
neither of those.

This fixes bug #1403.

Don't return valid time() for the cursor pair

Style and architecture fixes

Show relative time of flags on hover

With this change, when the mouse hovers over any flag or cursor, the
flags change labels to show their time relative to the item being
hovered over. When hovering over the interval label of a cursor pair,
the relative time to the nearest end of the interval is shown. While
pressing shift, the regular labels are shown again.

Accept new position on enter press in cursor popups

Fix ruler double-click to add cursor in right position

Trace View: Move ruler time conversion from View to Ruler

Show ruler timestamps in cursors and popups

Prior to this absolute timestamps would be shown, even with the zero
point set somewhere else.

Fixes

Make cursor pair drop precision when too small

Currently, often the label just displays "...". With this change, it
will gracefully drop precision and less important parts (frequency,
unit) when there's not enough space to print the full string. The full
string is still available in the tooltip appearing on hover.

Fixes

Trace view: Allow setting cursors via shift-drag

decoder_selector: Apply decoder upon pressing enter.

If only one decoder matches the filter, apply it when the user presses enter.

decoder_selector: Fix an issue with PD auto-stacking.

Before the fix, PDs with common prefix would incorrectly be found as
a match. Example: Both "spi" and "spiflash" would be considered valid
candidates when looking for PDs which emit "spi" OUTPUT_PYTHON output.

decoder_selector: Expand "All Decoders" if the user filtered

decoder_selector: Auto-place the keyboard cursor in the filter.

Session: Use a monotonic clock to measure acquisition time.

Session: Print duration of last acquisition only.

This resets the timer at the start of a new acquisition.

Style fixes

Remove redundant isNull() check - isEmpty is a superset of isNull

Add -s / --settings parameter to load a session setup file

Fix #1375 by searching across all model columns

This way, a search for "i2c" includes the ID field, not just the name.

Session: Measure acquisition time

Merge RowItem() into ViewItem()

Use multiple drawLine() calls instead of drawPolyline()

Fix variable naming

Disable antialiasing on high-DPI displays

DecoderSelector: Set alignment=top for the PD info text.

DecoderSelector: Make window as wide as necessary when opening

DecoderSelector: Put body label into a QScrollArea

INSTALL: Bump libsigrokdecode requirement to >= 0.6.0.

We now depend on the decoder tags feature introduced in the
libsigrokdecode 0.6.x series.

DecoderSelector: Make first line of PD info italic too.

DecoderSelector: Avoid non-standard variable type.

DecoderSelector: Reduce indentation from 20 to 10.

This wastes a little less screen space.

DecoderSelector: Make decoder tag names fully visible

DecoderSelector: Set alignment=top for the PD info text.

This looks nicer if the text area is resized to be larger than the
PD info text itself.

DecoderSelector: Make PD info text selectable.

This allows for e.g. highlighting and copy-pasting.

DecoderSelector: Allow word-wrap for PD header/footer as well.

Without this, resizing would be unnecessarily limited by the header/footer
length in some cases.

DecoderSelector: Fix catching polymorphic types by value.

  [...]/pv/subwindows/decoder_selector/item.cpp: In member function ‘std::shared_ptr<pv::subwindows::decoder_selector::DecoderCollectionItem> pv::subwindows::decoder_selector::DecoderCollectionItem::subItem(int) const’:
  [...]/pv/subwindows/decoder_selector/item.cpp:44:11: warning: catching polymorphic type ‘class std::out_of_range’ by value [-Wcatch-value=]
    } catch (out_of_range) {
             ^~~~~~~~~~~~
  [...]/pv/subwindows/decoder_selector/item.cpp: In member function ‘QVariant pv::subwindows::decoder_selector::DecoderCollectionItem::data(int) const’:
  [...]/pv/subwindows/decoder_selector/item.cpp:88:11: warning: catching polymorphic type ‘class std::out_of_range’ by value [-Wcatch-value=]
    } catch (out_of_range) {
             ^~~~~~~~~~~~

Fix build without decoders enabled

DecoderSelector: Use srd_decoder_get_by_id()

DecoderSelector: Make PD names italic

DecoderSelector: Properly name utility methods

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