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.

README: Cosmetics, update URL.

updated URLs and description

restore pre-split ignores

device detail now lists probes

cli: support for new output module API

cli: fix driver option conversion

cli: use --show for PD detail

cli: obsolete SR_HWCAP_PROBECONFIG

cli: use new sr_dev_config_set() API call

cli: don't try to set device options before opening it

cli: use new API calls instead of static option data

cli: rename sr_hw_hwcap_get() to sr_devopt_get()

cli: use new generic arg parser API

cli: don't call sr_session_stop() when SR_DF_END is seen

This was effectively an infinite loop in some cases.

cli: fix session saving

cli: code cleanup

cli: use probe numbers starting from 0

cli: change to new struct sr_dev_inst based libsigrok API

cli: remove dead code

cli: add --show option, and switch to new info_get driver API

cli: add --driver option, and switch to new sr API scanner

cli: more flexible generic arg parser

Allow setting samplerate when reading binary files

I had a binary file that I needed to decode using UART decoder. UART
decoder needs to know the sample rate for the data, but currently it's
not possible to pass parameters to input formats and so the "binary"
file format always sets the samplerate to 0.

This patch adds the possibility to append a colon-separated list of
key=value options to the -I argument, in the same way -d supports it.
Also, it makes the "binary" format support the "samplerate" option.

I included the GHashTable containing input format options directly in
the sr_input struct. I'm not sure if that's the right way to do it. I
saw that -d uses a much more elaborate system with device capabilities
and typed options, but that seemed like an overkill for input formats.

cli: Use sr_session_stop() instead of sr_session_halt()

Use sr_session_stop() instead of the deprecated sr_session_halt().

Signed-off-by: Lars-Peter Clausen <redacted>

cli: switch to new <libsigrok/libsigrok.h>

cli: flush output on write

cli: Fix typo in protocol decoder stacking example

Probe assignments are separated by colon and not by comma.

Signed-off-by: Lars-Peter Clausen <redacted>

sr/srd/cli: require glib version >= 2.28

Needed for g_get_monotonic_time(). Thanks to Tomaž Šolc.

cli: Add missing fflush(stdout) for PD annotation output.

This is required/helpful to be able to pipe sigrok-cli PD output into
other tools, like this:

  sigrok-cli -i ... -a ... | othertool

cli: use output API for analog and frame begin/end packets

cli: not doing sigrokdecode directly from analog data

It's just a completely different can of beans.

cli: output API says to check data_out, not length_out

cli/gtk/qt: Now all require libsigrok >= 0.2.0 (API changes).

cli: strip off PD arguments before selecting for annotation

This fixes the bug where -A didn't work if the PD in question had arguments.

sr: Made hwcap const

cli: support for SR_HWCAP_COUPLING

cli: support SR_HWCAP_VDIV and the new SR_T_RATIONAL_VOLT type

cli: use SR_T_RATIONAL_PERIOD

cli: support for SR_HWCAP_FILTER and SR_DI_FILTERS

cli: parameter support various new HWCAPs and types

cli: support for frames, and limiting capture by number of frames.

cli: use SR_DF_META_*, support for analog packets

cli: configure.ac: Bump version number to 0.3.1.

cli: Constify a few char *pointers.

cli: Update NEWS in preparation of 0.3.1 the release.

cli: Cosmetic fix for --help output.

cli: Fix compile error.

cli: Make --protocol-decoder-annotations plural.

This option supports (will support) multiple annotations from multiple
PDs, as well as multiple annotation formats per PD, so use plural.

cli: Add -A (alias for --protocol-decoder-annotation).

cli: manpage cosmetics, updates.

sr/srd/cli/gtk: ChangeLog -> MAINTAINERCLEANFILES.

If the (generated) ChangeLog file is marked for DISTCLEANFILES, it'll be
removed whenever "make distclean" is called (and is then gone forever if
you use the tarballs, for example).

cli: Output which file format was autodetected.

document --protocol-decoder-annotation, and note PD stack is optional.

cli: factor out output format setup

main() is now clean.

cli: option to specify which annotation(s) to show

Also factor out PD stack setup, and remove redundant error messages.

cli: stop immediately if SRD PD registration fails

cli: loose string comparison helper