The terminating null byte for the C string result is written at
the index given by std::string.size() for the C++ string. It had
been written at one past that index.

WS_Release() must then ensure that the terminating null byte is
within the allocated portion of workspace.

NULL is no longer possible as an ENUM value.

Ensure that the retrieved object was saved via set.add().

Checked with gcc/g++ and clang/clang++.

Otherwise builds can fail due to partially written files from vmodtool
running on another target.

vrt.h is included by cache.h, and may not be included after it.

Take advantage of the fact that the array of indexes returned from
the vre2 call is now sorted (but still using a linear search).

for some early versions of re2.

Also get rid of an unnecessary #include "config.h".

Mention autoconf-archive and C++ build requirements and setting
ACLOCAL_PATH.

Closes #2

Please replace this with a better solution if known:

When re2 is installed in a non-standard location, pkg-config --libs re2
(as invoked by PKG_CHECK_MODULES) returns -L<path>, but not -R<path>,
so we fix it up to get the right rpath in the vmod so.

Closes !1

References merge request !1

This reverts commit 35e21c5f.

No longer needed since the fix for Varnish issue 2449.

See Varnish issue #2449