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.

Change main() to have only one exit code flow, used also on errors

Pixel aligned signal labels

Initial code to render text inside cursor markers

Moved LabelPadding into View

Added cursor dragging

Made TimeMarker and Cursor QObjects

Added cursors show/hide menu action

Added cursors

Documented scale, offset, set_scale_offset, _scale and _offset in pv::view::View

Added Cursor class

Added TimeMarker base class

Added header time hover arrow

Added hover point to pv::view::View

Render ruler and signals with sub-pixel accuracy

Moved unit test build commands into test/CMakeLists.txt

Fix #include guards.

 - Add missing #include guards.

 - Fix incorrect #endif comments for some #include guards.

 - Use consistent naming for them, as per HACKING.

Add a HACKING file.

Added a context menu for set name

Cache signal label text size at draw time for later use

Fixed timeline scale

Added mouse-over behaviour to signal labels

Moved Signal label layout to get_label_rect

Enable -Werror -Wall

Removed uneeded statements  in LogicDataSnapshot test

Corrected more signed/unsigned inconsistencies in LogicDataSnapshot test

Corrected signed/unsigned inconsistencies in Ruler::paintEvent

Removed unused return value from LogicSignal::paint_caps

Corrected intializer order in LogicSignal

Removed uneeded statements from LogicDataSnapshot

Corrected signed/unsigned inconsistencies in LogicDataSnapshot

Only check for libboost-test if tests are enabled.

Add initial manpage.

Add icons/sigrok-logo-notext.ico.

This is (will be) needed on Windows (and for the Windows installer).

Cosmetics.

Do not pass -DBOOST_TEST_DYN_LINK to pulseview

-DBOOST_TEST_DYN_LINK is meant to be passed only to pulseview-test,
not to pulseview. When test were enabled, this flag was defined
globally, and as a result, also passed to pulseview.

Create a CMakeLists.txt for test, and add test as a subdirectory
when tests are enabled. This way, -DBOOST_TEST_DYN_LINK is only
defined for our test app.

Signed-off-by: Alexandru Gagniuc <redacted>

Added config.h to .gitignore

Added --help option

Use ${PROJECT_NAME} instead of pulseview in CMakeLists.txt

Added --version option

Made extdef.h a global include

Use C++ style comments in main.cpp

Generate a config.h file for versioning

Versioning was already handled in CMakeLists.txt, but it was handled
in a non-standard and unclear manner. We used
    set(VERSION
for starters. While this is valid CMake syntax, VERSION is also a
reserved word in cmake, depending on the context. We were
communicating the version information by a compile-time define, again,
not a recommended practice.

Change versioning to the more standard way of
    set(*VERSION_MAJOR
    set(*VERSION_MINOR
    set(*VERSION_MICRO
    set(*VERSION_STRING

Instead of defining the version at compile time, generate a config.h
file with the version information.

Signed-off-by: Alexandru Gagniuc <redacted>

Unify the if(WIN32) defines

The if(WIN32)/else(WIN32)/endif(WIN32) is an archaic cmake syntax.
Remove all the if(WIN32) clauses, and use an option to specify whether
we want to link to the static versions of libsigrok*. That's basically
what the WIN32 clauses were doing anyway. This makes the "what is
going on" more readeable.

On WIN32, automatically set this option to link to the static libraries.

Offer unit tests with a cmake option