cosmetic: a few updates/changes to debugging messages and assertions

minor: now that we're passing StoreIOState to these callbacks again,
we can replace the assertion that "mem->swapout.sio == sio".

cosmetic: don't need to use getRaw() when checking for NULL pointer

Make access_log.cc compile again with --enable-forw-via-db after
CacheManager changes.

Removed comm_accept_check_event() and related code since we have
the AcceptLimiter class which does a better job of deferring new
connections when we start to run out of FDs.

Bug #1554: HTCP RFC compliance

Fix field order to comply with the RFC. Bump minor version to 1 to
make migration easier. New cache-peer option to support old Squid HTCP
implementaitons.

Bootstrapped

Suspend 2.5 snapshot releases, hopefully for good.

snapshot 2.6 releases

Move clientCachable logic onto HttpRequest, as all the used fields in the
function where on the HttpRequest object.

Apply Gonzalo Aranas bugfix for the replacement to cachemgrRegister,
as having some options disabled/changed did not compile.

Remove obsolete stub file.

Bug #1536: Finding out what word in acl list is blocked

Add a CacheManager class which provides the cachemanager menu registration facility, and remove all compile time dependencies upon cachemgrRegister from squid outside of main.cc. This allows much simpler tests, as modules exporting cache mgr information no longer drag in all of squid via cache_manager.cc.

Forward port of latest 2.6 changes

Include getopt.h

Bootstrapped

Use ccpunit-config to get information about the system installed ccpunit
instead of hardwired probes

unbreak testStore testcase.

Fixed TestSwapDir test after I changed StoreIOState.

Fix that strange looking "Notice: Short Strings is 36 bytes instead of requested 36 bytes"
message to say what it meant to say..

There is still code relying on fd_set so we must keep the FD_SETSIZE
filedescriptor limit for a while still...

Rename no_cache to cache to better reflect the logics of the directive
(no_cache kept as an alias)

cosmetic: The debug message "WARNING! Your cache is running out of
filedescriptors" appears in two places.  I changed it to debugs()
macro with HERE argument to differentiate the two messages.

nitpick: This "Fc = &fdc_table[fd]" optimization seems like a waste here.

cosmetic change: rename doCallback to doCloseCallback to clarify
what this function does.

Remove redundant newline in cache.log

patch by Pawel Worach

Bootstrapped

Block copy constructor. Not safe.

Should also block the assignment constructor, but it is currently in use
(privately) as part of the reset method.

cleanup of how the Vary marker object is created
to look more like a normal object creation, not violating layers too much

Fix NULL pointer access with asyncio when Squid starts up over the disk
space limit.

We used to call squidaio_init() from DiskThreadsDiskFile::operator new.
But this only gets called when opening a file.  If the first transaction
is to unlink a file, squidaio will be uninitialized.

I didn't see any other good places to call squidaio_init().  For now
I am calling it from within the squidaio module itself at the
start of each open, unlink/truncate, and stat call.

-mc command line option to disable memory pools

Fix a memory corruption when building the magic Vary marker object headers

somehow this cast made things go very bad. Luckily it isn't needed as
we already hold a reference to the reply a few lines earlier.

Fix --disable-carp

StoreIOState typedef fixes for Null storage scheme

Bootstrapped

StoreIOState updates for COSS

dbopen configure test typo

Added "self" refcounted Pointer parameters to StoreIOState's STRCB, STFNCB,
and STIOCB callback functions.  This is so that the StoreIOState
refcount doesn't go to zero during one of these callbacks.

Do the dbopen compile test proper to see if -ldb is needed

Moved STRCB, STFNCB, STIOCB typdefs to inside StoreIOState class itself.
This should allow us to use these types as parameters in the callbacks.

Fix COSS so it works with my recent StoreIOState changes.

Bootstrapped

Do a full compile test of dbopen to see if -ldb is needed

From 2.5: Properly quote wbinfo arguments, just in case.

2.5 commit log reads "Minor fix for Samba-3", but later changes made it
work anyway..

Merged storeIOState and StoreIOState into a single StoreIOState
class.  Previously, StoreIOState was just a refcounted storeIOState.

I had concerns that mixed use of refcounted and non-refcounted
StoreIOState may be leading to cases where the object gets referenced
via the old-style pointer after the refcounted version had its count
go to zero and the memory was freed.  This came about while
investigating bugzilla #1465.

removed StoreIOState-related prototypes from protos.h, and removed
some typedefs from typedefs.h.

Modified STFNCB (storeSwapInFileNotify, storeSwapOutFileNotify) and
STIOCB (storeSwapOutFileClosed, storeSwapInFileClosed) so that they
no longer take StoreIOState arguments.  Their "data" arguments
easily lead to the corresponding StoreIOState.  The only thing we
really lose here is the ability to assert that "data->sio" equals
the passed StoreIOState.

temporary hack to handle case where body_reader becomes NULL
in http module before sendRequestEntity gets called.

Bootstrapped

Fix http_range test to run again.

Minor cleanup of the meta TLV format to not add the incorrect STORE_META_END
at the end of the list (list ends by it's size). STORE_META_END has changed
a number of times over the history, and we have never cared about it but some
external tools (i.e. purge) apparently beleives what they read in the docs..

Made stream outputting more consistent in Mem::PoolReport.  Use a
const char * for the column delimiter instead of repeating "\t "
on each line.  Always put the delimiter at the end of the (source code) line.

Try to clean up cachemgr 'mem' output.  Don't need to double up on
'%' now that we're not using printf.  Use std::setprecision() on some
fields so they don't show up as "1e+01" etc.

struct lock_ctrl_t and callback type SIH seem to be unused anywhere
in the code.

Converted storeRelease() to a StoreEntry method.

Fully deprecated storeLockObject().  Everything now directly calls the
StoreEntry->lock() method.

Fully deprecated storeUnlockObject().  Now everything directly uses
the StoreEntry unlock() method.

put correct function names in debugging statements

This debugging statement should not be at level 0.

The patch for bug #1499 included debugging statments where printf tokens
did not match time_t type on FreeBSD.  The time_t variables are now cast
in debugging calls.

Bug #1475: Squid does not need to be limited by FD_SETSIZE when using kqueue/epoll.

Technically poll doesn't need to be limited either, but is due to the delay
pool hooks still using fd_set...

Inspired by the patch by Gonzalo Arana

Bug #1501: ftp downloads may last up to read_timeout
by Gonzalo Arana

Bug #1499: external acls are somewhat broken
by Gonzalo Arana

Give I/O threads a smaller stack.

Added usage of configure results

Syslog is not available on all platforms

Need SquidTime for squid_curtime

Bug #1489: add syslog priority and facility to access_log

Patch provided by Gonzalo Arana: Thanks !

Bug# 1430: feature: multi-line directives in squid.conf

Patch provided by Gonzalo Arana. Thanks !

Bug #1347: Modify squid MIB to incorporate squid.conf visible_hostname

Patch provided by John Horne, thanks !

Don't disable assert on PURIFY

Bug #1553: Memleak in HTCP client code

Forward port of 2.5 changes.

Fix Cygwin build errors.

Bug #1532: Mime icons are not displayed when viewing ftp sites when
visible_hostname is a short hostname (without domain).

Forward port of 2.5 changes.

Bug #1558 (cosmetic): cachmgr leaks a stdio buffer

Bootstrapped

bug: ICAPAccessCheck should use cbdataReference and cbdataReferenceValidDone
in case the upstream ClientRequestContext goes away during processing.

need to include <netinet/in.h> on FreeBSD for definition of sockaddr_in

Refcount::Pointers don't need getRaw() when comparing to NULL

bug: clientAbortBody should not try to close the connection
if it is already being closed.

Need config.h before any system heaers

Bootstrapped

Forgotten to dist SquidTime.h

Fix copy-paste install dependenices when separating cachemgr & squidclient
from src

Bootstrapped

Move squidclient and cachemgr from src to tools

Need SquidTime for squid_curtime

Put squid.h before <iosfwd> to avoid FD_SETSIZE redefined errors
with 'make check'

Bootstrapped

Unwind HTCP dependencies

Bootstrapped

util.c depends on assert.c

Bootstrapped

Also need a generic xassert to go with out assert, to be used by helpers
etc.

Broke out assert define into it's own include. Relied upon by MemPool

Take 2 of the FD_SETSIZE redefined fix.. eleminate the need of squid.h
completely (was only needed for definition of NULL which isn't needed in C++)

Fix include order to make sure config.h is always first (indirectly via squid.h)

Bug 1224 / 1530: Crash related to failed PUT/POST requests where
body processing may close the connection "in the middle" of client
side request processing.

Bug #1530: Make "foo" a more intuitive error message by just using
assert instead.

Need SquidTime.h to squid_curtime

Removed the HttpHeaderEntry constructor that created a "blank" object.
It created problems with the parse() method because id had to be set
to something, but none of the http_hdr_type values were really appropriate,
not to mention they either triggered assertions or confused the statistics.

Now parse() is a static method that calls new HttpHeaderEntry after
parsing has been successful.

Bug #1531: squid crashes short after starting

When HttpHeaderEntry was converted to a class with methods I added a
constructor that took no arguments. It initialized id to HDR_BAD_HDR.
If header parsing fails, HDR_BAD_HDR triggers an assertion in
the destructor.  As a workaround id is now initialized to HDR_OTHER.