see varnish-cache aa8875a44d587bc5530554963f93fffba99efadd

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

Because we don't copy the subject's terminating null byte, we can't use
strlen().

Fixes #4

and don't work around the previous off-by-one error in WS_Assert_Allocated
any more.
Fixes #3 for master.

This reverts commit bf2b7a78.

There are actually two off-by-one errors, one in WS_Assert_Allocated(),
and another in the use of -1 to have WS_Assert_Allocated() compute
the size of the string, including the null terminator.

We don't need the subject with the null terminator in the ov_t
structure (PRIV_TASK data), because we just need it to get backrefs
from the indices in the ov vector returned from pcre.

Also, there's a system in the VTC tests that may not be obvious.
The b*.vtc tests are about regex matching, c*.vtc for backrefs.

and add a test for it

Fixes #3

Closes #2

The .failed() and .error() methods have been removed.

The .match_dyn method has been removed, in its place we have the
functions match_dyn and backref_dyn.

Closes #1

for example if there was an error initializing the object.

required since 81bc4cef2a5e651a9e34ae5400672a88663db7ef

ensure that it builds with make -j

and it does since 49712a5fae5919e6e333d7164b98ef986b149c62

the addresses used as values point to workspace or static memory,
which are free'd by other means