Session: Fix issue #67 by improving error handling

AnalogSignal: Recalculate scale when restoring div height

When restoring analog signal settings from a *.pvs file changes the div height, the vertical scale must be recalculated.
Otherwise, the signal is scaled as if the div height had not changed, until a UI event triggers recalculation.

Add new translators to "About" section

- A typo on the code prevented to add previous Spanish translator.

Update Spanish translation to latest changes

Full review on the used terms to improve the consistency.

- This patch includes a lot of grammatical improvements.
- Finish the translation

trace: Whitespace fix

trace: fix crash on empty channel name in popup form

sr_dev_channel_name_set returns SR_ERR_ARG when a channel name is
empty and pulseview crashes once user deletes all characters from
a channel name in trace popup form.

This patch forbids to set empty channel name and keeps last non-empty
name provided by user.

Session: Record acquisition start time and use it when saving

Trace view: Add setting to keep ruler item selected

Fix warning and remove cursor pair selected state

Fixes warning "QObject::connect: Cannot connect (nullptr)::closed() to pv::views::trace::Ruler::on_popup_closed()" and
removes showing the selected state for the cursor pair as it doesn't serve a purpose.

TimeMarker: Automatically deselect item when popup closes

If this is not done, the item remains selected and the user has to manually deselect it, which is bothersome.

Flags: Always save raw text, not display text in session setup

Before, get_text() would return the delta time to a selected flag, so if one was selected as the session was saved, this delta time would be saved as the name.

Add Chinese translate files

Manual: Make timestamp less ambiguous so people don't think it's old

Fix CMake CXX_STANDARD handling, part 2

VERSION_GREATER_EQUAL was introduced with cmake 3.7, so we have to use the older, more verbose expression.

https://cmake.org/cmake/help/latest/command/if.html#version-greater-equal

Fix CMake CXX_STANDARD handling

https://sigrok.org/gitweb/?p=pulseview.git;a=commit;f=CMakeLists.txt;h=fa8d0fcb4cff4f19943fe3ff152dc1428b400b01 introduced a CXX_STANDARD value of 17 while still only requiring cmake 2.8.12.

However, https://cmake.org/cmake/help/latest/prop_tgt/CXX_STANDARD.html#cxx-standard shows that a value of 17 was introduced with cmake 3.8, leading to

"CXX_STANDARD is set to invalid value '17'" when trying to build with cmake < 3.8.

Hence, for versions below 3.8, we only check for up to 14.

Fix glibmm dependency check

https://sigrok.org/gitweb/?p=pulseview.git;a=commit;f=CMakeLists.txt;h=33c5ac28f73aa35c5fc4bbcf69a5ae61c0b50989 introduced the line

pkg_check_modules(GLIBMM_2_4 glibmm-2.4>2.28.0)

but https://cmake.org/cmake/help/v3.0/module/FindPkgConfig.html states

A <MODULE> parameter can have the following formats:

{MODNAME}            ... matches any version
{MODNAME}>={VERSION} ... at least version <VERSION> is required
{MODNAME}={VERSION}  ... exactly version <VERSION> is required
{MODNAME}<={VERSION} ... modules must not be newer than <VERSION>

Hence, ">" is an invalid atom and always makes the check fail. On systems without glibmm-2.68, this leads to glibmm not being found and the build aborting.

MainWindow: Move show_session_error to signals

Before various capture error handlers called Mainwindow::show_session_error()
directly, and thus outside of the main UI thread. Qt widgets aren't thread-
safe and thus this resulted in crashes or freezes at least on OSX.

cmake: only optionally modify in-source .ts files (Qt lupdate)

The creation of .qm output files from .ts input data undoubtedly needs
to be an unconditional part of the build process. The modification of
.ts source files in contrast needs to remain an intentional activity
during development. The mere act of compiling sources to binaries shall
never modify the source tree. Move the lupdate(1) invocation out of the
default build process, while lrelease(1) remains in place.

Ideally we could have a "make ts-update" target for developers to invoke
as needed. But that'd be more involved since the qt5_create_translation()
cmake routine is involved, which needs to be passed variables content
that is only available at configure time. A future implementation could
investigate the add_custom_target() approach. This commit introduces the
ENABLE_TS_UPDATE cmake option to quickly address the issue.

How to reproduce:

  $ cd $SRC
  $ cmake --build $OUT
  $ git status -s
  $ cmake -DENABLE_TS_UPDATE=ON $OUT
  $ cmake --build $OUT
  $ git status -s

This commit also happens to bring build steps in closer promixity, while
making development iterations stand out more perceivably.

manual: symlink manual/images/ subdir for out-of-source builds

Execution of "make manual" created HTML output, but document inspection
suffered from missing screenshots. Symlink the images/ subdirectory so
that source file content is re-used in output hierarchies.

This has gone unnoticed because "make install" references source files,
and the manual author appears to not use out of source builds. Only
out of source builds and only HTML output inspection before installation
were affected. Installed manuals were not affected.

How to reproduce:

  $ cmake --build $OUT --target manual
  $ xdg-open $OUT/manual/manual.html

DecodeTrace: Change code path for obsolete Qt constants (Qt >= 5.12)

This unbreaks macOS 11 builds with qt@5 from homebrew.
This ammends commit 1ed73eb

StoreSession: Use Glib::DateTime

StoreSession: Also send SR_DF_HEADER packet

Some output modules rely on it, e.g. CSV

CMakeLists.txt: adjust cmake build rules for Qt6 support

Check for the availability of Qt5 as well as Qt6. Prefer Qt5 for
backwards compatibility, but accept when only Qt6 is available.
This unbreaks Mac OSX 12 builds with homebrew.

This change is based on work that was submitted by Dominik Sliwa
<redacted>.

It is assumed that failed tests for timestamps to text conversion
(the format_time_minutes() routine) and Qt widgets not being thread
safe are issues that do reproduce more often on the Mac platform
but are independent from Qt6 support. These issues will be dealt with
separately.

pv: adjust application source code for Qt6 support

Prepare application source code (C++ programming language files) for Qt6
compatibility. Implement alternative code paths where the API has changed
compared to Qt5. This unbreaks Mac OSX 12 builds with homebrew.

This change is based on work that was submitted by Dominik Sliwa
<redacted>.

CMakeLists.txt: always use highest available C++17/C++14/C++11 standard

Check for the availability of the C++17, C++14, and C++11 language
standards. Prefer the highest available to build any of the feature
tests or applications. Factor out common conditions into a central spot
in the CMake build rules. Rename variables to avoid special chars in
their name.

Setup both the CMAKE_CXX_STANDARD cmake variable which internally is
used transparently, as well explicitly pass the -std= compile flag in
build instructions for feature checks and application code. It's what
the smuview build does, should also work for pulseview, and is assumed
to not harm either in case it's redundant.

This unbreaks operation in most generic ways on platforms like MacOS 12
(system library), and for external libraries of differing degrees of
aggressivness (boost, glibmm, sig++).

manual: fix a clipboard error in PDF creation build rules

This amends commit f2f536aa7be2 which introduced manual build rules and
duplicated HTML in two locations. One of them is PDF creation. Fix it.
This was spotted by smuview maintenance.

CMakeLists.txt: Enable use of C++17 on Apple systems

The stdlib on OSX 12 Monteray requires C++17 to compile. This change
tests for and conditionally enables C++17 support on Apple devices.
Other platforms remain unaffected.

CMakeLists.txt: Detect glibmm-2.68 in addition to glibmm-2.4

On OS X with homebrew only glibmm-2.68 series is available. As far
as I can tell there are no breaking changes for PulseView on the
glibmm-2.68 ABI change.

This change still prefers glibmm-2.4 if found, and uses 2.4 if both
2.4 and 2.68 are available.

CMakeLists.txt: move includes to the top, earlier pkg-config lookup

Move some of the common cmake includes to a location near the top of the
file. Lookup the pkg-config(1) utility before the construction of the
list of dependendencies.

Remaining subsequent includes are strictly conditional (backtrace), or
belong to different groups than source compilation (VCS revision, CPack).

CMakeLists.txt: consistent whitespace in atomics check

Previous CMakeLists.txt rules used to mix TAB and SPACE indentation.
Consistently use TAB instead. This amends commit 4da54b6be6a1 which
introduced the check for the atomics library.

[ no change in behaviour, see whitespace ignoring diff to verify ]

Signal: Add missing item separator in channel names list

The SCL item in the ChannelNames[] table lacked the separator, which
made it "run into" the next item (preprocessor string concatenation).
Add the missing separator.

This amends commits 9e40e83daf6a (which introduced the lack of the
separator while it did not take effect yet) and 7d5a9c3e79cb (which
made it take effect when items got added and rearranged).

Reported-By: Dominik Sliwa <redacted>

viewport: event->position() isn't Qt 5.12, tweak version check

According to https://doc.qt.io/qt-5/qwheelevent.html#position the
QWheelEvent::position() method was introduced in Qt 5.14. Adjust
the Qt version check to unbreak builds with Qt 5.12.

Man: Document shift+mouse wheel modifier

Update translations.qrc

Add Japanese translation file

AnalogSignal: Fix storing of autoranging setting

Spanish translation updated

Signed-off-by: hasecilu <redacted>

Settings: Fix the default item in the language combobox

If the language is not set, the default language will be English, but the combobox will show "German"(The first item in language combobox)
This is because the current_language will set to "" if the language is not set, and it doesn't match the default language "en"

DecodeSignal: Process late annotations which are sent upon EOF

When protocol decoders learn about the end of the input data, they may
emit more annotations before their operation terminates or gets reset
and re-fed. Do process these late annotations in the application, too.
Without this change, the last annotations are not seen. See bug #292
for an example.

Viewport: scroll horizontally with the vertical wheel and the shift key

Some software (notably Gimp and Inkscape) allows horizontal scrolling
with the vertical wheel when the shift key is pressed. Implement this
behaviour for easier navigation with single wheel mice.

manual: add cmake project() directive for standalone operation

The cmake rules in the manual/ subdirectory are designed to work both in
the context of the application build, as well as for exclusive creation
of the manual.

Add a project() directive in the manual/ cmake rules. Its absence was
not fatal but resulted in warnings. This amends commit 074da67ee25a.

Segment: Include <memory> so we don't get error at compile time

Example build failure:
  In file included from /home/paul/sigrok-util/cross-compile/mingw/build_release_64/pulseview/pv/data/segment.cpp:21:
  /home/paul/sigrok-util/cross-compile/mingw/build_release_64/pulseview/pv/data/segment.hpp:130:14: error: 'shared_ptr' in namespace 'std' does not name a template type
    130 | typedef std::shared_ptr<pv::data::Segment> SharedPtrToSegment;
        |              ^~~~~~~~~~
  /home/paul/sigrok-util/cross-compile/mingw/build_release_64/pulseview/pv/data/segment.hpp:32:1: note: 'std::shared_ptr' is defined in header '<memory>'; did you forget to '#include <memory>'?
     31 | #include <QObject>
    +++ |+#include <memory>
     32 |

Fix #1663 by handling the case where annotations aren't assigned a row

Add missing version check

Fix depreciation warnings caused by newer Qt versions

Trace View: Allow context menu to show basic options in empty area

INSTALL: Clarify translation package requirements

DecodeSignal: Optionally send EOF to decoder session

Communicate EOF to the decoder session when the input stream of
sample data is exhausted (and when the library supports it).

TraceView: Set a lower MinScale

This allows to view smaller scale (ps scalle) signals, e.g. from simulation.

Device: Display "continuous" checkbox when settable in the driver.

This has gone unnoticed because the majority of acquisition devices
either have local memory or require streaming. Only few support both
approaches and users can select the most appropriate approach for
their respective use case. Observed with Kingst LA devices.

cmake: check for optional libsigrokdecode features (send EOF)

Implement a feature check for the srd_session_send_eof() routine which
depends on the libsigrokdecode version. This test implementation is a
little more redundant than necessary, check_symbol_exists() would be
preferred instead but would not work in my local setup.

Application: Do not show unsupported device drivers in version info

Device: Ignore some more false flags

GlobalSettings: Use gsize instead of guint

Binary output view: Highlight byte range currently in view

GlobalSettings: Use g_memdup2 if available

Fix Bug #1649 (Save dialog does not close automatically)

Save data: Catch Error exception

How to reproduce: Save data to Read-Only disk

Save dialog: Fix cancellation

Without the fix cancellation of save data does not work

How to reproduce:
1. Start PV with demo device and generate sufficient data, e.g. demo device 1GSa/s, 10MSa
2. Save data (should take at least 30s) and try to cancel saving
3. PV continues saving data and dialog reappears after next progress update

Ensure QT plugin imports are only done when compiling statically

Fix broken build due to C++ template behind C linkage

glib/gatomic.h since 2.68 includes type_traits, which causes a compilation
error:
In file included from /usr/include/glib-2.0/glib/gatomic.h:31,
                 from /usr/include/glib-2.0/glib/gthread.h:32,
                 from /usr/include/glib-2.0/glib/gasyncqueue.h:32,
                 from /usr/include/glib-2.0/glib.h:32,
                 from /usr/include/libsigrokdecode/libsigrokdecode.h:25,
                 from /home/abuild/rpmbuild/BUILD/pulseview-0.4.2/pv/data/decode/annotation.cpp:21:
/usr/include/c++/10/type_traits:2308:3: error: template with C linkage
2308 |   template<typename _CTp, typename _Rp>

As libsigrokdecode.h declares extern C linkage itself where necessary,
remove it from pulseviews include statements from the last two occasions.

doc: update IRC reference to Libera.Chat

DecodeSignal: Support unitsize > 1 for logic output

DecodeSignal: Receive RLE-encoded PD logic output data

DecodeSignal: Increase robustness to catch errors in PDs

DecodeSignal: Use better colors and names for decoder output signals

Use separate logic output muxing vectors for each decoder

Continue work on SRD_OUTPUT_LOGIC

Allow re-loading sessions to work with PD output signals

Initial support for SRD_OUTPUT_LOGIC

Don't set unicode to true

NSIS 2.x doesn't know the unicode switch and chokes, 3.07 defaults
it to true as well.

As there doesn't seem to be a graceful way to handle 2.x versions,
we have to remove the unicode switch and the warning in 3.x (< 3.7)
installers shall be ignored.

Fix NSIS warning 7998

Thanks to knarfS for the tip!

Simplify segment complete notifications and fix error msg handling

Fix some clang-tidy complaints

Fix handling of 64 channel devices

Fix some clang-tidy complaints

Fix malformed declaration in MathSignal and add missing signal call

Signal: Add clarifying comment

AnalogSignal: Enforce min neg/pos div count for logic

Make AnalogSignal inherit LogicSignal

This way, we can re-use the logic drawing methods and don't
have them duplicated in two different trace types.

Fix remaining invalid disconnect() calls

Fix #1629 more by fixing the notification handling

Add "allow vertical dragging in the view" setting

DecodeTrace: Add annotation class to hover text

Fix #1629 by not reallocating DecodeSegments

Doing so called the empty DecodeSegment copy constructor,
creating empty DecodeSegment instances.

Also fixes an off-by-one bug that prevented the final input
segment from being processed.

Prevent Qt from crashing PV by only showing the first error

Fix #1637 by auto-assigning only to unused signals

Add "start acquisition for all devices" option

Fix #1591 by updating Logic and Analog if needed

nsis: Fix for installing 64bit Windows programs into 'Program Files'

This fixes bug #1082.

MainWindow: Make on_run_stop_clicked() public

handle SIGUSR1 signal to run/stop the capture, UNIX only

Fix #1607 by ignoring common read-only config keys

CMakeLists.txt: Only use -Wa,-mbig-obj and -O3 on Windows.

Note: -mbig-obj has been available on 64bit bit builds for a while now, but
on 32bit builds it needs a rather recent binutils version (2.35, 07/2020).

Fix for cross-compiling for Windows using mingw MXE.

The -mbig-obj param fixes a 'too many sections for PE file' error.

The -O3 param fixes a 'section or string table size too large for PE file'
error.

MathSignal: Implement custom math signal dialog

Application: Also show exprtk version in application information

MathSignal: Add config dialog

pv/exprtk.hpp: Fix Windows build issues.

Define NOGDI and NORESOURCE in order to avoid unneeded namespace
pollution caused by <windows.h>, which leads to errors such as:

  libsigrokcxx/enums.hpp:374:36: error: expected unqualified-id before numeric constant
    static const QuantityFlag * const RELATIVE;
                                      ^

  libsigrokcxx/enums.hpp:161:32: error: expected unqualified-id before numeric constant
    static const Quantity * const DIFFERENCE;
                                  ^

DecodeSignal: Update thread handling to match previous changes