output/wav: track and free memory for float conversion buffer

input: fix leak of config data in several input modules

output: fix options memory leak

output/analog: track and free memory for float conversion buffer

==18779== 800,000 bytes in 196 blocks are definitely lost in loss record 29 of 29
==18779==    at 0x4C29110: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==18779==    by 0x4E635C3: receive (analog.c:319)
==18779==    by 0x40870B: datafeed_in (session.c:316)
==18779==    by 0x4E59D4E: sr_session_send (session.c:1201)
==18779==    by 0x4E59F8B: sr_session_send (session.c:1159)
==18779==    by 0x4E62595: send_chunk (wav.c:234)
==18779==    by 0x4E62A06: process_buffer (wav.c:290)
==18779==    by 0x40954A: load_input_file_module (input.c:123)
==18779==    by 0x4097AB: load_input_file (input.c:157)
==18779==    by 0x40531E: main (main.c:288)

session: free memory for datafeed callbacks, reported by valgrind

==17549== 32 (16 direct, 16 indirect) bytes in 1 blocks are definitely lost in loss record 22 of 39
==17549==    at 0x4C29110: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==17549==    by 0x5359200: g_malloc (in /usr/lib64/libglib-2.0.so.0.4200.2)
==17549==    by 0x536EE2D: g_slice_alloc (in /usr/lib64/libglib-2.0.so.0.4200.2)
==17549==    by 0x5370165: g_slist_append (in /usr/lib64/libglib-2.0.so.0.4200.2)
==17549==    by 0x4E595C3: sr_session_datafeed_callback_add (session.c:512)
==17549==    by 0x409527: load_input_file_module (input.c:111)
==17549==    by 0x4097AB: load_input_file (input.c:157)
==17549==    by 0x40531E: main (main.c:288)

session: fix use after free of session->devs as reported by valgrind

==7478== Invalid write of size 8
==7478==    at 0x4E59182: sr_session_dev_remove_all (session.c:302)
==7478==    by 0x4E591CD: sr_session_destroy (session.c:265)
==7478==    by 0x4095D9: load_input_file_module (input.c:143)
==7478==    by 0x4097AB: load_input_file (input.c:157)
==7478==    by 0x40531E: main (main.c:288)
==7478==  Address 0x7877eb8 is 88 bytes inside a block of size 96 free'd
==7478==    at 0x4C2A37C: free (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==7478==    by 0x4E5F454: sr_input_free (input.c:573)
==7478==    by 0x4095C3: load_input_file_module (input.c:140)
==7478==    by 0x4097AB: load_input_file (input.c:157)
==7478==    by 0x40531E: main (main.c:288)

Make all sizeof() consistently use parenthesis.

sysclk-lwla: Simplify trigger mask generation

sysclk-lwla: Drain pending replies on initialization

When opening the device, drain any pending data from the USB
buffers so that the device won't get stuck on crashes etc.

sysclk-lwla: Remove double USB set configuration

This was a pointless attempt to make the reset hiccups go away.
It didn't help, the problem must be something else.

sysclk-lwla: Cut down on size_t overuse

Do not use size_t for values whose width is defined by the device,
not the host. Also don't use size_t for simple indices with known
small range, unless type compatibility considerations apply.

sysclk-lwla: Fix copy'n'paste mistake in comment

Use the ALLZERO macro in more places.

Support for Korad KA3005P

korad-kdxxxxp: Rename driver to korad-kaxxxxp.

This matches the supported / supportable devices better.

doxygen: Only use @since on public API functions.

z60_libsigrok.rules: Add sigrok fx2lafw VID/PID pairs.

VCD input: Chunk up the samples in 1MB blocks.

This gives about 100x speed improvement when converting VCD->SR.

Also should allow practically unlimited number of channels.

This fixes bug #551.

session: Remove debug spew on I/O events

This really is a bit much with some drivers that do lots of small
I/O transfers.

contrib: Add SysClk LWLA1016 to udev rules

sysclk-lwla: Implement support for LWLA1016

Refactor the sysclk-lwla driver to separate the generic logic from
the model-specific implementation. Based on this, implement support
for the SysClk LWLA1016 device.

sysclk-lwla: Remove global driver instance pointer

Obtain the sr_dev_driver pointer from the device instance so that
the remaining references to the global di pointer can be removed.

sr_usb_find(): Increase the 'bus' limit to 255.

On some systems it can happen that the USB 'bus' number is a lot larger
than 64, but sr_usb_find() currently errors out if it is > 64.

Example:
Bus 250 Device 006: ID 1ab1:04ce 1ab1 DS1000Z Series[...]

Increase that limit so that the code will work everywhere. This bus number
is queried via libusb_get_bus_number() which returns an uint8_t, so we're
limiting to 255 here.

Thanks to 'ssi' on IRC for reporting the issue.

serial-dmm: Add support for Velleman DVM4100 & PeakTech 3415.

(both are using the new dtm0660 DMM parser)

Add dtm0660 15-byte DMM protocol parser.

usb.c: Fix usb_get_port_path() issue on Mac OS X.

Apparently (some versions of) Mac OS X have the same problem with
usb_get_port_path() as FreeBSD does. Work around this in the same way.

Thanks to hanyazou@gmail.com for the patch!

This fixes bug #673.

chronovu-la: Add missing libusb dependencies.

The chronovu-la driver now uses libusb (in addition to libftdi), so add
a missing <libusb.h> #include and libusb dependency in configure.ac.

chronovu-la: Properly handle multiple devices.

The driver should now be able to cope with e.g. multiple ChronoVu LA8
and/or ChronoVu LA16 devices being connected to the same PC.

It now also provides the serial number and connection ID, which can be
used by frontends to differentiate multiple devices.

Also improve the scanopts / drvopts / devopts lists handling.

This fixes bug #504.

asix-sigma: Drop unneeded asix_sigma_driver_info.

This is not needed, but also causes linker issues on some platforms.

Constify a lot more items.

This fixes various compiler warnings when -Wdiscarded-qualifiers is used.

build: Delete generated files on make clean

build: Enable uninstall of Java bindings

This is needed for make distcheck to work.

build: Skip Python bindings during distcheck

setup.py attempts to relink the bindings at installation time,
which breaks distcheck. Thus, disable the Python bindings during
distcheck for now.

build: Distribute missing files

build: Make ChangeLog generation distcheck-safe

Constify sr_dev_driver::name and sr_dev_driver::long_name.

This fixes a bunch of warnings when compiling with -Wwrite-strings.

asix-sigma: Split into api.c and protocol.[ch] modules.

Java: Remove explicit constructor deletes since SWIG 2 doesn't like them

  bindings/java/org/sigrok/core/classes/classes.i:247: Error: \
    Syntax error in input(3).

Java: Fix leaking global refs

This fixes issue #690.

Java: Don't use JNIEnv* captured by lambdas, it may be invalid for the context

Java: Fill jlongs with 0 when doing SWIG style pointer marshalling

This prevents valgrind complaints and also makes pointer comparison
from the Java side work.

Revert "Java: Avoid dangerous writes via casted pointers"

This reverts commit 865de99391194e45a0e80ea534fc5e27d5287925.

This fixes bug #688.

python: Fix string conversions for Python 3.

Fixes bug #478.

build: Set CXXFLAGS when building Python module

Set both CFLAGS and CXXFLAGS when executing setup.py to build
the Python bindings. Newer versions of distutils/setuptools have
apparently started to pick up the latter when compiling C++.

build: Match up Python headers with interpreter version

Check for either Python 2 or Python 3 header files depending on
the version of the Python interpreter. Also require the module
version to exactly match the interpreter version.

This may fix bug #645.

openbench-logic-sniffer: Avoid recreating event source

With the current driver API and the corresponding session event
handling, it is not possible to destroy and then re-create an
event source with the same key within the same main loop iteration.

The next generation driver API will fix this problem. But for now,
just change the driver to make do without that sort of thing. Also
increase the I/O timeout to 100 ms to be safer in the event of all
kind of delays the OS environment may induce.

This fixes bug #678.

Remove obsolete SR_INPUT_META_MIMETYPE metadata key.

This was never really implemented, since getting the mimetype of a
file or stream in a cross-platform way is a gigantic tangle.

input/csv: Remove obsolete mimetype format match.

Mimetypes never worked, and in any case this caused a segfault due to a
missing SR_INPUT_META_REQUIRED flag.

This fixes bug #681.

build: Also look for python-2.7 pkg-config module

Apparently Gentoo names the module python-2.7 instead of python27.

sysclk-lwla: Read full 64 bit of capture duration field

Evaluate all 64 bit of the duration field in the capture status
record. Although unlikely in practical use, due to compression
it is possible for the duration in ms to exceed 32 bit.

sysclk-lwla: Define constants for long registers

sysclk-lwla: Clarify use of SRAM control registers

Assign more meaningful names to things and introduce new constants.

zip: Provide fallback if zip_discard() is unavailable

zip_discard() was introduced in libzip 0.11, which some systems
do not have yet. Provide a fallback replacement for zip_discard(),
and reduce the requirement to libzip 0.10 again.

This fixes bug #674.

C++: Declare std::default_delete friend as struct

std::default_delete<> is originally defined as struct, not class.
This does not really make much of a difference, but some compilers
complain when the struct/class tag is not consistent across all
declarations of the type.

C++: Suppress deprecation warnings from glibmm

Ignore warnings due to use of std::auto_ptr<> in the glibmm headers.
This should be reverted once glibmm is fixed.

build: Use common SWIG defines for Python and Java

Make the Python and Java bindings use the same set of preprocessor
macros for the SWIG parsing stage, taken from a make variable. Add
G_GNUC_{BEGIN,END}_IGNORE_DEPRECATIONS to that set.

sysclk-lwla: Read test word twice during initialization

During initialization of the LWLA1034, read the 64-bit test word
twice and verify the result of the second read only. This better
matches what the original vendor software does.

sysclk-lwla: Clarify function of registers 10B0 to 10BC

Apparently, these four registers form an interface for indirect
access to another internal 64 bit wide memory. This is likely the
same memory as that accessed by the bulk transfer commands 7 and 8.

Makefile.am: Avoid non-portable -t option for $(INSTALL).

The 'install' tool doesn't have a -t option on all platforms
(e.g. Mac OS X or FreeBSD).

korad-kdxxxxp: Minor cosmetics and consistency changes.

Over voltage and current protection support

Developed against Velleman LABPS3005D and seems to work.

Support for regulation status and fix for mysterious M

Added support for SR_CONF_REGULATION which returns value for CH1
Also VELLEMAN LABPS3005D (only device currently supported) sends single
'M' character in beginning of return value, which is specially discarded.

Java: Avoid dangerous writes via casted pointers

Writing to an object through a pointer of incompatible type is
really evil. Even when the data size matches, it is still a
violation of strict aliasing rules.

Replace all instances by direct casts of the value, without the
unnecessary and dangerous indirection.

SWIG: Avoid unseparated template closing brackets

SWIG 2.0.12 fails with a syntax error on ">>".

C++: Catch exceptions from stoi() and stod().

Fixes bug #483.

java: Add doxygen output directory to .gitignore.

java: Usable wrapping of Configurable.config_keys().

Makefile.am: Make SWIG wrappers also depend on new templates.i file.

python: Give all enum values __doc__ strings.

python: Add docstrings for enum constants.

java: Add docstrings for enum constants.

C++: Include enum classes when generating documentation.

C++: Declare namespace in enums.hpp so it can be used independently.

java: Make enum values available as normal constants.

Java: Remove obsolete SourceCallback interface

C++: Avoid const ref args to appease Java bindings

The Java bindings currently have some weird problem with function
arguments passed by const reference. Not all types are affected,
but the collection types that involve custom typemaps are.

For now, revert back to pass-by-value for the problematic types.

C++: Use C++98 syntax for default arguments to appease SWIG

Looks like SWIG silently ignores default arguments specified via
aggregate initialization. This is rather unfortunate, especially
if the argument types are complex.

C++: Replace custom deleters with std::default_delete

Replace custom Deleter classes with std::default_delete<>, declared
as friend so it can invoke the private destructor. Inexplicably,
std::shared_ptr<> does not use default_delete<> by default, so it
is still necessary to explicitly specify the deleter when creating
shared_ptr instances.

With this, unique_ptr and shared_ptr instances now use the same
default delete mechanism.

C++: Use smart pointers instead of manual delete

Make use of std::unique_ptr<> to manage the lifetime of members
or container elements, so that manual invocations of delete can
be avoided. This also provides for exception safety.

Since std::unique_ptr<> is only movable but not copyable, adapt
the code to avoid copies and assignments of these pointers.

C++: Move C struct pointers out of ownership classes

Reduce needless over-generalization. There is no design need
for the ParentOwned and UserOwned classes to contain the C base
struct pointer. Instead, just make the _structure pointer a
private member of each class that needs one.

C++: Rename get_shared_pointer() to share_owned_by()

This makes it clearer that this method assigns the parent
(owner) reference.

C++: Use shared_from_this() exclusively on this

Never call shared_from_this() on any object other than "this".
Adapt the API so that it can be made protected.

C++: Make most members private instead of protected

Use protected only for members which are actually needed by
sub-classes. Declare all the rest private.

SWIG: Hack around SWIG segfault on private destructors

Apparently this problem has been fixed in SWIG 3.0.6. However,
until we can require that version, define "private" as "protected"
when running the SWIG parser.

C++: Make value get accessors const

Declare accessor methods that return value members const. For now,
skip all cases where constness would have to be applied transitively
to shared objects.

C++: Make some methods static to match the C API

Context::package_version() and Context::lib_version() do not access
context state and should be static. However, leave the logging
related methods alone for now, as making them static would entail
making the callback data a global static, since the C API lacks
destroy notification callbacks.

C++: Declare all callbacks invoked from C noexcept

If one of these functions does throw an exception, std::terminate()
will be called. Without this, the behavior is undefined since the C
stack is not prepared to deal with exceptions.

C++: Use noexcept instead of throw()

Runtime-checked exception specifications via throw() are
deprecated in C++11.

SWIG: Define "noexcept" empty to work around SWIG bug

The SWIG 2.0.12 on my system bails out with a syntax error on the
"noexcept" keyword in C++11 code. Define "noexcept" to nothing (for
the SWIG parser only) to work around this problem.

C++: Make most constructors explicit

Unless implicit conversion is desired, constructors that can be
called with one argument should be marked as "explicit".

C++: Do not use C-style casts

Never ever.

C++: Consistently use nullptr instead of NULL

C++: Use move() and avoid passing containers by value

Make use of std::move() to transfer arguments passed in by value.
Take complex container arguments by const reference, as passing
those by value is rather unorthodox even for C++11 style code.

tests: Drop another obsolete sr_analog_float_to_string() test.

scpi: Move closing of discovered devices to sr_scpi_scan_resource().

rigol-ds: After successfully finding a device, close it properly.

java: Don't use SWIG attribute mechanism.

Using the attribute mechanism results in badly named wrappers like
getLog_level(), as well as incompletely applied typemaps for templated
container types. If we just avoid this mechanism entirely, we get the
same foo() and set_foo() accessors as we have in the C++ API.

java: Remove overrides for overloaded methods.

These are now wrapped correctly without needing this.

java: Remove need for conversion methods on container wrappers.

SWIG: Declare template specialisations for containers before typemaps.

It seems this is necessary to correctly apply typemaps involving
these template specialisations.