Ruler: Prevent ticks from disappearing at high zoom levels

Build: Substitute build configuration into NSIS script

This enables out-of-source builds of the Windows installer,
and no longer hardcodes the MXE/mingw prefix.

Build: Fix build with Qt 4 moc

ViewItem: Adjust the threshold between dark and light colors

Previously, the function returned 'Qt::black' for all used colors except
when black itself was passed in, making the text on brown, blue or violet
backgrounds very hard to read.

Viewport: Don't draw the cursors and markers antialiased

After adding them or zooming around, the cursors and markers could
appear washed out.

Ruler: Don't draw the tick marks antialiased

Next to the sharp font of the timestamp labels the washed out markers
didn't look that good.

Untangle the time formatting functions a bit

The formatting depending on the distance between two timestamps is
reintroduced with the responding function residing in the 'Ruler' class
(the prime user of that function).

Fixes a rounding bug for the least significant digit in the
'format_time_minutes()' function.

The 'Cursor' and 'CursorPair' classes now use the same precision when
formatting timestamps as the rest of the program.

View: Do not emit superfluous signals in a loop

View: Change the type of tick_period to pv::util::Timestamp

This makes the tick position/label calculation in the ruler more precise,
avoiding rounded values for zero which would yield to "+0" or "-0" labels.

Ruler: Factor out the calculation of the tick spacing

This avoids useless recalculations of the tick spacings on every mouse
pointer move and makes the function easier to test.

Ruler: Fix tick mark calculation

Prior to this change, due to floating point errors the loop that
calculates the tick mark positions could loop thousands of iterations
while zooming in at high time values, blocking the whole UI and leading
to distorted text in the end.

Use typesafe enum classes in pv::util

Add a spin box widget for timestamp values

format_time(): Use the timestamp type in the calculation

format_si_value(): Use the timestamp type in the calculation

Use a type with a greater resolution to represent time values

Fixes #627.

Remove unused arguments from 'format_time()'

This function is already complicated enough (it tries to guess in what
context it is currently used and changes its behaviour accordingly), so
remove the unused parameters to make it somewhat comprehensible.

Fix #605 by closing current device when another is selected

As the device manager class holds a pointer to all devices
they will never be destroyed until PV exits. Devices are
opened with the first call to create() but only closed in
the destructor. Together, this results in devices never
being closed.

This patch fixes this by renaming create() to open()
and introducing a matching close() method that the
session class calls when a different device is to be
selected.

Fix #592 completely by showing the actual error message

Before, the supplied infoMessage was ignored (unnamed parameter)
and instead the local variable infoMessage was used, which was
empty. For this reason, the cause of the error never showed.

Don't try to store a reference to a shared pointer.

Fixes #596.

Ruler scale calculation optimization

Fix wrong ruler scale after loading a file

Fix #595 by correctly handling UINT64_MAX and INT_MAX boundaries

By using an int64_t to hold UINT64_MAX, it was made to overflow
to -1. This lead to range_min=0 and range_max=-1, resulting in
a disabled spin button.

Fix #597 by disabling device-related toolbar buttons as necessary

Fix #510 by disabling removal of the last decoder from the popup

Provide short-term fix for #627 by limiting zoom range to 100 ps

Util: Fix malformed time formatting

Restore the default of "pulseview foo.sr" opening the file.

This used to work in e.g. the 0.2.0 release and is much more convenient
than having to supply "-i" for every file. It also allows for (easier, at
least) association of the .sr extension with PulseView on most OSes.

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