sigrok-cli: Fix two memory leaks in parse_probestring().

This patch fixes two memory leaks if multiple probe names or probe ranges are
specified.

Use annotation-specific struct

Revert "Use annotation-specific struct".

This reverts commit fa498bbb3baf5d9bf62fe0e013aaf434cb2732fd (which
was accidentally committed).

Update man page for --probe-group

Use annotation-specific struct

Report options requiring probe groups by passing them in if given

New --probe-group option

Devices that don't use probe groups don't need this option at all;
it only needs to be specified when using a device that has probe
groups, and then only for options that require it.

Report probe groups with --show

Probe group API fixes

Use new probe_groups API

sigrok-cli: Fix leaks in device_scan().

If a bogus driver name is specified on the command line or for some reason we
fail to initialize a driver device_scan() leaks data.

This patch adds appropriate free calls.

Adjust to libsigrokdecode API changes

Actually exit when a fatal error is logged

Use the new libsigrokdecode session API

sigrok-cli: Allow both hex and decimals parameters

When strtoll() base parameter is set to 0 it can handle both hex and
decimals values.

Signed-off-by: Matt Ranostay <redacted>

Fix issue with optional PD probes.

Running (or showing info about) PDs with optional probes was not working
and causing segfaults.

The reason was a g_slist_concat() call that was applied to a list,
whereas it needs to be applied to a copy of the list.

This fixes #138.

List dummy "sigrok" file format, and accept it as option

See bug 131

Remove datafeed callbacks to prevent memory leak.

Avoid trying to save to a file if there's nothing to save

This works around the division by zero when trying to save analog
data.

NEWS: Fix typo in the year.

Thanks codyps1 on IRC for the report.

More generic handling of SR_T types, and SR_T_UINT64 support

manpage: Escape minus characters.

Fix error message and compiler warning

Thanks to Christian Deussen for the patch.

Fix various memory leaks

configure.ac: Update version number to 0.4.0.

NEWS: Add changes since the last release.

Adapt to new <libsigrokdecode/libsigrokdecode.h> header.

sigrok-cli depends on libsigrokdecode >= 0.2.0.

Update man page

Update cmdline options to better reflect functionality

Remove anykey input source after it fired

Fix error message

Use new output API receive() function

Open device before adding it to the session

GPL headers: Use correct project name.

README: Add "Copyright and license" section.

Add NSIS file for use with cross-compiled sigrok-cli.

Show driver name and optional conn string

Update for libsigrok datafeed callback API changes.

Small variable naming fix to avoid confusion.

configure.ac: Show $build, $host, and lib versions.

SR_CONF_TIMEBASE and _VDIVS lists are now an array of tuples

More GVariant conversions

fix options setting to use GVariant

avoid deprecated struct sr_rational

More GVariant conversions

Adjust to GVariant-based sr_config_* functions

fix time-based limit logic

use new libsigrok GVariant-based option/probe passing

README: Refer to INSTALL and new Building wiki page.

autotools: Don't use "foreign" option.

This adds INSTALL and allows for a few more sanity checks being done.

show SR_CONF_SAMPLE_INTERVAL (debug)

make libsigrokdecode optional

Only if you really need to compile without libsigrokdecode.

support for live status of SR_CONF_DATALOG

add --set option

dump obsolete multiple-device leftover code

fix SR_DF_END flow

Output modules were getting SR_DF_END after their object was already
free()d, and the output channel closed.

properly check for result after output module data()

use input API properly

Update, fix, and extend manpage.

 - Probe indices start at 0 now (not 1).
 - Update manpage date.
 - Mention the 'analog' output format.
 - Various smaller cosmetic fixes.
 - Add more examples for some options.
 - Document all possible suffixes for --time/--samples values.
 - Mention the PulseView(1) manpage in "SEE ALSO".
 - Mention Bugzilla for bugreports.

Use sr_dev_has_option() where appropriate.

Use sr_dev_has_option().

Use sr_config_get().

Use sr_config_set()

Deprecate SR_DI_HWOPTS.

Deprecate SR_DI_HWCAPS.

Deprecate SR_DI_TRIGGER_TYPES.

Deprecate SR_DI_TRIGGER_SOURCES.

Deprecate SR_DI_TIMEBASES.

Deprecate SR_DI_FILTERS.

Deprecate SR_DI_VDIVS.

Deprecate SR_DI_COUPLING.

Deprecate SR_DI_BUFFERSIZES.

Deprecate SR_DI_PATTERNS.

Deprecate SR_DI_CUR_SAMPLERATE.

deprecate SR_DI_SAMPLERATES

Use new SR_CONF_* keys.

Use new sr_config_info*_get() calls.

handle meta packets, specifically SR_HWCAP_SAMPLERATE

It's actually unused, but a good example of how to handle these.

use new sr_config struct

update to newer options

Makefile.am: Fix typo in 'ChangeLog' target.

don't use the obsolete datastore.

This also removes the 64-probe limit.

Additionally, sigrok-cli now no longer requires a meta packet to be sent;
this was an entirely avoidable limitation.

parserc.c: Fix memory leak from g_strsplit

g_strsplit() returns a newly-allocated NULL-terminated array of strings.
g_strfreev() must be used to free it, once it is no longer needed.

The probe 'names' were not freed properly, causing a "definitely lost" memory
leak.

Call g_strfreev(names) to properly free the memory.

Signed-off-by: Alexandru Gagniuc <redacted>

sigrok-cli.c: Fix memory leak in parsing of hardware options

During parsing of hardware options, hash_to_hwopt() creates a GSList that is
passed to device_scan(). The list is discarded by device_scan() and never freed.
This is identified by valgrind as "definitely lost".

Call g_slist_free_full() on the list when it is no longer needed.

Signed-off-by: Alexandru Gagniuc <redacted>

Corrected a comment typo

clean up unused code

use constified SRD decoder list

fix device-specific options setting

configure.ac: Fix CFLAGS settings.

Don't override/overwrite CFLAGS in configure.ac, but rather amend it
with (currently) "-Wall -Wextra".

This properly allows users/packagers to do things like:

 ./configure (this will default to using "-g -O2" additionally)

 CFLAGS="-g -O2" ./configure (same as above)

 CFLAGS="" ./configure (no additional flags)

 CFLAGS="-g -O0" ./configure (disable optimization, e.g. for valgrind use)

 etc. etc.

Make datafeed_in packet parameter a const pointer

This patch corresponds to bf53457d in libsigrok.

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().

Drop unneeded comments.

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.

Thanks to the new struct sr_context * the error path code flow can also
be simplifed, and allow a single point in main() to clean up after any
possible program state.

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

Move sr_exit() in main() to correctly mirror order of initialization

Re-add HACKING file after repo split.

Merge branch 'master' of ssh://sigrok.org/data/git/sigrok-cli

more compact display of probe list

don't accept numbers as probe indexes in probe string

Unless the driver actually names them by number -- but the comparison
is done by string now.

Syntax for --probes is unchanged; even a range like 0-7 still works,
but only if there really are probes named "0", "1" etc.