Return int from mexp_expect

GCC 13 highlights the mismatch between int and enum in the declared vs
implemented return type.  The return types are in this case
compatible, but it's not generally safe to implicitly convert between
int and enum.

miniexpect.c:248:1: warning: conflicting types for ‘mexp_expect’ due to enum/integer mismatch; have ‘enum mexp_status(mexp_h *, const mexp_regexp *, pcre2_match_data_8 *)’ {aka ‘enum mexp_status(mexp_h *, const mexp_regexp *, struct pcre2_real_match_data_8 *)’} [-Wenum-int-mismatch]
  248 | mexp_expect (mexp_h *h, const mexp_regexp *regexps,
      | ^~~~~~~~~~~
In file included from miniexpect.c:41:
miniexpect.h:100:12: note: previous declaration of ‘mexp_expect’ with type ‘int(mexp_h *, const mexp_regexp *, pcre2_match_data_8 *)’ {aka ‘int(mexp_h *, const mexp_regexp *, struct pcre2_real_match_data_8 *)’}
  100 | extern int mexp_expect (mexp_h *h, const mexp_regexp *regexps,
      |            ^~~~~~~~~~~

Makefile.am: add "-Wextra -Wshadow"

Both of these options enable useful warnings; let's use them.

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

tests: mark main()'s "argc" as unused

Otherwise gcc emits warnings such as

> test-spawn.c: In function ‘main’:
> test-spawn.c:30:11: warning: unused parameter ‘argc’ [-Wunused-parameter]
>    30 | main (int argc, char *argv[])
>       |       ~~~~^~~~

when "-Wextra" is combined with "-Wall", and "-Wextra" is useful.

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

tests: add "missing" initializers

With "-Wextra", gcc emits warnings such as:

> test-multi-match.c:58:24: warning: missing initializer for field
> ‘options’ of ‘mexp_regexp’ [-Wmissing-field-initializers]

Although the code is not incorrect, let's just shut up the warnings, so we
can enable "-Wextra" later (which did catch an actual bug; see the
previous patch).

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

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.