miniexpect.c: fix always-true comparison

According to <http://www.pcre.org/current/doc/html/pcre2api.html#SEC15>:

> STRING LENGTHS AND OFFSETS
>
> The PCRE2 API uses string lengths and offsets into strings of code units
> in several places. These values are always of type PCRE2_SIZE, which is
> an unsigned integer type, currently always defined as size_t. The
> largest value that can be stored in such a type (that is ~(PCRE2_SIZE)0)
> is reserved as a special indicator for zero-terminated strings and unset
> offsets. Therefore, the longest string that can be handled is one less
> than this maximum.

Therefore check the validity of ovector[1] with

  ovector[1] != ~(PCRE2_SIZE)0

rather than the always-true

  ovector[1] >= 0

This was caught by gcc's

> miniexpect.c:359:45: warning: comparison of unsigned expression in ‘>=
> 0’ is always true [-Wtype-limits]

which is part of "-Wextra".

Signed-off-by: Laszlo Ersek <lersek@redhat.com>

Add a bit of explanation of what "expect" is

Kids these days.

docs: Small revisions to the manual and readme file

tests: test-multi-match.c: Fix memory leaks in test

tests: valgrind: Rebuild tests before running them

docs: Fix example in the manual

It still contained references to PCRE functions pcre_exec and
pcre_compile.  Replace with PCRE2 equivalents.

Fixes: commit 9a43a3bea411be5de83c64a2f1d28f365de7133f

Port to pcre2

The last version of PCRE (v1) 8.45 is end of life and will receive no
more updates.  PCRE2 is its replacement and is widely available in
distros.  So port the library to PCRE2.

miniexpect: Debug next_match field

Add debugging capability at runtime.

Debugging can now be enabled at runtime.

When debugging, escape the buffer output.

doc: Use -u (UTF8) flag when generating the man page.

documentation: Fix markup in flags.

Fix documentation of mexp_spawnl to include terminating NULL.

Code is correct, but the documentation was wrong.

int -> const int

Add mexp_send_interrupt, mexp_spawnvf, mexp_spawnlf and various flags.

Set signals to defaults and close other file descriptors after forking.

Small fix to documentation.

Document use of mexp_get_pcre_error when calling pcre_get_substring.

virt-p2v uses this, and it surprised me as well.

Introduce methods to access some fields in the handle.

Actually these are implemented as C macros.

Document how to handle errors from mexp_close.

p2v: properly call va_end

RHEL 6: p2v: Add fix for older PCRE.

RHEL 6 pcre did not define PCRE_PARTIAL_SOFT.  However PCRE_PARTIAL is
a synonym so use that.

p2v: close the mexp handle if mexp_spawnv fails

Document behaviour when matching against multiple regular expressions.

Implement and test multi-matching on strings.

If the subprocess prints multiple matching strings, and you call
mexp_expect repeatedly, the code now does the right thing.

tests: Add 'make check-valgrind' to run test suite under valgrind.

Free handle storage.

Memory leak found by valgrind.

configure: Move AC_USE_SYSTEM_EXTENSIONS to avoid autoconf warning.

tests: Add a test of running 'ls --version' and matching the output.

tests: Accurately list sources of test-spawn program.

Minor updates to the manual page.

clean: Remove miniexpect.3 if we have pod2man, otherwise leave it.

Add a note about clearing the input buffer in mexp_expect.

miniexpect.h: Change the error codes again.

No functional change (although it changes the ABI).

Add a man page.

Move all the documentation from the header file to the man page.

miniexpect.h: Just some tidying up.

mexp_expect: Change the expect API to allow multiple regexps.

Add miniexpect library.