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

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