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>

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