Pass unitsize to srd_session_send() directly.

Follow an API change in libsigrokdecode where srd_inst_channel_set_all()
no longer gets passed a unitsize, but rather srd_session_send() gets it.

Fix #626 by stopping acquisition gracefully

Fix #622 by using the actual sample count per segment

Currently, each segment is as big as the sample limit
(in the bug report: 1G samples), wasting huge amounts
of memory.

View: Allow ScaleUnits[0] to actually be used

Without this patch, no major tick on the ruler will ever
begin with a '1' but only with '2' or '5'. This behavior
can also be observed with current sigrok git head, so it
is independent of my util/view/ruler patch set.

View: Use max time to calculate label length, not offset_

Using offset_ gives false results, in particular when
offset_ happens to be 0. With this fix, realistic values
are used for the label length calculation.

Util: Fix issue where t=0 randomly shows up as +0 or -0

Util: Refactor format_time()

View/Ruler: Calculate and use tick precision

Util: Introduce DD:HH:MM:SS.mmm nnn ppp fff format

This also fixes bug #371.

View: Separate data changed/capture state changed events

View: Determine time unit and use it in other classes

Introduce time units

Fix #615 by opening the output stream only if needed

nsis: Add start menu entries for Zadig.

This fixes bug #542.

Bump Boost requirement to >= 1.48.

Older versions of Boost have issues with C++11 due to boost::shared_ptr.

Details:
http://stackoverflow.com/questions/18900730/boostshared-ptrshared-ptrconst-boostshared-ptr-is-implicitly-declared
https://svn.boost.org/trac/boost/changeset/73202

This seems to have been fixed in Boost 1.48 (untested, though).

This closes bug #593.

Fix include guards and accidential const removal

Fix #404 by only saving enabled signals

Fix #599 by supplying correct length to create_logic_packet()

StoreSession: Supply filename to the output module directly

This fixes parts of bug #570.

Simplify signal clearing in Session::start_capture

Change hotkey for sticky scrolling from 'r' to 's'

build: Only try to read the VERSION file if it exists

about: Show the list of supported output formats.

PulseView now officially supports output formats, so list them too.

Build: Make hardcoded version override git tag

Reintroduce the hard-coded version number in CMakeList.txt, but
still check for a tagged version. If the hard-coded version is
more recent than the latest tag, then use that but with a "-pre"
suffix in place of the patch level.

In either case, still append the revision hash and possibly the
dirty flag to the version, unless git HEAD exactly matches a tag.

Build: Use -dirty version suffix for local changes

Pass --dirty to git describe.  This needed a slight hack to
GetGitRevisionDescription.cmake to make it work.  Also pass
--tags to make git include lightweight tags in its search.

build: Derive PulseView version from git tags

Derive the PulseView version numbers and suffix from the git
repository if available. In order to make builds from source
tarballs work, also include a generated VERSION file in the
source package generated by CPack.

The included GetGitRevisionDescription.cmake file is a copy
from the https://github.com/rpavlik/cmake-modules repository.

Minor whitespace fixes.

Implement "always zoom to fit" feature

Fix #121 by implementing sticky scrolling

Tests: Add required files to CMakeLists.txt

Session: Clear signal data before starting capture

decoderstack: Fix incorrect end sample number.

PulseView was invoking srd_session_send() with an incorrect end
sample number (and thus incorrect sample count).

Example 1: 5000000 samples, unitsize 1

Before the fix:
srd: Calling decode(), start sample 0, end sample 5000000 (5000000 samples, 4096 bytes, unitsize = 1), instance am230x.
srd: Calling decode(), start sample 4096, end sample 5004096 (5000000 samples, 4096 bytes, unitsize = 1), instance am230x.
srd: Calling decode(), start sample 8192, end sample 5008192 (5000000 samples, 4096 bytes, unitsize = 1), instance am230x.
[...]
srd: Calling decode(), start sample 4988928, end sample 9988928 (5000000 samples, 4096 bytes, unitsize = 1), instance am230x.
srd: Calling decode(), start sample 4993024, end sample 9993024 (5000000 samples, 4096 bytes, unitsize = 1), instance am230x.
srd: Calling decode(), start sample 4997120, end sample 9997120 (5000000 samples, 2880 bytes, unitsize = 1), instance am230x.

After the fix:
srd: Calling decode(), start sample 0, end sample 4096 (4096 samples, 4096 bytes, unitsize = 1), instance am230x.
srd: Calling decode(), start sample 4096, end sample 8192 (4096 samples, 4096 bytes, unitsize = 1), instance am230x.
srd: Calling decode(), start sample 8192, end sample 12288 (4096 samples, 4096 bytes, unitsize = 1), instance am230x.
[...]
srd: Calling decode(), start sample 4988928, end sample 4993024 (4096 samples, 4096 bytes, unitsize = 1), instance am230x.
srd: Calling decode(), start sample 4993024, end sample 4997120 (4096 samples, 4096 bytes, unitsize = 1), instance am230x.
srd: Calling decode(), start sample 4997120, end sample 5000000 (2880 samples, 2880 bytes, unitsize = 1), instance am230x.

Example 2: 5000 samples, unitsize 5

Before the fix:
srd: Calling decode(), start sample 0, end sample 5000 (5000 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 819, end sample 5819 (5000 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 1638, end sample 6638 (5000 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 2457, end sample 7457 (5000 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 3276, end sample 8276 (5000 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 4095, end sample 9095 (5000 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 4914, end sample 9914 (5000 samples, 430 bytes, unitsize = 5), instance z80.

After the fix:
srd: Calling decode(), start sample 0, end sample 819 (819 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 819, end sample 1638 (819 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 1638, end sample 2457 (819 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 2457, end sample 3276 (819 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 3276, end sample 4095 (819 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 4095, end sample 4914 (819 samples, 4095 bytes, unitsize = 5), instance z80.
srd: Calling decode(), start sample 4914, end sample 5000 (86 samples, 430 bytes, unitsize = 5), instance z80.

MainBar: Re-check sample count/rate when needed

HardwareDevice: Whitespace fix

Fix #602 by keeping track of device state internally

Added initial Doxygen configuration

Build-depend on the upcoming libsigrok + libsigrokdecode 0.4.0.

Bump version to 0.3.0 (the upcoming next major release).

Append "-git" to the version string.

This avoids confusion with the released tarballs.

This fixes bug #609.

MainBar: Change calling order and flag usage for sample count, too

MainBar: Fix #594 by changing calling order and flag usage

The reason that the sampling rate button disappears after a
sampling rate was selected is that there's a flaw in the
chain of events:

on_sample_rate_changed() which calls
commit_sample_rate() which sets updating_sample_rate_ and calls
on_config_changed() which calls
update_sample_rate_selector() which...
...clears the list and exits because updating_sample_rate_ is set.

View: Update signals without destroying layout

This fixes bug #536

RowItemOwner: Added list_by_type

RowItemOwner: Added list_row_item_owners

InputOutput: replace nullptr with boost::none in Double()

Passing nullptr as the fourth argument in Double's constructor
seems to build fine on x86, but causes build failures on many
other architectures.

Replace it with boost::none.

Signed-off-by: Bartosz Golaszewski <redacted>

Signal: Fix index variable declaration

Fix #501 by deactivating the autocompleter for the signal popup

Device: Fix bug introduced by b48daed which broke analog channels

Fix #590 by adding menu actions to main window also

View: Use more reasonable default time scale

importmenu: Fix libsigrokcxx.hpp #include.

test: session: Fix a compiler warning/error.

 [ 30%] Building CXX object
 test/CMakeFiles/pulseview-test.dir/__/pv/storesession.cpp.o
 [...]/pv/session.cpp:408:62: error: unused parameter ‘device’ [-Werror=unused-parameter]
  void Session::sample_thread_proc(shared_ptr<devices::Device> device,

Support specifying input formats on the command line

Also removes the signal dispatch that was previously used to delay
the file loading until after the constructor.

Moved default device selection into MainWindow

MainWindow: Tidied restore_ui_settings

View: Support having no selected device

DeviceToolButton: Support having no selected device

MainBar: Support having no selected device

Replaced NULL with nullptr

Reordered argument handling

MainWindow: Added import options dialog

MainWindow: Added initial import menu with no options

MainWindow: Added format parameters to load_file

Added InputFile

Session: Support capture before the sigrok::Device object has been created

Session: Use Device::read_config to read sample limit

Device: Replaced Session::read_sample_rate with read_config

Session: Converted data_mutex_ into a std::recursive_mutex

Session: Removed device parameter from update_signals

Session: Made update_signals handle having a missing device gracefully

Session: Catch errors listing the config keys

binding::Device: Catch errors listing the config keys

MainBar: Catch errors listing the config keys

Added devices::File as a common base of file devices

DeviceManager: Deprecated build_display_name and friends

devices::Device: Moved in full_name and display_name

Use device::Devices to represent sigrok Devices

LogicSignal: Tidied up forward declarations

Connect: Tidied up forward declarations

DeviceManager: Tidied up forward declarations

Session: Fixed header guard

Added pv::widgets::ImportMenu

pv::binding::InputOutput: Bound up more integer types and Bool

Session: Simplified logic of set_device

StoreSession: Use DeviceManager to get context

DeviceManager: Added const context accessor

Session: Made update_signals non-destructive

LogicSignal: Added set_logic_data

SignalData: Renamed get_max_sample_count

pv::data::Logic: Renamed get_num_channels and fixed return types

Session: Store signals_ in an unordered_set

Device: Add bindings for SR_CONF_POWER_OFF and SR_CONF_PROBE_FACTOR.

These options are used by baylibre-acme driver and should be available
from pulseview configuration pop-up.

Signed-off-by: Bartosz Golaszewski <redacted>

android: Add missing libintl.so to APK.

This fixes bug #575.

Add a workaround for QTBUG-22829.

Qt4's MOC has issues parsing some C++ constructs, yielding errors like:

  [...]/include/boost/type_traits/detail/has_binary_operator.hp:50:
    Parse error at "BOOST_JOIN"

Full details:

  https://bugreports.qt.io/browse/QTBUG-22829

Use -DBOOST_NEXT_PRIOR_HPP_INCLUDED as MOC option to workaround these
issues. This is currently sufficient, however, depending on the future
PulseView code it may be necessary to add further options mentioned in the
above bugreport (e.g.  BOOST_TT_HAS_OPERATOR_HPP_INCLUDED,
BOOST_LEXICAL_CAST_INCLUDED, BOOST_NO_TEMPLATE_PARTIAL_SPECIALIZATION).

This fixes bug #532 (fixing the build on at least Mac OS X and MinGW).

win32: Fix the Windows build (namespace pollution via windows.h).

One of the multiple build errors:

  In file included from [...]/windows.h:72:0,
                   from [...]/boost/detail/interlocked.hpp:27,
                   from [...]/boost/thread/win32/thread_primitives.hpp:18,
                   from [...]/boost/thread/win32/thread_data.hpp:11,
                   from [...]/boost/thread/thread_only.hpp:15,
                   from [...]/boost/thread/thread.hpp:12,
                   from [...]/boost/thread.hpp:13,
                   from [...]/pulseview/pv/session.hpp:32,
                   from [...]/pulseview/pv/devicemanager.cpp:22:
  [...]/libsigrok/enums.hpp:153:32: error: expected unqualified-id before numeric constant
    static const Quantity * const DIFFERENCE;
                                  ^

windows.h is included by boost/thread.hpp, hence the conflict can be avoided
by defining NOGDI and NORESOURCE before including thread.hpp.

This fixes bug #517.

Bring back sigrok-logo-notext.png (needed on Android).

On Android sigrok-logo-notext.png is used as app icon, and the build
thus fails if it is not available.

Changing android/custom_rules.xml to use the SVG version instead doesn't
seem to work. It appears Android doesn't really support SVGs as app
icons, or at least not without further changes. Continue using the PNG
for now to keep the Android build in a working state.

Drop unneeded /* ENABLE_DECODE */ comments.

(to match the rest of the code base where these are not used either)

mainwindow: add missing checks against ENABLE_DECODE

Pulseview doesn't compile with -DENABLE_DECODE=FALSE. Add missing
checks against ENABLE_DECODE in mainwindow.cpp/.hpp.

Signed-off-by: Bartosz Golaszewski <redacted>

device: ensure bind_enum() checks availability of Capability::LIST.

This avoids a segfault calling bind_enum() with a non-initialized gvar_list.

This closes bug #453.

Fix the build for older glibmm versions.

With older glibmm versions the following error occured:

  pv/toolbars/mainbar.cpp:228:6: error: no match for ‘operator!’ (operand
  type is ‘Glib::VariantContainerBase’)
    if (!gvar_dict) {
        ^

Work around the above issue by using .gobj() and not relying on
"Glib::VariantBase::operator BoolExpr() const", which was only
added in glibmm 2.36.

This fixes bug #548.

Tested on a Debian stable system with glibmm 2.32.1 and Boost 1.49.