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>

tests: Add check for handling of wide data (16 probes)

The details of the test are explained in-source. This test is designed to
provide a test case for bug 33.

Signed-off-by: Alexandru Gagniuc <redacted>

Fixed pv::dialogs::About to accomodate 42515b0f in libsigrokdecode

Added status icons to Run/Stop button

Tidied pv::MainWindow menu setup

Removed status bar

Set HwCap dialog title

Added fallback toolbar icons

Added Tango configure icon

Implemented HwCap binding

Added commit support to HwCap dialog

Added commit support to Binding

Added get/set support to Property and Enum

Integrated property widgets with dialog

Added Enum property

Added Property

Added empty HwCap dialog

Added empty HwCap binding

Moved data and snapshot classes into pv::data namespace

Tidied include of view in mainwindow.cpp

Moved pv::About into the pv::dialogs::About namespace

Viewport: Allow horizontal scrolling via mouse wheel tilting

On mice with a horizontal scroll wheel, or tilting wheel, enable the user to
scroll horizontally.

Signed-off-by: Alexandru Gagniuc <redacted>

Viewport: Only zoom in/out at vertical wheel events

Viewport always zoomed the signals in and out on a wheel event. Moving the
scroll wheel up or down caused the intended effect of zooming in or zooming out,
respectively. This works flawlessly on mice with only a vertical wheel.

On mice with a horizontal or tilting wheel, horizontal scrolling or tilting was
still treated as zooming in and out. This created a non-intuitive experience,
as the user expects the signals to scroll left or right with this gesture.

Check to make sure the wheel event is a vertical one, and only then zoom in/out.
Do nothing if the wheel event is not vertical.

Signed-off-by: Alexandru Gagniuc <redacted>

Added basic analog signal plotting

Added skeleton analog support

Renamed pv::SigSession::_state_mutex to pv::SigSession::_sampling_mutex

Factored out feed_in_logic

Factored out feed_in_meta_logic

Make data_feed_in and data_feed_in_proc packet parameter a const pointer

This patch corresponds to bf53457d in libsigrok.

Moved file load into a thread

README: Document Boost system + thread requirement.

Some smaller whitespace fixes.

cmake: Add boost-thread as explicit dependency.

Apparently boost-thread requires boost-system even if not explicitly
used in the code.

See also: https://svn.boost.org/trac/boost/ticket/7241

On Debian with libboost-thread1.50.0 and libboost-system1.50.0 the
following error occurs during linking:

  /usr/bin/ld: CMakeFiles/pulseview.dir/main.cpp.o: undefined reference to symbol 'boost::system::system_category()'
  /usr/bin/ld: note: 'boost::system::system_category()' is defined in DSO /usr/lib/libboost_system.so.1.50.0 so try adding it to the linker command line
  /usr/lib/libboost_system.so.1.50.0: could not read symbols: Invalid operation
  collect2: ld returned 1 exit status

This commit fixes the issue.

It's unclear whether this is an issue that should be fixed in PulseView
in general, or whether it's a Boost issue.

Made LogicDataSnapshot::get_sample private rather than mutex protecting it

Mutex protected snapshot data

Keep Run/Stop button state updated

Added proof of concept start/stop support

Initial real time display during capture

Mutex protected SigSession::_signals

Added SigSession::signals_changed signals to indicate when signals are created

Initial threaded capture

Add struct sr_context * to sr_driver_init() calls

libsigrok.git commit 193691ae1590eb31025d943ffbb0ea5c1047e6ca changes
the API of sr_driver_init() to take a struct sr_context * parameter as
allocated by sr_init().

Move our struct sr_context *sr_ctx out of main() into file global scope
so that it can be used easily in all calls to sr_driver_init().

removed std namespace to build with llvm

Added pv::view::View::normalize_layout to normalize out-of-bounds drags

Moved signal margin out of signal rendering into initial layout

Improved propagation of drag event so that the scroll bars can be updated

Don't highlight when dragging

Improved drag selection logic

Added snap to grid drag behaviour

Added initial support for dragging

Added signal (multi) selection

Use static signal offsets instead of offsets calculated on-the-fly

This will be needed for dragging

Moved forward declaration of SignalData to correct namespace

Moved Signal and LogicSignal into pv::view

Added a label colour chooser dialog

Add struct sr_context to the sr_init() and sr_exit() calls

libsigrok.git commit b8072700c1bc7d13ba004fd897668b56cec4ac62 adds
struct sr_context to the public API, and changes sr_init() and sr_exit()
to take a struct sr_context ** and struct sr_context * parameter
respectively.

struct sr_context is opaque, and all sr_init() and sr_exit() calls must
be balanced.