Draw analog with a polyline rather than points

Implemented Ctrl+Q quit shortcut

Added analog envelope painting

Make Snapshot::get_sample_count() a const function

Implemented AnalogSnapshot::get_envelope_section

Implemented Basic analog tests

Added code to construct analog envelope levels

Added a scale factor to AnalogSignal

Various tidyups in LogicSnapshot

Setup analog probe colours

Renamed pv::view::LogicSignal::LogicSignalColours to SignalColours

Moved SamplingBar into pv::toolbars namespace

Tolerate unexpected packets rather than asserting

Analog buffer sharing crash by memcpying for the AnanlogSignal::paint

Added Enable/Disable All Buttons

INSTALL: Refer to new Building wiki page.

Check that at least one probe is enabled before acquisition begins

Corrected order of MainWindow class declaration

Added UI error handling for file loading and capturing

Added a probe selector

Ensure sr_config_get is called even when assert is compiled out

Added missing pthread linkage in tests

Deleted unused pv::view::Viewport::mouseReleaseEvent handler

This resolves the compile failure reported by arekm

Initialise sample_rate in pv::SigSession::feed_in_header

This resolves the compile failure reported by arekm

Whitespace fixes.

bugfix: Resolved -Werror message in main.cpp

pulseview/main.cpp: In function ‘int main(int, char**)’:
pulseview/main.cpp:91:13: error: comparison is always false due to limited range of data type [-Werror=type-limits]

Signed-off-by: Matt Ranostay <redacted>

Prevent Ok being pressed when no device is selected

Disable Ok until a device has been selected, and select the first device when scan is pressed

Process selected device

Added pv::SamplingBar::set_selected_device

Moved device enumeration out of pv::SamplingBar into pv::MainWindow

SamplingBar::update_sample_rate_selector should abort if no devices is selected

Fixed indentation in SamplingBar::update_sample_rate_selector_value

Implemented device probing

Only allow monotonic devices in the device list

Initial manual connect dialog with a serial port selector

README: Explicitly state meaning of year ranges in copyright statements

Year ranges in copyright statements have no legal meaning. That's why we
explicitly state what they mean in README.

For details, see:
http://www.gnu.org/prep/maintain/html_node/Copyright-Notices.html

Signed-off-by: Alexandru Gagniuc <redacted>

Separate the build/install instructions from the README.

Distros include the README in the package, and it doesn't make sense to have
build and install instructions in a binary package.

Signed-off-by: Alexandru Gagniuc <redacted>

Sampling bar now commits the sample rate into the device instance

Removed sample_rate parameter from start_capture

We now rely on the statefulness of the device instance instead

Added a binding for SR_CONF_SAMPLERATE

Added a Double property object

Implemented getter behaviour in DeviceOptions

Implemented getter behaviour in Enum

Use namespaces in enum.cpp

Corrected const of DeviceOptions::_sdi

Tidied whitespace

Minor whitespace fixes.

Pad mip-map buffers to allow for uint64_t write word

Set the colour of ruler annotations to the correct palette colour

Added open file command line argument

Show samplerate as 1Hz when its value is unknown

Added shorter record lengths

Explicitly set analog signal painting colour

About box: Make URL clickable.

Thanks R. Diez <redacted> for the patch!

Drop a few more unneeded 'extern "C"'.

Remove unneeded 'extern "C"'.

System headers, glib headers, and libsigrok/libsigrokdecode headers all
use 'extern "C"' already, so there's no need to explicitly add these
in PulseView (for these cases).

Thanks R. Diez <redacted> for the patch!

Print --help/--version output to stdout (not stderr).

Thanks R. Diez <redacted> for the patch!

This is not a feasible practice for CLI tools where the output might
be piped into other tools (and you don't want to pipe help messages or
other non-data). However, for the PulseView GUI this is acceptable since
it's not meant to be used that way.

Fix compiler warnings/errors due to -Wall.

Add -Wall compiler option.

Thanks R. Diez <redacted> for the patch!

.gitignore: Add missing CPack files.

Thanks R. Diez <redacted> for the patch!

Cosmetics, typos.

main: quit on SIGINT

Signed-off-by: Alexandru Gagniuc <redacted>

main: Add signal handler for SIGINT

Add a SIGINT handler as requested here:
http://www.airwebreathe.org.uk/projects/index.php?title=PulseView#Make_PulseView_respond_properly_to_SIGINT

Signed-off-by: Alexandru Gagniuc <redacted>

buildsystem: Abort cmake config if Boost is not found

When cmake didn't find Boost, it would not abort; instead the linking
process after the build would fail with non-obvious error messages.

Thus, make Boost a required dependency (fixes bug 59).

Signed-off-by: Alexandru Gagniuc <redacted>

Get sample rate from device instance rather than storing it

Moved signal creation into feed_in_header

Ported sampling to new sigrok API

Renamed pv::dialogs::HwCap to DeviceOptions

Renamed pv::prop::binding::HwCap to DeviceOptions

Ported pv::prop::binding::HwCap to new sigrok API

samplingbar: Replace sr_info_get(SR_DI_SAMPLERATES) with sr_config_list(SR_CONF_SAMPLERATE)

View: Decouple horizontal scrolling from the scoll bar

Scrolling horizontally, regardless if through the tilt wheel, dragging the
signals, or moving the scroll bar, updates the viewport and the scroll bar. This
triggered a valueChanged() signal from the scroll bar, causing the viewport to
be updated again (redundantly). The operation of the viewport became dependent
on the scroll bar. On high zoom levels, where the scroll bar's resolution is
limited, the coupling manifested as jitter. The view was rendered with the
proper resolution of, caused the mouse drag event, then immediately updated and
rendered with the limited resolution of the scroll bar.

Decouple the viewport from the valueChanged() signal, and use the sliderMoved()
signal instead. The scroll bar becomes a simple input element, and only causes
a viewport update when acted upon by the user. Using the scroll bar is still
limited in resolution, but other scrolling methods are no longer affected by the
limitations of the scroll bar.

This fixes bug 7.

Signed-off-by: Alexandru Gagniuc <redacted>

buildsystem: Link against the platform's thread library

At least on Linux, boost-thread depends on pthread. The linking is not
automatic, and users have had issues with this.

We need to link against the platform's thread library, as it is needed for
boost-thread. CMake takes care of getting the exact settings to be passed
to the linker for us with ${CMAKE_THREAD_LIBS_INIT}.

This fixes bug #42.

Signed-off-by: Alexandru Gagniuc <redacted>

Add missing #include <limits.h>.

The file ruler.cpp uses INT_MAX from <limits.h>, so add the #include line.
Without this, the build would fail on some systems (due to -Werror).

Lower Boost dependency to 1.42.

PulseView builds fine with Boost 1.42, too.

MinGW: Drop hardcoded BOOST_ROOT.

This was somewhat useful when PulseView only needed header-only parts of
Boost, but since boost-thread is now also needed this is not sufficient
anymore and needs a different setup for native MinGW builds on Windows.

Cross-compile builds using 'sigrok-cross-mingw' from sigrok-util work
fine without it, though.

Add missing settings for boost-thread on MinGW.

CPack: Only generate a .tar.gz file.

We don't need the tarball as additional *.tar.bz2 and .tar.Z files.

README: Minor fixes, typos.

Documented packing procedure

Prevent CPack from generating a source package with -Source appended to the name

Set the package version number

Add a way to generate a tarball without requiring git

  $ make package_source

This will use CPack to generate tarballs of the source directory.

Signed-off-by: Alexandru Gagniuc <redacted>

Added axes to each signal

Eliminated get_nominal_offset

Lower Boost dependency to 1.45, which is sufficient.

FreeBSD 9.0 ships with Boost 1.45, and builds fine (without errors), so
lower the dependency to 1.45.

samplingbar.cpp: Add missing "ULL".

10000000000 is too large for an int/long on 32bit systems, properly list it
as 10000000000ULL, otherwise the compile will fail.

samplingbar.cpp:48: error: integer constant is too large for 'long' type.

Add missing newlines in two header files.

Without the newlines, some compilers will spew warnings such as:

  cursor.h:83:37: error: no newline at end of file

(which will break the build due to -Werror, which is default-enabled)

This has been (for example) observed on FreeBSD 9.0's (patched) gcc 4.2.1.

CMakeLists.txt: Fix typo.

boost-thread: Use 'thread_win32' on Windows/MinGW.

Added -Wextra, and an option to disable -Werror

Explicitly initialise QObject in the copy constructor

Supressed unused parameters

pulseview.qrc: Add missing application-exit.png entry.

Without this, the icon won't be shown in the menu.

Added File>Quit menu item

Do not fast_forward if there is no data at the minimum mip-map level

Added Sixteen test case

This test case is a replica of sixteen.sr attached to the Bug #33

Fixed LongPulses test packet setup

The length field is now populated with the number of bytes rather
than the number of samples

Fixed usage of sr_datafeed_logic::length

The length field is a measure of bytes not samples. The previous
usage would fail unless the unitsize was set to 1.

global: Do not assert >= 0 for unsigned values

Unsigned values are always >= 0. These assertions are redundant, so remove them.

$ grep assert -r ./ |grep ">=" |grep 0
shows the assert(something >= 0). Remove those which assert(unsigned >= 0).

Signed-off-by: Alexandru Gagniuc <redacted>