Don't use std:: in the code directly (where possible).

Use "using std::foo" to make the actual code itself a lot more readable.

There are some exceptions where we usually cannot do this, e.g. std::thread
often conflicts with "thread" from Qt or Boost.

Only show sampling points when zoomed in far enough.

Show sampling points in analog traces.

This fixes (parts of) bug #485.

AnalogSignal::paint_trace(): Factor out a const variable.

Add a hotkey to show/hide sampling points.

This is also hooked up into the global settings, so it will be
persistent across multiple PulseView runs.

This fixes (parts of) bug #485.

Show sampling points in logic traces.

This fixes (parts of) bug #485.

Increase session saving chunk size for much better performance.

Increasing the StoreSession chunk size from 1Msamples to 10Msamples
leads to massively faster session saving due to reduced overhead.

In some cases a 2x (or more) speed-up has been observed. A further increase
to, say, 100Msamples shows no further noticeable improvements.

This also has the additional benefit of generating fewer entries/files
within the sigrok session (*.sr) output/ZIP files, which can potentially
be beneficial for very long captures.

TraceView: Use sticky scrolling setting directly

Settings: Activate first item upon dialog creation

Settings: Change icon for "Views"

Tie the "sticky scrolling" setting in with the settings mgmt

Settings: Change view settings shortcuts

Integrate the about dialog into the settings dialog

Settings: Prettify the settings dialog

Session: Don't reset session name for file devices

Use the "default" keyword.

This patch was generated using clang-tidy:

  clang-tidy -checks="-*,modernize-use-equals-default" -fix

(with manual fixups to move the "= default" to the .hpp)

Use emplace_back() where possible.

This patch was generated using clang-tidy:

  clang-tidy -checks="-*,modernize-use-emplace" -fix

Using emplace_back() has multiple advantages:

 - It's usually shorter and easier to read.

 - It's more efficient.

   V1: v.push_back("foo");
   V2: v.emplace_back("foo");

   V1 will construct a temporary std::string from the string literal "foo",
   another copy of that temporary object will be constructed and placed
   into the vector 'v', then the temporary object's destructor will be called.

   V2 will simply create a std::string directly in the vector 'v', i.e.
   there's only one construction (not 2) and no destructor needs to be called.

Use bool literals for boolean values.

This patch was generated using clang-tidy:

  clang-tidy -checks="-*,modernize-use-bool-literals" -fix

Don't use redundant void argument.

This patch was generated using clang-tidy:

  clang-tidy -checks="-*,modernize-redundant-void-arg" -fix

Prefer std::make_shared().

This patch was generated using clang-tidy:

  clang-tidy -checks="-*,modernize-make-shared" -fix

(with some additional manual fixups)

Using make_shared() over manual construction has multiple advantages:

 - It's shorter to write and easier to read.

   V1: auto sb = shared_ptr<Foo>(new Foo());
   V2: auto sb = make_shared<Foo>();

 - The type "Foo" is repeated less often (less code duplication, lower
   risk of forgetting to update one of the "Foo"s upon copy-paste etc.)

 - Manual construction leads to two individual allocations (actual data and
   the control block of the shared_ptr). Using make_shared() will only lead
   to one allocation, which has performance, cache-locality and memory
   consumption benefits.

 - It's exception-safe, whereas manual construction is not necessarily:

   V1: func(shared_ptr<Foo>(new Foo()), shared_ptr<Foo>(new Foo()));
   V2: func(make_shared<Foo>(), make_shared<Foo>());

   In "V1", one of the "new" invocations could throw, potentially
   causing memory leaks. No leaks will happen with make_shared().

Don't use deprecated headers.

This patch was generated using clang-tidy:

  clang-tidy -checks="-*,modernize-deprecated-headers" -fix

(with some additional manual fixups)

Use nullptr in a few places.

This patch was generated using clang-tidy:

  clang-tidy -checks="-*,modernize-use-nullptr" -fix

GlobalSettings: Implement undo function for the "cancel" button

TraceView: Fix "always zoom to fit" feature

TraceView: Only perform "always zoom to fit" for the main view

Introduce ViewBase::is_main_view

MainWindow: Restructure add_view()

Tie the "always zoom to fit" setting in with the settings mgmt

Tie the "use coloured bg" setting in with the settings mgmt

Implement initial version of the settings management

Manpage update

Fix #829 by not throwing when device can't be used

Update sr_period_string usage

Increase decoding chunk size for much better performance.

Increasing the (max) decoding chunk size from 4ksamples to 10Msamples
leads to massively faster protocol decoding due to reduced overhead
related to srd_session_send() and other functions invoked internally.

In some cases a 2x (or more) speed-up has been observed.

Bring back sticky scroll and coloured background shortcuts.

Converted the actions to QShortcut. We do not use the actions anywhere
else in the code and they were not being triggered using the setShortcut
setting. Additionally expanded the view background color to be a
toggleable attribute.

This fixes bugs #907 and #908.

Name session tabs "Session 1" per default.

This is a bit more specific than "Untitled-1".

manpage: Document CTRL+w for closing session tabs.

Added Ctrl-Q and Ctrl-W shortcuts.

Ctrl-Q is commonly used to close the application, and Ctrl-W usually is
used to close the current tab. According to the Qt::Modifier
documentation this setup should work seamlessly on Mac OS where Meta is
usually used in place of Ctrl.

This fixes bug #899.

Add Run/Stop shortcut.

This is a proposed solution to reenable the <Space> shortcut for
Run/Stop. It seems that setShortcut on QToolButton is not working, but
it seems that adding QShortcut works as a workaround.

DecoderStack: Avoid re-feeding old data after wait_for_data()

This fixes #876 and #850.

Segment: Do not alter chunks when there are active iterators

View: Fix view item positioning

Segment: Move definition of MaxChunkSize

This fixes a compile error with clang:

pv/data/segment.cpp.o: In function `unsigned long const& std::min<unsigned long>(unsigned long const&, unsigned long const&)':
/usr/bin/../lib/gcc/x86_64-linux-gnu/6.3.0/../../../../include/c++/6.3.0/bits/stl_algobase.h:200: undefined reference to `pv::data::Segment::MaxChunkSize'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

pulseview_cross.nsi.in: Use Python 3.4 (Windows XP support).

The last Python version to officially support Windows XP was 3.4.x.

We'll keep the Windows installers at that version for the time being,
until Windows XP support is no longer feasible (e.g. because important
sigrok requirements such as Qt, glib, or libusb drop XP suppport).

Update unit tests

AnalogSignal: Split up divs during autoranging if needed

AnalogSegment: Calculate min/max also for small sample sizes

Perform autoranging for analog traces also when new data comes in

Provide notifiers for when signal data changes

Implement autoranging for analog channels

StoreSession: Remove outdated comment

MainBar: Properly limit the lower value of the sample range

This fixes bug #890.

Session: Handle frame markers properly

AnalogSignal: Allow separate vdiv counts for pos/neg

MainBar: Make sure device and GUI are in sync on startup

Free unused segment memory after acquisition

Segments allocate chunks of MaxChunkSize bytes each.
Most likely, the last allocated chunk isn't fully used,
so there's memory going to waste. This patch fixes this
by allocating a chunk of the required size that replaces
the last standard chunk.

Tests: Add tests for Segment class

Switch segment storage from single vector to vector of arrays

Previously, PV would run out of storage space for the data
segments because data was stored in a vector. As a vector allows
contiguous access to the underlying data (much like an array),
it needs a contiguous section of memory. With incoming data and
constant resizing of the vector, the OS at some point can no
longer supply such a section of memory, causing PV to abort
acquisition.

This change fixes this by using several chunks that are never
grown in size. Instead, new chunks are allocated and added to
the vector as needed. This way, the OS will be able to provide
memory until it runs out of system memory.

Unify get_samples() semantics for AnalogSegment and LogicSegment

pulseview_cross.nsi.in: Bump Python version to 3.6.

Build fixes for Qt5 Windows/mingw/MXE support.

We currently need to (ab)use pkg-config to get all the required
Qt5 static linking dependencies right, since this doesn't yet
work properly in MXE's cmake.

We use ${PKGDEPS_STATIC_LDFLAGS} instead of ${PKGDEPS_STATIC_LIBRARIES}
to avoid some linker issues related to libbz2.

We need to add Qt5::QSvgPlugin, Qt5::QWindowsIntegrationPlugin,
Qt5PlatformSupport and all the Qt5 libs and their dependencies to
the link libraries list (for both PulseView and the unit tests).

In one of the source code files we need to explicitly list all
static Qt plugins via Q_IMPORT_PLUGIN to make static builds work,
which is currently QWindowsIntegrationPlugin and QSvgPlugin.

We're only focusing on having a working Qt5 build for Windows,
as we no longer need to or want to support Qt4 there.

Details:
https://github.com/mxe/mxe/issues/1642

Thanks to Tony Theodore for the help!

This should also fix bug #871, since we're now building with Qt >= 5.6
which has high-DPI support in general.

Tested via manual specification (might need further changes, though):

  set QT_SCALE_FACTOR=2
  pulseview.exe

Confirm with user also when trying to close the session's main dock

Confirm with user when trying to close sessions with unsaved data

MainWindow: Improve view removal

TraceView: Center traces more than once

This is to make sure the traces are centered when the view is
first drawn but also properly centered when the final size is
known. This fixes a "jumping" display when opening a new trace
view in addition to an existing one.

TraceView: Center traces vertically after view creation

TraceView: Fix variable name

TraceView: Restore vertical offset

license: remove FSF postal address from boiler plate license text

Remove the FSF postal address as it might change (it did in the past).
Reference the gnu.org website instead which is more stable.

MainWindow, View::View: Fix two memory errors reported by valgrind

(use of freed data, use of uninitialized variable)

Add tooltips

Fix unit tests

DecoderStack: Fix "data may be unitialized" error

MainWindow: Fix "main_window may be uninitialized" error

MainBar: Correct event handler name

Fix "always zoom to fit" feature

Implement views::trace::StandardBar and derive MainBar from it

Move file loading from MainBar to Session

Do not change session name when performing only a selective save

Fix #849 by making sure no references to the DecodeTrace instance remain

MainWindow: Update tab text when session name changes

DecoderStack: Remove unneeded shared_ptr use

DecodeTrace: Remove unneeded pointer to the signalbase instance

MainWindow: Add settings button

MainWindow: Add separator

MainWindow: Don't use get_active_view() to determine active session

While it would be neat if it worked, it unfortunately doesn't as
the currently focused item may not be related to any view - e.g.
when one of the tabs of the QTabWidget was clicked.
For this reason, we store the last session the user interacted
with and treat it as the currently focused session.

MainWindow: Fix session error slot declaration

Move run/stop button from the menu bar to the tab widget

MainWindow: Select new session after creation

MainWindow: Prevent the QTabWidget from collapsing when empty

MainWindow: Update session tab when focusing a different view

Move the "new session" button to the main window's tab area

MainWindow: Fix dock nesting

The nesting flag was applied to the wrong QMainWindow
after the introduction of the tab widget.

MainWindow: Allow tabs to be closed

MainWindow: Use regular pointer for QDockWidgets

Using shared_ptrs conflicts with the Qt memory management,
so we can't use them if we don't have control over when
those objects are deleted by Qt. In this case, we need to
handle the dock widgets properly.

MainWindow: Use a QTabWidget to display the QDockWidgets

Change namespace for the trace view and implement ViewBase

MainWindow: Make session naming consistent

CMakeLists.txt: Avoid -fext-numeric-literals, (not supported by clang).

This (re-)fixes bug #863.

Workaround for a gcc 6.2.1 bug causing boost-related build issues.

Patch provided by Aurélien Jacobs, thanks a lot!

This fixes bug #863.