I'm not sure how valuable this test is but put it in the right place
until it's decided.

Fixes #1616

The varnish-counters man page looked so sad for the
counters without long descriptions. Not there yet, but a step
on the way.

In the long description we have sufficient room to
use the full word.

The sms_ counters were removed previously

Fixes #1611

Submitted by:	Muhammad Adeel via github

but do insist on it being "close" if we need that.

Test case by daghf

Fixes #1613

no threads available in the pool, fail the fetch and count it.

Fixes #1620
Fixes #1621

Right Thing instead.

it is sane and pass it to streaming clients, provided we don't munge
the body along the way.

Fixes: #1506

Always interpret C-L in decimal

Found by:   Geoff

emit it as C-L if it looks sane.

Source data from file time on repo.varnish-cache.org.

vtc_log does a pthread_exit when reporting an error condition. When
this happened from within the API callback function for logexpect, the
API internal data structures were left in an inconsistent state,
causing asserts do be thrown during VSLQ_Delete().

Signal the failed expect back from the callback function and do the
vtc_log error level after the API function has returned.

Fixes: #1349