Remove the inline qualification from sr_rational_set().

Inlining can only happen in the same compilation unit where the
function was defined, so there is no sense declaring an inline
function in a header if this function is not defined in this
same header.

This fixes the following gcc-5 warning:

In file included from include/libsigrok/libsigrok.h:1066:0,
                 from src/version.c:21:
include/libsigrok/proto.h:36:20: warning: inline function 'sr_rational_set' declared but never defined
 SR_API inline void sr_rational_set(struct sr_rational *r, uint64_t p, uint64_t q);
                    ^

Correctly copy sr_datafeed_meta in sr_packet_copy().

Commit 5801d558 replaced g_slist_copy_deep() by some incorrect code
that actually leaks the newly allocated memory, instead of doing
a deep copy.

This new version should be more correct, more concise, and it fixes
the following warning:

src/session.c: In function 'sr_packet_copy':
src/session.c:1025:38: warning: passing argument 2 of 'g_slist_foreach' from incompatible pointer type [-Wincompatible-pointer-types]
   g_slist_foreach(meta_copy->config, (GCopyFunc)copy_src, NULL);
                                         ^
In file included from /usr/include/glib-2.0/glib/gmain.h:26:0,
                 from /usr/include/glib-2.0/glib/giochannel.h:33,
                 from /usr/include/glib-2.0/glib.h:54,
                 from src/session.c:24:
/usr/include/glib-2.0/glib/gslist.h:125:10: note: expected 'GFunc {aka void (*)(void *, void *)}' but argument is of type 'void * (*)(const void *, void *)'
void     g_slist_foreach                 (GSList           *list,
         ^

sr_strerror_name(): Add missing SR_ERR_IO entry.

SR_ERR_IO: new error code

Add new error code which can be used to notify the user about
general input/output errors.

Signed-off-by: Bartosz Golaszewski <redacted>

configure.ac: Add AC_CANONICAL_SYSTEM macro.

In order to determine the target OS when cross-compiling libsigrok
we need autotools to set the 'target_os' variable. This macro
determines the system type and sets output variables to the names
of the canonical system types.

Signed-off-by: Bartosz Golaszewski <redacted>

backend: Add basic transform module sanity checks.

transform: Add a few basic unit tests.

transform: Add an "invert" transform module.

This inverts the data values:

 - An analog value of x becomes 1/x.

 - A digital value of 0 becomes 1 (and vice versa).

transform: Add a "scale" transform module.

transform: Add a "nop" transform module.

This doesn't do anything, just passes input packets on unmodified.

transform: Hook up transforms.

transform: Add a basic set of API calls.

transform: Add struct sr_transform and struct sr_transform_module.

Fix problem building with BSD make

This fixes bug #556:

Bug 556 - libsigrok fails to build with BSD Make

Makefile.am: Add NEED_SERIAL condition.

Makefile.am: Unconditionally build src/lcr/es51919.c.

We do the same with DMM parsers currently, and this change also
fixes an issue in sigrok-util's 'new-driver' script.

This fixes bug #545.

Update list of files ignored by git

configure: Avoid bashism breaking the C++ bindings

sysclk-lwla: Widen constant to 64 bit before shifting

(lwla_convert_trigger): Fix trigger mask computation bug introduced
by recent change:  Widen constant to 64 bit before shifting so that
channel nunmbers beyond 32 are processed correctly.

Lower dependency to glib 2.32.

By avoiding g_slist_copy_deep() for now, we can easily allow libsigrok
to build against glib 2.32 (less hassle for users of stable/older
distros or OSes).

output/gnuplot: Use .dat as suggested file extension.

Gnuplot doesn't have any "official" file name extension(s). It uses
(at least) two different types of files basically:

 - "control files": These can have many different somewhat commonly
   used extensions such as .gpi, .gnu, .gnuplot, .gp, .plt, .gih,
   others. These files don't contain data, only Gnuplot commands such
   as 'set yrange [75:105]', 'set ylabel "foo" offset 1', and so on.

 - "data files": This is what libsigrok reads and writes. These files
   contain actual data to be graphed by Gnuplot (with the help of a
   specially-crafted control file, see above). The data is usually in
   a tab-separated format. The common file extension is usually .dat,
   though many others are possible as well.

in/out: Minor consistency renames.

 - 'struct sr_input *' variables are consistently named 'in'.
 - 'struct sr_input_module *' variables are consistently named 'imod'.

 - 'struct sr_output *' variables are consistently named 'o'.
 - 'struct sr_output_module *' variables are consistently named 'omod'.

output/wav: Improved description

output: Added preferred file extension field

This fixes parts of bug #541.

input: Added preferred file extension field

This fixes parts of bug #541.

cxx: Fix a linking issue.

Fix "undefined reference to `sigrok::EnumValue<sigrok::LogLevel,
sr_loglevel>::_values'", which happens at least when using clang(++),
e.g. on Linux, Mac OS X, or FreeBSD.

This fixes bug #534.

Thanks to Uffe Jakobsen and Martin Ling for reporting and investigating!

Fix FreeBSD issue with libusb_get_port_numbers()

Currently (as of date 20150122) an ioctl problem within the
FreeBSD kernel is preventing libusb_get_port_numbers() from working.
Hence calls to libusb_get_port_numbers() will always return 0.
This makes it impossible to establish a physical path the the usb device.

This problem has existed "forever" -
meaning that libusb_get_port_numbers() has never worked.

A fix is committed to FreeBSD "current" head  -
and will later be merged (MFC'ed) to maintenance branches.
See: https://svnweb.freebsd.org/base?view=revision&revision=277417

Additionally FreeBSD requires that devices prior to calling
libusb_get_port_numbers() have been opened with libusb_open().

The patch is "forwards-compatible".
Currently it acts specificly to libusb_get_port_numbers()
currently returning 0 on FreeBSD.
In these situations it constructs an artificial path to the device.
When FreeBSD kernels appears with proper working ioctl
supporting libusb_get_port_numbers() the code will construct
proper physical paths for newer kernels - while still generating
artificial physical paths for older defective kernels.

demo: implement averaging support

Add support for averaging and avg_samples option to the demo device.

Signed-off-by: Bartosz Golaszewski <redacted>

new config options: averaging

Add new config options to libsigrok - 'averaging', which allows to
enable averaging of samples and 'avg_samples' for setting the number
of samples to be averaged over in each cycle.

Signed-off-by: Bartosz Golaszewski <redacted>

Check for librevisa >= 0.0.20130412 - the latest official release

sr_input_new(): Add missing @param comment.

This fixes a Doxygen warning:

  src/input/input.c:209: warning: The following parameters of sr_input_new(const struct sr_input_module *imod, GHashTable *options) are not documented:
    parameter 'imod'

C++: Fix segfault where input/output options are NULL.

Fix two warnings when libserialport is not used.

    CC       src/fallback.lo
  src/fallback.c: In function 'sr_serial_list':
  src/fallback.c:26:59: warning: unused parameter 'driver' [-Wunused-parameter]
   SR_API GSList *sr_serial_list(const struct sr_dev_driver *driver)
                                                             ^
  src/fallback.c: In function 'sr_serial_free':
  src/fallback.c:31:51: warning: unused parameter 'serial' [-Wunused-parameter]
   SR_API void sr_serial_free(struct sr_serial_port *serial)

Install libsigrokcxx headers in separate directory.

Don't mix libsigrokcxx header files into the $prefix/include/libsigrok
directory. Use a separate $prefix/include/libsigrokcxx instead.

cxx: Rename include/libsigrok/ to include/libsigrokcxx/.

Rename libsigrok.hpp to libsigrokcxx.hpp.

This avoids confusion of libsigrok.h vs. libsigrok.hpp and makes it
clearer that this is the main libsigrokcxx header.

libsigrok.hpp: Fix incorrect glibmm.h #include.

As per upstream docs "#include <glibmm.h>" is the correct usage,
the "glibmm-2.4" directory is handled correctly via pkg-config:

 $ pkg-config --cflags glibmm-2.4
 -I/usr/include/glibmm-2.4 [...]

libsigrokcxx: Drop Requires.private entry.

libsigrokcxx.pc has a "Requires" field listing "libsigrok" which
will cause libsigrok's "Requires.private" entries to be used/inherited
when 'pkg-config --libs --static libsigrokcxx' is used.

Rename libsigrokxx to the more common libsigrokcxx.

Makefile.am: Use $(MKDIR_P) everywhere.

scpi: Fix incomplete data issue for e.g. Hameg HMO1024.

On a Hameg HMO1024 you get incomplete data because the USB transfer takes
longer than the scpi->read_timeout_ms of 1 second that is defined in
scpi_dev_inst_new(). Therefore reset the timeout in sr_scpi_get_string()
whenever the device sends a partial response.

hameg-hmo: Fix double-free issue.

hameg-hmo: Set SR_CONF_GET | SR_CONF_SET for frame limit.

Reading analog data from an HMO1024 (Firmware 04.527) failed because the
frame limit could not be set.

Python bindings: respect DESTDIR.

Thanks Aurelien Jacobs <redacted> for a suggested
fix/improvement.

Limit frames instead of samples.

Since every individual measurement is represented by a single frame
and a "sample" isn't all that meaningful concept in this context,
it makes more sense to define possible limit in number of frames.
Make the es51919 driver to support setting a frame limit instead of
a sample limit.

Use frames to group a single measurement result together.

In most, but not all, modes the ES51919 reports two separate
analog values for each measurement sample. These values are
mapped to two separate channels and sent in two separate
packets.

A client program needs a way to determine which results are
parts of the same measurement and also know when a complete
measurement is received so it can process the sample. Use
the frame begin and end packets to separate groups that each
represent a single complete measurement.

Fix the channel indexes of the es51919 driver.

Set the channel indexes to unique values instead of setting
them all zero.

fix autogen.sh on OS X. This fixes bug #516.

Fix compile error on FreeBSD

serial.c: Cosmetics/consistency/documentation fixes.

Add a public API to list available serial ports.

Mention the Building#FAQ wiki page for common issues.

fix numpy include path

README: Require SWIG >= 2.0.0.

We use %shared_ptr for example, which was introduced in SWIG 2.0.0.

Only save enabled channels to a session file.

This reverts bc96d5f08fe59ea7c51799b148946f5003c90927 which is not
needed anymore since we have a better fix for #410 and #495.

session_file: Enable only the probes that are actually listed in metadata.

This is a better fix for #410 and #495.

README: Improve and update the requirements list.

saleae-logic16: Show libusb transfer status name instead of code.

saleae-logic16: Change two errors into warnings.

Related to #466, convert two more sanity checks from errors into warnings.
This may allow more devices to work with libsigrok.

saleae-logic16: Clearer error message.

beaglelogic: Add SR_CONF_CAPTURE_RATIO support.

saleae-logic16: Add SR_CONF_CAPTURE_RATIO support.

fx2lafw: Add SR_CONF_CAPTURE_RATIO support.

soft-trigger: Add support for pre-triggering.

Always save all channels to a session file.

This works around limitations of the current API that screw up saving
only enabled channels. See bug 410. And bug 495.

bindings: Session::set_trigger(): Fix segfault condition.

sr_session_trigger_set(sess, NULL) is a valid thing to do, meaning that
any trigger shall be removed from the session.

This closes bugs #491 and #496.

sr_session_trigger_{get,set}: Document, add error checks.

Add a few unit tests for sr_session_trigger_{get,set}.

manson-hcs-3xxx: Fix incorrect SR_CONF_SCAN_OPTIONS handling.

The SR_CONF_SCAN_OPTIONS key must be listable with or without sdi,
otherwise the device will not be detected by frontends.

Do not check for JDK headers if Java bindings are disabled.

This removes an unnecessary build dependency on JDK and fixes
build troubles on systems where javac is present but JDK is not
installed and Java bindings are disabled by ./configure --disable-java.

usb_get_port_path(): fix libusb error checking

When libusb cannot access a device, libusb_get_port_numbers() will return
an error. Check the return code rather than doing invalid pointer
operations (out-of-bound read).

Avoid segfaults at sigrok-cli startup on my setup where some USB devices are
not accessible and also make Valgrind happier.

Signed-off-by: Vincent Palatin <redacted>

Publish config key capabilities on session driver.

Add sr_rational_set() convenience function.

saleae-logic16: Downgrade error during capture to a message.

Some clone doesn't set this to the exact same value, and both bits in 0x48 are
marked as unknown at
http://sigrok.org/wiki/Saleae_Logic16/Firmware#FPGA_variables

This fixes bug #466.

unit tests: Disable timeout for one of the test cases.

This one can take a while, thus disable the timeout.

python: Silence some warnings via -Wno-uninitialized.

Silence some warnings when building the Python bindings:

  sigrok/core/classes_wrap.cpp: In function ‘PyObject* _wrap_new_OutputFormatMap(PyObject*, PyObject*)’:
  sigrok/core/classes_wrap.cpp:5232:4: warning: ‘argv[0]’ may be used uninitialized in this function [-Wmaybe-uninitialized]
      res = SWIG_ConvertPtr(obj,(void**)&p,swig::type_info<map_type>(),0);
      ^
  sigrok/core/classes_wrap.cpp:14383:13: note: ‘argv[0]’ was declared here
     PyObject *argv[2];
               ^
  sigrok/core/classes_wrap.cpp: In function ‘PyObject* _wrap_new_ChannelGroupMap(PyObject*, PyObject*)’:
  sigrok/core/classes_wrap.cpp:5232:4: warning: ‘argv[0]’ may be used uninitialized in this function [-Wmaybe-uninitialized]
      res = SWIG_ConvertPtr(obj,(void**)&p,swig::type_info<map_type>(),0);
      ^
  sigrok/core/classes_wrap.cpp:23356:13: note: ‘argv[0]’ was declared here
     PyObject *argv[2];
               ^

We add -Wno-uninitialized since the warnings are harmless and we really
don't care about them in the generated classes_wrap.cpp.

This fixes parts of #417.

Fix a bug causing one of the unit tests not being run.

trigger: Add some more error handling.

All of these error conditions are checked via the unit tests.

Also, add the following missing entries (analog trigger types):

 - SR_TRIGGER_RISING
 - SR_TRIGGER_EDGE

Add a few unit tests for sr_trigger_*().

trigger: Add API documentation.

Add missing entries to sr_config_info_data[].

Newly added keys:

 - SR_CONF_SAMPLE_INTERVAL
 - SR_CONF_NUM_TIMEBASE
 - SR_CONF_NUM_VDIV
 - SR_CONF_CENTER_FREQUENCY
 - SR_CONF_DEVICE_MODE
 - SR_CONF_SCAN_OPTIONS
 - SR_CONF_DEVICE_OPTIONS
 - SR_CONF_DEVICE_MODE
 - SR_CONF_TEST_MODE

Also, keep the same ordering and grouping as in libsigrok.h.

Use g_malloc0() consistently, simplify error handling.

Use g_malloc0() for small allocations and assume they always
succeed. Simplify error handling in a few places accordingly.

Don't always sanity-check parameters for non-public (SR_PRIV)
functions, we require the developers to invoke them correctly.
This allows further error handling simplifications.

Don't check sr_channel_new() return value (always succeeds).

We now use g_malloc0() for the allocation and assume the allocation
will always succeed, thus sr_channel_new() will always return a
valid new channel.

Consistently use g_malloc0() for allocating devc.

We assume the allocation will always succeed, hence no need for
checking the returned value.

Eliminate sr_dev_inst_new().

Fix invalid pointer dereference.

yokogawa-dlm: Publish driver options.

motech-lps-30x: Publish driver options.

manson-hcs-3xxx: Publish driver options.

hameg-hmo: Publish driver options.

Finish fixing broken sr_config_list() logic.

Make sr_analog_*_to_string() functions allocate the necessary buffers.

mic-985xx: Correctly report SR_CONF_THERMOMETER/_HYGROMETER.

Some models only log temperature, others log temperature and humidity
(so they need different drvopts).

mic-985xx: Drop unneeded #define.

mic-985xx: Publish driver options.

center-3xx: Drop unneeded #define.

center-3xx: Publish driver options.

Revert "victor-dmm: Set spec digits to 3, matching the display."

This reverts commit 28b424349363a9002e7af7300a7f2941a8469004.

ols: Publish driver options.

agilent-dmm: Now really fix driver options.