- Improved high-resolution profiling
- Windows overlapped-IO and thread support added to the Async IO disk code
- Improvements for handling large DNS replies
- - Adds standard API for string handling
Changes to squid-2.5 ():
/*
- * $Id: ACLExtUser.cc,v 1.10 2007/05/18 06:41:21 amosjeffries Exp $
+ * $Id: ACLExtUser.cc,v 1.11 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 28 Access Control
* AUTHOR: Duane Wessels
ACLExtUser::match(ACLChecklist *checklist)
{
if (checklist->request->extacl_user.size()) {
- return data->match(checklist->request->extacl_user.c_str());
+ return data->match(checklist->request->extacl_user.buf());
} else {
return -1;
}
/*
- * $Id: ACLHTTPHeaderData.cc,v 1.4 2007/05/18 06:41:21 amosjeffries Exp $
+ * $Id: ACLHTTPHeaderData.cc,v 1.5 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 28 Access Control
* AUTHOR: Duane Wessels
if (hdr == NULL)
return false;
- debugs(28, 3, "aclHeaderData::match: checking '" << hdrName << "'");
+ debugs(28, 3, "aclHeaderData::match: checking '" << hdrName.buf() << "'");
- string value = hdrId != HDR_BAD_HDR ? hdr->getStrOrList(hdrId) : hdr->getByName(hdrName.c_str());
+ String value = hdrId != HDR_BAD_HDR ? hdr->getStrOrList(hdrId) : hdr->getByName(hdrName.buf());
- return regex_rule->match(value.c_str());
+ return regex_rule->match(value.buf());
}
wordlist *
ACLHTTPHeaderData::dump()
{
wordlist *W = NULL;
- wordlistAdd(&W, hdrName.c_str());
+ wordlistAdd(&W, hdrName.buf());
wordlist * regex_dump = regex_rule->dump();
wordlistAddWl(&W, regex_dump);
wordlistDestroy(®ex_dump);
char* t = strtokFile();
assert (t != NULL);
hdrName = t;
- hdrId = httpHeaderIdByNameDef(hdrName.c_str(), hdrName.size());
+ hdrId = httpHeaderIdByNameDef(hdrName.buf(), strlen(hdrName.buf()));
regex_rule->parse();
}
bool
ACLHTTPHeaderData::empty() const
{
- return (hdrId == HDR_BAD_HDR && !hdrName.c_str()) || regex_rule->empty();
+ return (hdrId == HDR_BAD_HDR && !hdrName.buf()) || regex_rule->empty();
}
ACLData<HttpHeader*> *
/*
- * $Id: ACLHTTPHeaderData.h,v 1.3 2007/05/18 06:41:21 amosjeffries Exp $
+ * $Id: ACLHTTPHeaderData.h,v 1.4 2007/05/29 13:31:36 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
private:
http_hdr_type hdrId; // set if header is known
- string hdrName; // always set
+ String hdrName; // always set
ACLData<char const *> * regex_rule;
};
/*
- * $Id: ACLUrlPath.cc,v 1.3 2007/05/18 06:41:21 amosjeffries Exp $
+ * $Id: ACLUrlPath.cc,v 1.4 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 28 Access Control
* AUTHOR: Duane Wessels
int
ACLUrlPathStrategy::match (ACLData<char const *> * &data, ACLChecklist *checklist)
{
- char *esc_buf = xstrdup(checklist->request->urlpath.c_str());
+ char *esc_buf = xstrdup(checklist->request->urlpath.buf());
rfc1738_unescape(esc_buf);
int result = data->match(esc_buf);
safe_free(esc_buf);
/*
- * $Id: AuthUser.cc,v 1.8 2007/05/18 06:41:21 amosjeffries Exp $
+ * $Id: AuthUser.cc,v 1.9 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 29 Authenticator
* AUTHOR: Robert Collins
if (!proxy_auth_username_cache) {
/* First time around, 7921 should be big enough */
proxy_auth_username_cache =
- hash_create((HASHCMP *) std::strcmp, 7921, hash_string);
+ hash_create((HASHCMP *) strcmp, 7921, hash_string);
assert(proxy_auth_username_cache);
eventAdd("User Cache Maintenance", cacheCleanup, NULL, Config.authenticateGCInterval, 1);
}
/*
- * $Id: CommonPool.h,v 1.4 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: CommonPool.h,v 1.5 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 77 Delay Pools
* AUTHOR: Robert Collins <robertc@squid-cache.org>
void *operator new(size_t);
void operator delete (void *);
static CommonPool *Factory (unsigned char _class, CompositePoolNode::Pointer&);
- char const* theClassTypeLabel() const { return typeLabel.c_str(); }
+ char const* theClassTypeLabel() const {return typeLabel.buf();}
protected:
CommonPool();
- string typeLabel;
+ String typeLabel;
};
#endif
/*
- * $Id: CompositePoolNode.h,v 1.7 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: CompositePoolNode.h,v 1.8 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 77 Delay Pools
* AUTHOR: Robert Collins <robertc@squid-cache.org>
struct IN_ADDR src_addr;
AuthUserRequest *user;
- string tag;
+ String tag;
};
protected:
/*
- * $Id: ConfigParser.h,v 1.7 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ConfigParser.h,v 1.8 2007/05/29 13:31:36 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
static void ParseUShort(u_short *var);
static void ParseBool(bool *var);
static void ParseString(char **var);
- static void ParseString(string &var);
+ static void ParseString(String *var);
static void ParseWordList(wordlist **list);
static char * strtokFile();
};
/*
- * $Id: DelayBucket.cc,v 1.6 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: DelayBucket.cc,v 1.7 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 77 Delay Pools
* AUTHOR: Robert Collins <robertc@squid-cache.org>
#include "ConfigParser.h"
#include "DelayId.h"
#include "Array.h"
-#include "SquidString.h"
+#include "String.h"
#include "CommonPool.h"
#include "CompositePoolNode.h"
#include "DelayPool.h"
/*
- * $Id: DelayTagged.cc,v 1.6 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: DelayTagged.cc,v 1.7 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 77 Delay Pools
* AUTHOR: Robert Collins <robertc@squid-cache.org>
DelayTaggedCmp(DelayTaggedBucket::Pointer const &left, DelayTaggedBucket::Pointer const &right)
{
/* for rate limiting, case insensitive */
- return strcasecmp(left->tag, right->tag);
+ return left->tag.caseCmp(right->tag.buf());
}
void
::operator delete (address);
}
-DelayTaggedBucket::DelayTaggedBucket(string &aTag) : tag (aTag)
+DelayTaggedBucket::DelayTaggedBucket(String &aTag) : tag (aTag)
{
debugs(77, 3, "DelayTaggedBucket::DelayTaggedBucket");
}
void
DelayTaggedBucket::stats (StoreEntry *entry) const
{
- storeAppendPrintf(entry, " %s:", tag.c_str());
+ storeAppendPrintf(entry, " %s:", tag.buf());
theBucket.stats (entry);
}
-DelayTagged::Id::Id(DelayTagged::Pointer aDelayTagged, string &aTag) : theTagged(aDelayTagged)
+DelayTagged::Id::Id(DelayTagged::Pointer aDelayTagged, String &aTag) : theTagged(aDelayTagged)
{
theBucket = new DelayTaggedBucket(aTag);
DelayTaggedBucket::Pointer const *existing = theTagged->buckets.find(theBucket, DelayTaggedCmp);
/*
- * $Id: DelayTagged.h,v 1.5 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: DelayTagged.h,v 1.6 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 77 Delay Pools
* AUTHOR: Robert Collins <robertc@squid-cache.org>
void operator delete (void *);
void stats(StoreEntry *)const;
- DelayTaggedBucket(string &aTag);
+ DelayTaggedBucket(String &aTag);
~DelayTaggedBucket();
DelayBucket theBucket;
- string tag;
+ String tag;
};
class DelayTagged : public CompositePoolNode
public:
void *operator new(size_t);
void operator delete (void *);
- Id (RefCount<DelayTagged>, string &);
+ Id (RefCount<DelayTagged>, String &);
~Id();
virtual int bytesWanted (int min, int max) const;
virtual void bytesIn(int qty);
/*
- * $Id: AIODiskFile.cc,v 1.5 2007/05/19 05:03:48 amosjeffries Exp $
+ * $Id: AIODiskFile.cc,v 1.6 2007/05/29 13:31:43 amosjeffries Exp $
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
* ----------------------------------------------------------
{
/* Simulate async calls */
#ifdef _SQUID_WIN32_
- fd = aio_open(path.c_str(), flags);
+ fd = aio_open(path.buf(), flags);
#else
- fd = file_open(path.c_str() , flags);
+ fd = file_open(path.buf() , flags);
#endif
ioRequestor = callback;
/*
- * $Id: AIODiskFile.h,v 1.3 2007/05/18 06:41:27 amosjeffries Exp $
+ * $Id: AIODiskFile.h,v 1.4 2007/05/29 13:31:43 amosjeffries Exp $
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
* ----------------------------------------------------------
CBDATA_CLASS(AIODiskFile);
void error(bool const &);
int fd;
- string path;
+ String path;
AIODiskIOStrategy *strategy;
RefCount<IORequestor> ioRequestor;
bool closed;
/*
- * $Id: ESI.cc,v 1.24 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ESI.cc,v 1.25 2007/05/29 13:31:36 amosjeffries Exp $
*
* DEBUG: section 86 ESI processing
* AUTHOR: Robert Collins
*/
return 0;
- if (strstr (sctusable->content, "ESI/1.0"))
+ if (strstr (sctusable->content.buf(), "ESI/1.0"))
rv = 1;
httpHdrScTargetDestroy (sctusable);
/*
- * $Id: ESIAssign.cc,v 1.5 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ESIAssign.cc,v 1.6 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 86 ESI processing
* AUTHOR: Robert Collins
variable = NULL;
if (unevaluatedVariable.size()) {
- varState->feedData(unevaluatedVariable.c_str(), unevaluatedVariable.size());
+ varState->feedData(unevaluatedVariable.buf(), unevaluatedVariable.size());
char const *result = varState->extractChar ();
/* Consider activating this, when we want to evaluate variables to a
if (!value)
return ESI_PROCESS_COMPLETE;
- varState->addVariable (name.c_str(), name.size(), value);
+ varState->addVariable (name.buf(), name.size(), value);
value = NULL;
ESIVariableExpression::~ESIVariableExpression()
{}
-ESIVariableExpression::ESIVariableExpression (string const &aString) : expression (aString)
+ESIVariableExpression::ESIVariableExpression (String const &aString) : expression (aString)
{}
void
ESIVariableExpression::eval (ESIVarState &state, char const *subref, char const *defaultOnEmpty) const
{
/* XXX: Implement evaluation of the expression */
- ESISegment::ListAppend (state.getOutput(), expression.c_str(), expression.size());
+ ESISegment::ListAppend (state.getOutput(), expression.buf(), expression.size());
}
/*
- * $Id: ESIAssign.h,v 1.4 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ESIAssign.h,v 1.5 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 86 ESI processing
* AUTHOR: Robert Collins
public:
~ESIVariableExpression();
- ESIVariableExpression (string const &value);
+ ESIVariableExpression (String const &value);
virtual void eval (ESIVarState &state, char const *, char const *) const;
private:
- string expression;
+ String expression;
};
/* ESIAssign */
void evaluateVariable();
esiTreeParentPtr parent;
ESIVarState *varState;
- string name;
+ String name;
ESIVariableExpression * value;
ESIElement::Pointer variable;
- string unevaluatedVariable;
+ String unevaluatedVariable;
};
MEMPROXY_CLASS_INLINE(ESIAssign)
/*
- * $Id: ESICustomParser.cc,v 1.9 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ESICustomParser.cc,v 1.10 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 86 ESI processing
* AUTHOR: Robert Collins
}
size_t openESITags (0);
- char const *currentPos = content.c_str();
+ char const *currentPos = content.buf();
size_t remainingCount = content.size();
char const *tag = NULL;
ESICustomParser::errorString() const
{
if (error.size())
- return error.c_str();
+ return error.buf();
else
return "Parsing error strings not implemented";
}
/*
- * $Id: ESICustomParser.h,v 1.7 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ESICustomParser.h,v 1.8 2007/05/29 13:31:37 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
char const *findTag(char const *a, size_t b);
ESIParserClient *theClient;
- string error;
+ String error;
/* cheap n dirty - buffer it all */
- string content;
+ String content;
/* TODO: make a class of this type code */
ESITAG_t lastTag;
};
/*
- * $Id: ESISegment.cc,v 1.5 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ESISegment.cc,v 1.6 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 86 ESI processing
* AUTHOR: Robert Collins
void
ESISegment::dumpOne() const
{
- string temp;
+ String temp;
temp.limitInit(buf, len);
- debugs(86, 9, "ESISegment::dumpOne: \"" << temp << "\"");
+ debugs(86, 9, "ESISegment::dumpOne: \"" << temp.buf() << "\"");
}
/*
- * $Id: ESIVarState.cc,v 1.9 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ESIVarState.cc,v 1.10 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 86 ESI processing
* AUTHOR: Robert Collins
}
void
-ESIVarState::removeVariable (string const &name)
+ESIVarState::removeVariable (String const &name)
{
- Variable *candidate = static_cast <Variable *>(variables.find (name.c_str(), name.size()));
+ Variable *candidate = static_cast <Variable *>(variables.find (name.buf(), name.size()));
if (candidate) {
/* XXX: remove me */
void
ESIVarState::addVariable(char const *name, size_t len, Variable *aVariable)
{
- string temp;
+ String temp;
temp.limitInit (name, len);
removeVariable (temp);
variables.add(name, len, aVariable);
if (!subref)
s = state.header().getStr (HDR_COOKIE);
else {
- string S = state.header().getListMember (HDR_COOKIE, subref, ';');
+ String S = state.header().getListMember (HDR_COOKIE, subref, ';');
if (S.size())
- ESISegment::ListAppend (state.getOutput(), S.c_str(), S.size());
+ ESISegment::ListAppend (state.getOutput(), S.buf(), S.size());
else if (found_default)
ESISegment::ListAppend (state.getOutput(), found_default, strlen (found_default));
}
if (state.header().has(HDR_ACCEPT_LANGUAGE)) {
if (!subref) {
- string S (state.header().getList (HDR_ACCEPT_LANGUAGE));
- ESISegment::ListAppend (state.getOutput(), S.c_str(), S.size());
+ String S (state.header().getList (HDR_ACCEPT_LANGUAGE));
+ ESISegment::ListAppend (state.getOutput(), S.buf(), S.size());
} else {
if (state.header().hasListMember (HDR_ACCEPT_LANGUAGE, subref, ',')) {
s = "true";
if (!tempstr[0])
return;
- string strVary (rep->header.getList (HDR_VARY));
+ String strVary (rep->header.getList (HDR_VARY));
- if (!strVary.size() || strVary[0] != '*') {
+ if (!strVary.size() || strVary.buf()[0] != '*') {
rep->header.putStr (HDR_VARY, tempstr);
}
}
+
/*
- * $Id: ESIVarState.h,v 1.3 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ESIVarState.h,v 1.4 2007/05/29 13:31:37 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
class Variable;
void addVariable (char const *, size_t, Variable *);
- void removeVariable (string const &);
+ void removeVariable (String const &);
void *operator new (size_t byteCount);
void operator delete (void *address);
/*
- * $Id: ExternalACLEntry.h,v 1.7 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: ExternalACLEntry.h,v 1.8 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 82 External ACL
* AUTHOR: Henrik Nordstrom, MARA Systems AB
ExternalACLEntryData() : result (-1) {}
int result;
- string user;
- string password;
- string message;
- string tag;
- string log;
+ String user;
+ String password;
+ String message;
+ String tag;
+ String log;
};
dlink_node lru;
int result;
time_t date;
- string user;
- string password;
- string message;
- string tag;
- string log;
+ String user;
+ String password;
+ String message;
+ String tag;
+ String log;
external_acl *def;
private:
/*
- * $Id: HttpHdrCc.cc,v 1.30 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHdrCc.cc,v 1.31 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 65 HTTP Cache Control Header
* AUTHOR: Alex Rousskov
/* local prototypes */
-static int httpHdrCcParseInit(HttpHdrCc * cc, const string * str);
+static int httpHdrCcParseInit(HttpHdrCc * cc, const String * str);
/* module initialization */
/* creates an cc object from a 0-terminating string */
HttpHdrCc *
-httpHdrCcParseCreate(const string * str)
+httpHdrCcParseCreate(const String * str)
{
HttpHdrCc *cc = httpHdrCcCreate();
/* parses a 0-terminating string and inits cc */
static int
-httpHdrCcParseInit(HttpHdrCc * cc, const string * str)
+httpHdrCcParseInit(HttpHdrCc * cc, const String * str)
{
const char *item;
const char *p; /* '=' parameter */
nlen = ilen;
/* find type */
- type = (http_hdr_cc_type ) httpHeaderIdByName(item, nlen, CcFieldsInfo, CC_ENUM_END);
+ type = (http_hdr_cc_type ) httpHeaderIdByName(item, nlen,
+ CcFieldsInfo, CC_ENUM_END);
if (type < 0) {
- debugs(65, 2, "hdr cc: unknown cache-directive: near '" << item << "' in '" << *str << "'");
+ debugs(65, 2, "hdr cc: unknown cache-directive: near '" << item << "' in '" << str->buf() << "'");
type = CC_OTHER;
}
if (EBIT_TEST(cc->mask, type)) {
if (type != CC_OTHER)
- debugs(65, 2, "hdr cc: ignoring duplicate cache-directive: near '" << item << "' in '" << *str << "'");
+ debugs(65, 2, "hdr cc: ignoring duplicate cache-directive: near '" << item << "' in '" << str->buf() << "'");
CcFieldsInfo[type].stat.repCount++;
{
assert(cc);
- if (cc->other.c_str())
- cc->other.clear();
+ if (cc->other.buf())
+ cc->other.clean();
memFree(cc, MEM_HTTP_HDR_CC);
}
if (EBIT_TEST(cc->mask, flag) && flag != CC_OTHER) {
/* print option name */
- packerPrintf(p, (pcount ? ", %s" : "%s"), CcFieldsInfo[flag].name.c_str());
+ packerPrintf(p, (pcount ? ", %s" : "%s"), CcFieldsInfo[flag].name.buf());
/* handle options with values */
}
if (cc->other.size())
- packerPrintf(p, (pcount ? ", %s" : "%s"), cc->other.c_str());
+ packerPrintf(p, (pcount ? ", %s" : "%s"), cc->other.buf());
}
/* negative max_age will clean old max_Age setting */
extern const HttpHeaderStat *dump_stat; /* argh! */
const int id = (int) val;
const int valid_id = id >= 0 && id < CC_ENUM_END;
- const char *name = valid_id ? CcFieldsInfo[id].name.c_str() : "INVALID";
+ const char *name = valid_id ? CcFieldsInfo[id].name.buf() : "INVALID";
if (count || valid_id)
storeAppendPrintf(sentry, "%2d\t %-20s\t %5d\t %6.2f\n",
/*
- * $Id: HttpHdrRange.cc,v 1.43 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHdrRange.cc,v 1.44 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 64 HTTP Range Header
* AUTHOR: Alex Rousskov
{}
HttpHdrRange *
-HttpHdrRange::ParseCreate(const string * range_spec)
+HttpHdrRange::ParseCreate(const String * range_spec)
{
HttpHdrRange *r = new HttpHdrRange;
/* returns true if ranges are valid; inits HttpHdrRange */
bool
-HttpHdrRange::parseInit(const string * range_spec)
+HttpHdrRange::parseInit(const String * range_spec)
{
const char *item;
const char *pos = NULL;
int count = 0;
assert(this && range_spec);
++ParsedCount;
- debugs(64, 8, "parsing range field: '" << *range_spec << "'");
+ debugs(64, 8, "parsing range field: '" << range_spec->buf() << "'");
/* check range type */
- if (strncasecmp(*range_spec,"bytes=", 6))
+ if (range_spec->caseCmp("bytes=", 6))
return 0;
/* skip "bytes="; hack! */
- pos = range_spec->c_str() + 5;
+ pos = range_spec->buf() + 5;
/* iterate through comma separated list */
while (strListGetItem(range_spec, ',', &item, &ilen, &pos)) {
++count;
}
- debugs(64, 8, "parsed range range count: " << count << ", kept " << specs.size());
+ debugs(64, 8, "parsed range range count: " << count << ", kept " <<
+ specs.size());
return specs.count != 0;
}
/*
- * $Id: HttpHdrSc.cc,v 1.5 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHdrSc.cc,v 1.6 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 90 HTTP Cache Control Header
* AUTHOR: Alex Rousskov
/* local prototypes */
-static int httpHdrScParseInit(HttpHdrSc * sc, const string * str);
+static int httpHdrScParseInit(HttpHdrSc * sc, const String * str);
/* module initialization */
/* creates an sc object from a 0-terminating string */
HttpHdrSc *
-httpHdrScParseCreate(const string * str)
+httpHdrScParseCreate(const String * str)
{
HttpHdrSc *sc = httpHdrScCreate();
/* parses a 0-terminating string and inits sc */
static int
-httpHdrScParseInit(HttpHdrSc * sc, const string * str)
+httpHdrScParseInit(HttpHdrSc * sc, const String * str)
{
const char *item;
const char *p; /* '=' parameter */
ilen = p++ - item;
/* find type */
- type = httpHeaderIdByName(item, ilen, ScFieldsInfo, SC_ENUM_END);
+ type = httpHeaderIdByName(item, ilen,
+ ScFieldsInfo, SC_ENUM_END);
if (type < 0) {
- debugs(90, 2, "hdr sc: unknown control-directive: near '" << item << "' in '" << *str << "'");
+ debugs(90, 2, "hdr sc: unknown control-directive: near '" << item << "' in '" << str->buf() << "'");
type = SC_OTHER;
}
if (EBIT_TEST(sct->mask, type)) {
if (type != SC_OTHER)
- debugs(90, 2, "hdr sc: ignoring duplicate control-directive: near '" << item << "' in '" << *str << "'");
+ debugs(90, 2, "hdr sc: ignoring duplicate control-directive: near '" << item << "' in '" << str->buf() << "'");
ScFieldsInfo[type].stat.repCount++;
if (!p || !httpHeaderParseQuotedString(p, &sct->content)) {
debugs(90, 2, "sc: invalid content= quoted string near '" << item << "'");
- sct->content.clear();
+ sct->content.clean();
EBIT_CLR(sct->mask, type);
}
if (EBIT_TEST(sc->mask, flag) && flag != SC_OTHER) {
/* print option name */
- packerPrintf(p, (pcount ? ", %s" : "%s"), ScFieldsInfo[flag].name.c_str());
+ packerPrintf(p, (pcount ? ", %s" : "%s"), ScFieldsInfo[flag].name.buf());
/* handle options with values */
packerPrintf(p, "=%d", (int) sc->max_age);
if (flag == SC_CONTENT)
- packerPrintf(p, "=\"%s\"", sc->content.c_str());
+ packerPrintf(p, "=\"%s\"", sc->content.buf());
pcount++;
}
}
if (sc->target.size())
- packerPrintf (p, ";%s", sc->target.c_str());
+ packerPrintf (p, ";%s", sc->target.buf());
}
void
extern const HttpHeaderStat *dump_stat; /* argh! */
const int id = (int) val;
const int valid_id = id >= 0 && id < SC_ENUM_END;
- const char *name = valid_id ? ScFieldsInfo[id].name.c_str() : "INVALID";
+ const char *name = valid_id ? ScFieldsInfo[id].name.buf() : "INVALID";
if (count || valid_id)
storeAppendPrintf(sentry, "%2d\t %-20s\t %5d\t %6.2f\n",
extern const HttpHeaderStat *dump_stat; /* argh! */
const int id = (int) val;
const int valid_id = id >= 0 && id < SC_ENUM_END;
- const char *name = valid_id ? ScFieldsInfo[id].name.c_str() : "INVALID";
+ const char *name = valid_id ? ScFieldsInfo[id].name.buf() : "INVALID";
if (count || valid_id)
storeAppendPrintf(sentry, "%2d\t %-20s\t %5d\t %6.2f\n",
while (node) {
HttpHdrScTarget *sct = (HttpHdrScTarget *)node->data;
- if (target && !sct->target.empty() && !strcmp(target, sct->target) )
+ if (target && sct->target.buf() && !strcmp (target, sct->target.buf()))
return sct;
- else if (!target && sct->target.empty())
+ else if (!target && !sct->target.buf())
return sct;
node = node->next;
/*
- * $Id: HttpHdrSc.h,v 1.2 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHdrSc.h,v 1.3 2007/05/29 13:31:37 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
extern void httpHdrScInitModule (void);
extern void httpHdrScCleanModule (void);
extern HttpHdrSc *httpHdrScCreate(void);
-extern HttpHdrSc *httpHdrScParseCreate(string const *);
+extern HttpHdrSc *httpHdrScParseCreate(String const *);
extern void httpHdrScDestroy(HttpHdrSc * sc);
extern HttpHdrSc *httpHdrScDup(const HttpHdrSc * sc);
extern void httpHdrScPackInto(const HttpHdrSc * sc, Packer * p);
/*
- * $Id: HttpHdrScTarget.cc,v 1.3 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHdrScTarget.cc,v 1.4 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 90 HTTP Cache Control Header
* AUTHOR: Alex Rousskov
httpHdrScTargetDestroy(HttpHdrScTarget * sc)
{
assert(sc);
+ sc->target.clean();
+ sc->content.clean();
delete sc;
}
{
HttpHdrScTarget *dup;
assert(sc);
- dup = httpHdrScTargetCreate(sc->target.c_str());
+ dup = httpHdrScTargetCreate(sc->target.buf());
dup->mask = sc->mask;
dup->max_age = sc->max_age;
dup->content = sc->content;
/*
- * $Id: HttpHdrScTarget.h,v 1.2 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHdrScTarget.h,v 1.3 2007/05/29 13:31:37 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
int mask;
int max_age;
int max_stale;
- string content;
- string target;
+ String content;
+ String target;
};
MEMPROXY_CLASS_INLINE(HttpHdrScTarget);
/*
- * $Id: HttpHeader.cc,v 1.132 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHeader.cc,v 1.133 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 55 HTTP Header
* AUTHOR: Alex Rousskov
#define assert_eid(id) assert((id) >= 0 && (id) < HDR_ENUM_END)
-static void httpHeaderNoteParsedEntry(http_hdr_type id, string const &value, int error);
+static void httpHeaderNoteParsedEntry(http_hdr_type id, String const &value, int error);
static void httpHeaderStatInit(HttpHeaderStat * hs, const char *label);
static void httpHeaderStatDump(const HttpHeaderStat * hs, StoreEntry * e);
if (e->id != HDR_OTHER)
delById(e->id);
else
- delByName(e->name.c_str());
+ delByName(e->name.buf());
addEntry(e->clone());
}
}
if (e->id == HDR_CONTENT_LENGTH && (e2 = findEntry(e->id)) != NULL) {
- if (e->value.compare(e2->value) != 0) {
+ if (e->value.cmp(e2->value.buf()) != 0) {
ssize_t l1, l2;
debugs(55, Config.onoff.relaxed_header_parser <= 0 ? 1 : 2,
"WARNING: found two conflicting content-length headers in {" << getStringPrefix(header_start, header_end) << "}");
goto reset;
}
- if (!httpHeaderParseSize(e->value.c_str(), &l1)) {
- debugs(55, 1, "WARNING: Unparseable content-length '" << e->value << "'");
+ if (!httpHeaderParseSize(e->value.buf(), &l1)) {
+ debugs(55, 1, "WARNING: Unparseable content-length '" << e->value.buf() << "'");
delete e;
continue;
- } else if (!httpHeaderParseSize(e2->value.c_str(), &l2)) {
- debugs(55, 1, "WARNING: Unparseable content-length '" << e2->value << "'");
+ } else if (!httpHeaderParseSize(e2->value.buf(), &l2)) {
+ debugs(55, 1, "WARNING: Unparseable content-length '" << e2->value.buf() << "'");
delById(e2->id);
} else if (l1 > l2) {
delById(e2->id);
}
}
- if (e->id == HDR_OTHER && strpbrk(e->name, w_space) != NULL) {
+ if (e->id == HDR_OTHER && stringHasWhitespace(e->name.buf())) {
debugs(55, Config.onoff.relaxed_header_parser <= 0 ? 1 : 2,
"WARNING: found whitespace in HTTP header name {" <<
getStringPrefix(field_start, field_end) << "}");
debugs(55, 9, "deleting '" << name << "' fields in hdr " << this);
while ((e = getEntry(&pos))) {
- if (!strcasecmp(e->name,name))
+ if (!e->name.caseCmp(name))
delAt(pos, count);
else
CBIT_SET(mask, e->id);
}
bool
-HttpHeader::getList(http_hdr_type id, string *s) const
+HttpHeader::getList(http_hdr_type id, String *s) const
{
HttpHeaderEntry *e;
HttpHeaderPos pos = HttpHeaderInitPos;
while ((e = getEntry(&pos))) {
if (e->id == id)
- strListAdd(s, e->value.c_str(), ',');
+ strListAdd(s, e->value.buf(), ',');
}
/*
}
/* return a list of entries with the same id separated by ',' and ws */
-string
+String
HttpHeader::getList(http_hdr_type id) const
{
HttpHeaderEntry *e;
assert(CBIT_TEST(ListHeadersMask, id));
if (!CBIT_TEST(mask, id))
- return "";
+ return String();
- string s;
+ String s;
while ((e = getEntry(&pos))) {
if (e->id == id)
- strListAdd(&s, e->value.c_str(), ',');
+ strListAdd(&s, e->value.buf(), ',');
}
/*
}
/* return a string or list of entries with the same id separated by ',' and ws */
-string
+String
HttpHeader::getStrOrList(http_hdr_type id) const
{
HttpHeaderEntry *e;
if ((e = findEntry(id)))
return e->value;
- return "";
+ return String();
}
/*
* Returns the value of the specified header.
*/
-string
+String
HttpHeader::getByName(const char *name) const
{
http_hdr_type id;
if (id != -1)
return getStrOrList(id);
- string result;
+ String result;
/* Sorry, an unknown header name. Do linear search */
while ((e = getEntry(&pos))) {
- if (e->id == HDR_OTHER && strcasecmp(e->name,name) == 0) {
- strListAdd(&result, e->value.c_str(), ',');
+ if (e->id == HDR_OTHER && e->name.caseCmp(name) == 0) {
+ strListAdd(&result, e->value.buf(), ',');
}
}
/*
* Returns a the value of the specified list member, if any.
*/
-string
+String
HttpHeader::getByNameListMember(const char *name, const char *member, const char separator) const
{
- string header;
+ String header;
const char *pos = NULL;
const char *item;
int ilen;
header = getByName(name);
- string result;
+ String result;
while (strListGetItem(&header, separator, &item, &ilen, &pos)) {
if (strncmp(item, member, mlen) == 0 && item[mlen] == '=') {
/*
* returns a the value of the specified list member, if any.
*/
-string
+String
HttpHeader::getListMember(http_hdr_type id, const char *member, const char separator) const
{
- string header;
+ String header;
const char *pos = NULL;
const char *item;
int ilen;
assert(id >= 0);
header = getStrOrList(id);
- string result;
+ String result;
while (strListGetItem(&header, separator, &item, &ilen, &pos)) {
if (strncmp(item, member, mlen) == 0 && item[mlen] == '=') {
}
}
+ header.clean();
return result;
}
assert(Headers[id].type == ftDate_1123); /* must be of an appropriate type */
if ((e = findEntry(id))) {
- value = parse_rfc1123(e->value.c_str());
+ value = parse_rfc1123(e->value.buf());
httpHeaderNoteParsedEntry(e->id, e->value, value < 0);
}
if ((e = findEntry(id))) {
httpHeaderNoteParsedEntry(e->id, e->value, 0); /* no errors are possible */
- return e->value.c_str();
+ return e->value.buf();
}
return NULL;
if ((e = findLastEntry(id))) {
httpHeaderNoteParsedEntry(e->id, e->value, 0); /* no errors are possible */
- return e->value.c_str();
+ return e->value.buf();
}
return NULL;
HttpHeader::getCc() const
{
HttpHdrCc *cc;
- string s;
+ String s;
if (!CBIT_TEST(mask, HDR_CACHE_CONTROL))
return NULL;
if (!CBIT_TEST(mask, HDR_SURROGATE_CONTROL))
return NULL;
- string s;
+ String s;
(void) getList(HDR_SURROGATE_CONTROL, &s);
HttpHeaderEntry *e;
if ((e = findEntry(HDR_CONTENT_RANGE))) {
- cr = httpHdrContRangeParseCreate(e->value.c_str());
+ cr = httpHdrContRangeParseCreate(e->value.buf());
httpHeaderNoteParsedEntry(e->id, e->value, !cr);
}
assert(Headers[id].type == ftETag); /* must be of an appropriate type */
if ((e = findEntry(id)))
- etagParseInit(&etag, e->value.c_str());
+ etagParseInit(&etag, e->value.buf());
return etag;
}
memset(&tot, 0, sizeof(tot));
if ((e = findEntry(id))) {
- const char *str = e->value.c_str();
+ const char *str = e->value.buf();
/* try as an ETag */
if (etagParseInit(&tot.tag, str)) {
Headers[id].stat.aliveCount++;
- debugs(55, 9, "created HttpHeaderEntry " << this << ": '" << name << " : " << value);
+ debugs(55, 9, "created HttpHeaderEntry " << this << ": '" << name.buf() << " : " << value.buf());
}
HttpHeaderEntry::~HttpHeaderEntry()
{
assert_eid(id);
- debugs(55, 9, "destroying entry " << this << ": '" << name << ": " << value << "'");
+ debugs(55, 9, "destroying entry " << this << ": '" << name.buf() << ": " << value.buf() << "'");
+ /* clean name if needed */
+
+ if (id == HDR_OTHER)
+ name.clean();
+
+ value.clean();
assert(Headers[id].stat.aliveCount);
/* is it a "known" field? */
http_hdr_type id = httpHeaderIdByName(field_start, name_len, Headers, HDR_ENUM_END);
- string name;
+ String name;
- string value;
+ String value;
if (id < 0)
id = HDR_OTHER;
if (field_end - value_start > 65534) {
/* String must be LESS THAN 64K and it adds a terminating NULL */
- debugs(55, 1, "WARNING: ignoring '" << name << "' header of " << (field_end - value_start) << " bytes");
+ debugs(55, 1, "WARNING: ignoring '" << name.buf() << "' header of " << (field_end - value_start) << " bytes");
if (id == HDR_OTHER)
- name.clear();
+ name.clean();
return NULL;
}
Headers[id].stat.aliveCount++;
- debugs(55, 9, "parsed HttpHeaderEntry: '" << name << ": " << value << "'");
+ debugs(55, 9, "parsed HttpHeaderEntry: '" << name.buf() << ": " << value.buf() << "'");
- return new HttpHeaderEntry(id, name.c_str(), value.c_str());
+ return new HttpHeaderEntry(id, name.buf(), value.buf());
}
HttpHeaderEntry *
HttpHeaderEntry::clone() const
{
- return new HttpHeaderEntry(id, name.c_str(), value.c_str());
+ return new HttpHeaderEntry(id, name.buf(), value.buf());
}
void
HttpHeaderEntry::packInto(Packer * p) const
{
assert(p);
- packerAppend(p, name.c_str(), name.size());
+ packerAppend(p, name.buf(), name.size());
packerAppend(p, ": ", 2);
- packerAppend(p, value.c_str(), value.size());
+ packerAppend(p, value.buf(), value.size());
packerAppend(p, "\r\n", 2);
}
assert_eid (id);
assert (Headers[id].type == ftInt);
int val = -1;
- int ok = httpHeaderParseInt(value.c_str(), &val);
+ int ok = httpHeaderParseInt(value.buf(), &val);
httpHeaderNoteParsedEntry(id, value, !ok);
/* XXX: Should we check ok - ie
* return ok ? -1 : value;
}
static void
-httpHeaderNoteParsedEntry(http_hdr_type id, string const &context, int error)
+httpHeaderNoteParsedEntry(http_hdr_type id, String const &context, int error)
{
Headers[id].stat.parsCount++;
if (error) {
Headers[id].stat.errCount++;
- debugs(55, 2, "cannot parse hdr field: '" << Headers[id].name << ": " << context << "'");
+ debugs(55, 2, "cannot parse hdr field: '" << Headers[id].name.buf() << ": " << context.buf() << "'");
}
}
{
const int id = (int) val;
const int valid_id = id >= 0 && id < HDR_ENUM_END;
- const char *name = valid_id ? Headers[id].name.c_str() : "INVALID";
+ const char *name = valid_id ? Headers[id].name.buf() : "INVALID";
int visible = count > 0;
/* for entries with zero count, list only those that belong to current type of message */
for (ht = (http_hdr_type)0; ht < HDR_ENUM_END; ++ht) {
HttpHeaderFieldInfo *f = Headers + ht;
storeAppendPrintf(e, "%2d\t %-20s\t %5d\t %6.3f\t %6.3f\n",
- f->id, f->name.c_str(), f->stat.aliveCount,
+ f->id, f->name.buf(), f->stat.aliveCount,
xpercent(f->stat.errCount, f->stat.parsCount),
xpercent(f->stat.repCount, f->stat.seenCount));
}
}
http_hdr_type
-httpHeaderIdByName(const char *name, unsigned int name_len, const HttpHeaderFieldInfo * info, int end)
+httpHeaderIdByName(const char *name, int name_len, const HttpHeaderFieldInfo * info, int end)
{
int i;
for (i = 0; i < end; ++i) {
- if (name_len >= 0 && name_len != (unsigned int)info[i].name.size())
+ if (name_len >= 0 && name_len != info[i].name.size())
continue;
- if (!strncasecmp(name, info[i].name,
+ if (!strncasecmp(name, info[i].name.buf(),
name_len < 0 ? info[i].name.size() + 1 : name_len))
return info[i].id;
}
assert(id >= 0 && id < HDR_ENUM_END);
- return Headers[id].name.c_str();
+ return Headers[id].name.buf();
}
int
assert(id >= 0);
- string header (getStrOrList(id));
+ String header (getStrOrList(id));
while (strListGetItem(&header, separator, &item, &ilen, &pos)) {
if (strncmp(item, member, mlen) == 0
assert(name);
- string header (getByName(name));
+ String header (getByName(name));
while (strListGetItem(&header, separator, &item, &ilen, &pos)) {
if (strncmp(item, member, mlen) == 0
{
if (has(HDR_CONNECTION)) {
/* anything that matches Connection list member will be deleted */
- string strConnection;
-
- (void) getList(HDR_CONNECTION, &strConnection);
+ String strConnection;
+
+ (void) getList(HDR_CONNECTION, &strConnection);
const HttpHeaderEntry *e;
HttpHeaderPos pos = HttpHeaderInitPos;
/*
int headers_deleted = 0;
while ((e = getEntry(&pos))) {
- if (strListIsMember(&strConnection, e->name.c_str(), ','))
+ if (strListIsMember(&strConnection, e->name.buf(), ','))
delAt(pos, headers_deleted);
}
if (headers_deleted)
/*
- * $Id: HttpHeader.h,v 1.20 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHeader.h,v 1.21 2007/05/29 13:31:37 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
int getInt() const;
MEMPROXY_CLASS(HttpHeaderEntry);
http_hdr_type id;
- string name;
- string value;
+ String name;
+ String value;
};
MEMPROXY_CLASS_INLINE(HttpHeaderEntry)
void refreshMask();
void addEntry(HttpHeaderEntry * e);
void insertEntry(HttpHeaderEntry * e);
- string getList(http_hdr_type id) const;
- bool getList(http_hdr_type id, string *s) const;
- string getStrOrList(http_hdr_type id) const;
- string getByName(const char *name) const;
- string getByNameListMember(const char *name, const char *member, const char separator) const;
- string getListMember(http_hdr_type id, const char *member, const char separator) const;
+ String getList(http_hdr_type id) const;
+ bool getList(http_hdr_type id, String *s) const;
+ String getStrOrList(http_hdr_type id) const;
+ String getByName(const char *name) const;
+ String getByNameListMember(const char *name, const char *member, const char separator) const;
+ String getListMember(http_hdr_type id, const char *member, const char separator) const;
int has(http_hdr_type id) const;
void putInt(http_hdr_type id, int number);
void putTime(http_hdr_type id, time_t htime);
extern void httpHeaderRegisterWithCacheManager(CacheManager & manager);
-extern int httpHeaderParseQuotedString (const char *start, string *val);
+extern int httpHeaderParseQuotedString (const char *start, String *val);
SQUIDCEXTERN int httpHeaderHasByNameListMember(const HttpHeader * hdr, const char *name, const char *member, const char separator);
SQUIDCEXTERN void httpHeaderUpdate(HttpHeader * old, const HttpHeader * fresh, const HttpHeaderMask * denied_mask);
int httpMsgIsPersistent(HttpVersion const &http_ver, const HttpHeader * hdr);
/*
- * $Id: HttpHeaderRange.h,v 1.9 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHeaderRange.h,v 1.10 2007/05/29 13:31:37 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
static size_t ParsedCount;
/* Http Range Header Field */
- static HttpHdrRange *ParseCreate(const string * range_spec);
+ static HttpHdrRange *ParseCreate(const String * range_spec);
HttpHdrRange();
HttpHdrRange(HttpHdrRange const &);
int canonize(size_t);
int canonize(HttpReply *rep);
/* returns true if ranges are valid; inits HttpHdrRange */
- bool parseInit(const string * range_spec);
+ bool parseInit(const String * range_spec);
void packInto(Packer * p) const;
/* other */
bool isComplex() const;
ssize_t debt() const;
void debt(ssize_t);
ssize_t debt_size; /* bytes left to send from the current spec */
- string boundary; /* boundary for multipart responses */
+ String boundary; /* boundary for multipart responses */
bool valid;
};
/*
- * $Id: HttpHeaderTools.cc,v 1.60 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpHeaderTools.cc,v 1.61 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 66 HTTP Header Tools
* AUTHOR: Alex Rousskov
int i;
for (i = 0; i < count; ++i)
- table[i].name.clear();
+ table[i].name.clean();
delete [] table;
}
int
httpHeaderHasConnDir(const HttpHeader * hdr, const char *directive)
{
- string list;
+ String list;
http_hdr_type ht;
int res;
/* what type of header do we have? */
res = strListIsMember(&list, directive, ',');
+ list.clean();
+
return res;
}
/* returns true iff "m" is a member of the list */
int
-strListIsMember(const string * list, const char *m, char del)
+strListIsMember(const String * list, const char *m, char del)
{
const char *pos = NULL;
const char *item;
/* returns true iff "s" is a substring of a member of the list */
int
-strListIsSubstr(const string * list, const char *s, char del)
+strListIsSubstr(const String * list, const char *s, char del)
{
assert(list && del);
return list->pos(s) != 0;
/* appends an item to the list */
void
-strListAdd(string * str, const char *item, char del)
+strListAdd(String * str, const char *item, char del)
{
assert(str && item);
* init pos with NULL to start iteration.
*/
int
-strListGetItem(const string * str, char del, const char **item, int *ilen, const char **pos)
+strListGetItem(const String * str, char del, const char **item, int *ilen, const char **pos)
{
size_t len;
static char delim[2][3] = {
else
(*pos)++;
} else {
- *pos = str->c_str();
+ *pos = str->buf();
if (!*pos)
return 0;
* RC TODO: This is too looose. We should honour the BNF and exclude CTL's
*/
int
-httpHeaderParseQuotedString (const char *start, string *val)
+httpHeaderParseQuotedString (const char *start, String *val)
{
const char *end, *pos;
- val->clear();
+ val->clean();
assert (*start == '"');
pos = start + 1;
/*
- * $Id: HttpReply.cc,v 1.94 2007/05/22 16:37:26 rousskov Exp $
+ * $Id: HttpReply.cc,v 1.95 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 58 HTTP Reply (Response)
* AUTHOR: Alex Rousskov
// virtual function instead, but it is not clear whether virtual methods
// are allowed with MEMPROXY_CLASS() and whether some cbdata void*
// conversions are not going to kill virtual tables
- const string pfx = protoPrefix;
+ const String pfx = protoPrefix;
clean();
init();
protoPrefix = pfx;
hdr->putStr(HDR_CONTENT_TYPE, ctype);
content_type = ctype;
} else
- content_type.clear();
+ content_type = String();
if (clen >= 0)
hdr->putInt(HDR_CONTENT_LENGTH, clen);
int
HttpReply::validatorsMatch(HttpReply const * otherRep) const
{
- string one,two;
+ String one,two;
assert (otherRep);
/* Numbers first - easiest to check */
/* Content-Length */
two = otherRep->header.getStrOrList(HDR_ETAG);
- if (one.empty() || two.empty() || strcasecmp (one, two)) {
+ if (!one.buf() || !two.buf() || strcasecmp (one.buf(), two.buf())) {
+ one.clean();
+ two.clean();
return 0;
}
two = otherRep->header.getStrOrList(HDR_CONTENT_MD5);
- if (one.empty() || two.empty() || strcasecmp (one, two)) {
- one.clear();
- two.clear();
+ if (!one.buf() || !two.buf() || strcasecmp (one.buf(), two.buf())) {
+ one.clean();
+ two.clean();
return 0;
}
if (str)
content_type.limitInit(str, strcspn(str, ";\t "));
else
- content_type = "";
+ content_type = String();
/* be sure to set expires after date and cache-control */
expires = hdrExpirationTime();
void
HttpReply::hdrCacheClean()
{
- content_type.clear();
+ content_type.clean();
if (cache_control) {
httpHdrCcDestroy(cache_control);
bool HttpReply::sanityCheckStartLine(MemBuf *buf, http_status *error)
{
- if (buf->contentSize() >= protoPrefix.size() && protoPrefix.compare(buf->content(), protoPrefix.size()) != 0) {
- debugs(58, 3, "HttpReply::sanityCheckStartLine: missing protocol prefix (" << protoPrefix << ") in '" << buf->content() << "'");
+ if (buf->contentSize() >= protoPrefix.size() && protoPrefix.cmp(buf->content(), protoPrefix.size()) != 0) {
+ debugs(58, 3, "HttpReply::sanityCheckStartLine: missing protocol prefix (" << protoPrefix.buf() << ") in '" << buf->content() << "'");
*error = HTTP_INVALID_HEADER;
return false;
}
/*
- * $Id: HttpReply.h,v 1.19 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpReply.h,v 1.20 2007/05/29 13:31:37 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
time_t expires;
- string content_type;
+ String content_type;
HttpHdrSc *surrogate_control;
HttpBody body; /* for small constant memory-resident text bodies only */
- string protoPrefix; // e.g., "HTTP/"
+ String protoPrefix; // e.g., "HTTP/"
bool do_clean;
/*
- * $Id: HttpRequest.cc,v 1.75 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpRequest.cc,v 1.76 2007/05/29 13:31:37 amosjeffries Exp $
*
* DEBUG: section 73 HTTP Request
* AUTHOR: Duane Wessels
{
method = METHOD_NONE;
protocol = PROTO_NONE;
- urlpath.clear();
+ urlpath = NULL;
login[0] = '\0';
host[0] = '\0';
auth_user_request = NULL;
safe_free(vary_headers);
- urlpath.clear();
+ urlpath.clean();
header.clean();
range = NULL;
}
- tag.clear();
+ tag.clean();
- extacl_user.clear();
+ extacl_user.clean();
- extacl_passwd.clear();
+ extacl_passwd.clean();
- extacl_log.clear();
+ extacl_log.clean();
}
void
assert(p);
/* pack request-line */
packerPrintf(p, "%s %s HTTP/1.0\r\n",
- RequestMethodStr[method], urlpath.c_str());
+ RequestMethodStr[method], urlpath.buf());
/* headers */
header.packInto(p);
/* trailer */
* check anonymizer (aka header_access) configuration.
*/
int
-httpRequestHdrAllowed(const HttpHeaderEntry * e, string * strConn)
+httpRequestHdrAllowed(const HttpHeaderEntry * e, String * strConn)
{
assert(e);
/* check connection header */
- if (strConn && strListIsMember(strConn, e->name.c_str(), ','))
+ if (strConn && strListIsMember(strConn, e->name.buf(), ','))
return 0;
return 1;
return urlCanonical((HttpRequest*)this);
if (urlpath.size())
- return urlpath.c_str();
+ return urlpath.buf();
return "/";
}
/*
- * $Id: HttpRequest.h,v 1.28 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpRequest.h,v 1.29 2007/05/29 13:31:38 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
#include "HttpRequestMethod.h"
/* Http Request */
-extern int httpRequestHdrAllowed(const HttpHeaderEntry * e, string * strConnection);
+extern int httpRequestHdrAllowed(const HttpHeaderEntry * e, String * strConnection);
extern int httpRequestHdrAllowedByName(http_hdr_type id);
extern void httpRequestPack(void *obj, Packer *p);
u_short port;
- string urlpath;
+ String urlpath;
char *canonical;
char *peer_domain; /* Configured peer forceddomain */
- string tag; /* Internal tag for this request */
+ String tag; /* Internal tag for this request */
- string extacl_user; /* User name returned by extacl lookup */
+ String extacl_user; /* User name returned by extacl lookup */
- string extacl_passwd; /* Password returned by extacl lookup */
+ String extacl_passwd; /* Password returned by extacl lookup */
- string extacl_log; /* String to be used for access.log purposes */
+ String extacl_log; /* String to be used for access.log purposes */
public:
bool multipartRangeRequest() const;
/*
- * $Id: HttpStatusLine.cc,v 1.33 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpStatusLine.cc,v 1.34 2007/05/29 13:31:38 amosjeffries Exp $
*
* DEBUG: section 57 HTTP Status-line
* AUTHOR: Alex Rousskov
* so NULL-termination assumed.
*/
int
-httpStatusLineParse(HttpStatusLine * sline, const string &protoPrefix, const char *start, const char *end)
+httpStatusLineParse(HttpStatusLine * sline, const String &protoPrefix, const char *start, const char *end)
{
assert(sline);
sline->status = HTTP_INVALID_HEADER; /* Squid header parsing error */
// XXX: HttpMsg::parse() has a similar check but is using
// casesensitive comparison (which is required by HTTP errata?)
- if (strncasecmp(protoPrefix,start, protoPrefix.size()) != 0)
+ if (protoPrefix.caseCmp(start, protoPrefix.size()) != 0)
return 0;
start += protoPrefix.size();
/*
- * $Id: HttpStatusLine.h,v 1.3 2007/05/18 06:41:22 amosjeffries Exp $
+ * $Id: HttpStatusLine.h,v 1.4 2007/05/29 13:31:38 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
SQUIDCEXTERN const char *httpStatusLineReason(const HttpStatusLine * sline);
/* parse/pack */
/* parse a 0-terminating buffer and fill internal structires; returns true on success */
-SQUIDCEXTERN int httpStatusLineParse(HttpStatusLine * sline, const string &protoPrefix,
+SQUIDCEXTERN int httpStatusLineParse(HttpStatusLine * sline, const String &protoPrefix,
const char *start, const char *end);
/* pack fields using Packer */
SQUIDCEXTERN void httpStatusLinePackInto(const HttpStatusLine * sline, Packer * p);
/*
- * $Id: ICAPConfig.cc,v 1.15 2007/05/18 06:41:29 amosjeffries Exp $
+ * $Id: ICAPConfig.cc,v 1.16 2007/05/29 13:31:43 amosjeffries Exp $
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
* ----------------------------------------------------------
ICAPConfig TheICAPConfig;
ICAPServiceRep::Pointer
-ICAPConfig::findService(const string& key)
+ICAPConfig::findService(const String& key)
{
Vector<ICAPServiceRep::Pointer>::iterator iter = services.begin();
}
ICAPClass *
-ICAPConfig::findClass(const string& key)
+ICAPConfig::findClass(const String& key)
{
if (!key.size())
return NULL;
wordlist *service_names = NULL;
wordlist *iter;
- ConfigParser::ParseString(key);
+ ConfigParser::ParseString(&key);
ConfigParser::ParseWordList(&service_names);
for (iter = service_names; iter; iter = iter->next) {
candidateClasses.clean();
- matchedClass.clear();
+ matchedClass.clean();
acl_checklist = NULL;
ICAPClass *c = *ci;
ICAPServiceRep::Pointer service = findBestService(c, false);
if (service != NULL) {
- debugs(93, 3, "ICAPAccessCheck::check: class '" << c->key << "' has candidate service '" << service->key << "'");
+ debugs(93, 3, "ICAPAccessCheck::check: class '" << c->key.buf() << "' has candidate service '" << service->key.buf() << "'");
candidateClasses += c->key;
}
}
*/
debugs(93, 3, "ICAPAccessCheck::check: NO candidates or matches found");
- matchedClass.clear();
+ matchedClass.clean();
ICAPAccessCheckCallbackWrapper(1, this);
ICAPAccessCheck *ac = (ICAPAccessCheck*)data;
if (ac->matchedClass.size()) {
- debugs(93, 5, "ICAPAccessCheckCallbackWrapper matchedClass = " << ac->matchedClass);
+ debugs(93, 5, "ICAPAccessCheckCallbackWrapper matchedClass = " << ac->matchedClass.buf());
}
if (!answer) {
debugs(93, 3, "ICAPAccessCheck::do_callback");
if (matchedClass.size()) {
- debugs(93, 3, "ICAPAccessCheck::do_callback matchedClass = " << matchedClass);
+ debugs(93, 3, "ICAPAccessCheck::do_callback matchedClass = " << matchedClass.buf());
}
void *validated_cbdata;
for (VI i = services.begin(); i != services.end(); ++i) {
const ICAPServiceRep::Pointer &r = *i;
- storeAppendPrintf(entry, "%s %s_%s %s %d %s\n", name, r->key.c_str(),
- r->methodStr(), r->vectPointStr(), r->bypass, r->uri.c_str());
+ storeAppendPrintf(entry, "%s %s_%s %s %d %s\n", name, r->key.buf(),
+ r->methodStr(), r->vectPointStr(), r->bypass, r->uri.buf());
}
};
Vector<ICAPClass*>::iterator i = classes.begin();
while (i != classes.end()) {
- storeAppendPrintf(entry, "%s %s\n", name, (*i)->key.c_str());
+ storeAppendPrintf(entry, "%s %s\n", name, (*i)->key.buf());
++i;
}
};
void
ICAPConfig::parseICAPAccess(ConfigParser &parser)
{
- string aKey;
- ConfigParser::ParseString(aKey);
+ String aKey;
+ ConfigParser::ParseString(&aKey);
ICAPClass *theClass = TheICAPConfig.findClass(aKey);
if (theClass == NULL)
fatalf("Did not find ICAP class '%s' referenced on line %d\n",
- aKey.c_str(), config_lineno);
+ aKey.buf(), config_lineno);
aclParseAccessLine(parser, &theClass->accessList);
};
Vector<ICAPClass*>::iterator i = classes.begin();
while (i != classes.end()) {
- snprintf(nom, 64, "%s %s", name, (*i)->key.c_str());
+ snprintf(nom, 64, "%s %s", name, (*i)->key.buf());
dump_acl_access(entry, nom, (*i)->accessList);
++i;
}
/*
- * $Id: ICAPConfig.h,v 1.12 2007/05/18 06:41:29 amosjeffries Exp $
+ * $Id: ICAPConfig.h,v 1.13 2007/05/29 13:31:44 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
{
public:
- string key;
+ String key;
acl_access *accessList;
Vector<ICAPServiceRep::Pointer> services;
ICAPAccessCheckCallback *callback;
void *callback_data;
ACLChecklist *acl_checklist;
- Vector<string> candidateClasses;
- string matchedClass;
+ Vector<String> candidateClasses;
+ String matchedClass;
void do_callback();
ICAPServiceRep::Pointer findBestService(ICAPClass *c, bool preferUp);
void parseICAPService(void);
void freeICAPService(void);
void dumpICAPService(StoreEntry *, const char *);
- ICAPServiceRep::Pointer findService(const string&);
- ICAPClass * findClass(const string& key);
+ ICAPServiceRep::Pointer findService(const String&);
+ ICAPClass * findClass(const String& key);
void parseICAPClass(void);
void freeICAPClass(void);
* XXX These should use HttpHdr interfaces instead of Printfs
*/
const ICAPServiceRep &s = service();
- buf.Printf("%s %s ICAP/1.0\r\n", s.methodStr(), s.uri.c_str());
- buf.Printf("Host: %s:%d\r\n", s.host.c_str(), s.port);
+ buf.Printf("%s %s ICAP/1.0\r\n", s.methodStr(), s.uri.buf());
+ buf.Printf("Host: %s:%d\r\n", s.host.buf(), s.port);
buf.Printf("Date: %s\r\n", mkrfc1123(squid_curtime));
if (!TheICAPConfig.reuse_connections)
// to simplify, we could assume that request is always available
- string urlPath;
+ String urlPath;
if (request) {
urlPath = request->urlpath;
if (ICAP::methodRespmod == m)
const HttpRequest *request = virgin.cause ?
virgin.cause :
dynamic_cast<const HttpRequest*>(virgin.header);
- const string urlPath = request ? request->urlpath : "";
+ const String urlPath = request ? request->urlpath : String();
size_t wantedSize;
if (!service().wantsPreview(urlPath, wantedSize)) {
debugs(93, 5, "ICAPModXact should not offer preview for " << urlPath);
void ICAPOptXact::makeRequest(MemBuf &buf)
{
const ICAPServiceRep &s = service();
- buf.Printf("OPTIONS %s ICAP/1.0\r\n", s.uri.c_str());
- buf.Printf("Host: %s:%d\r\n", s.host.c_str(), s.port);
+ buf.Printf("OPTIONS %s ICAP/1.0\r\n", s.uri.buf());
+ buf.Printf("Host: %s:%d\r\n", s.host.buf(), s.port);
buf.append(ICAP::crlf, 2);
}
// future optimization note: this method is called by ICAP ACL code at least
// twice for each HTTP message to see if the message should be ignored. For any
// non-ignored HTTP message, ICAP calls to check whether a preview is needed.
-ICAPOptions::TransferKind ICAPOptions::transferKind(const string &urlPath) const
+ICAPOptions::TransferKind ICAPOptions::transferKind(const String &urlPath) const
{
if (theTransfers.preview.matches(urlPath))
return xferPreview;
// TODO: HttpHeader should provide a general method for this type of conversion
void ICAPOptions::cfgIntHeader(const HttpHeader *h, const char *fname, int &value)
{
- const string s = h->getByName(fname);
+ const String s = h->getByName(fname);
- if (!s.empty() && xisdigit(s[0]))
- value = atoi(s.c_str());
+ if (s.size() && xisdigit(*s.buf()))
+ value = atoi(s.buf());
else
value = -1;
void ICAPOptions::cfgTransferList(const HttpHeader *h, TransferList &list)
{
- const string buf = h->getByName(list.name);
+ const String buf = h->getByName(list.name);
bool foundStar = false;
list.parse(buf, foundStar);
wordlistAdd(&extensions, extension);
};
-bool ICAPOptions::TransferList::matches(const string &urlPath) const {
+bool ICAPOptions::TransferList::matches(const String &urlPath) const {
const int urlLen = urlPath.size();
for (wordlist *e = extensions; e; e = e->next) {
// optimize: store extension lengths
if (eLen < urlLen) {
const int eOff = urlLen - eLen;
// RFC 3507 examples imply that extensions come without leading '.'
- if (urlPath[eOff-1] == '.' &&
- strcmp(&urlPath[eOff], e->key) == 0) {
+ if (urlPath.buf()[eOff-1] == '.' &&
+ strcmp(urlPath.buf() + eOff, e->key) == 0) {
debugs(93,7, "ICAPOptions url " << urlPath << " matches " <<
name << " extension " << e->key);
return true;
return false;
}
-void ICAPOptions::TransferList::parse(const string &buf, bool &foundStar) {
+void ICAPOptions::TransferList::parse(const String &buf, bool &foundStar) {
foundStar = false;
const char *item;
/*
- * $Id: ICAPOptions.h,v 1.10 2007/05/18 06:41:29 amosjeffries Exp $
+ * $Id: ICAPOptions.h,v 1.11 2007/05/29 13:31:44 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
time_t timestamp() const { return theTimestamp; };
typedef enum { xferNone, xferPreview, xferIgnore, xferComplete } TransferKind;
- TransferKind transferKind(const string &urlPath) const;
+ TransferKind transferKind(const String &urlPath) const;
public:
const char *error; // human-readable information; set iff !valid()
// ICAP server MUST supply this info
Vector<ICAP::Method> methods;
- string istag;
+ String istag;
// ICAP server MAY supply this info. If not, Squid supplies defaults.
- string service;
- string serviceId;
+ String service;
+ String serviceId;
int max_connections;
bool allow204;
int preview;
TransferList();
~TransferList();
- bool matches(const string &urlPath) const;
+ bool matches(const String &urlPath) const;
- void parse(const string &buf, bool &foundStar);
+ void parse(const String &buf, bool &foundStar);
void add(const char *extension);
void report(int level, const char *prefix) const;
char *service_type = NULL;
- ConfigParser::ParseString(key);
+ ConfigParser::ParseString(&key);
ConfigParser::ParseString(&service_type);
ConfigParser::ParseBool(&bypass);
- ConfigParser::ParseString(uri);
+ ConfigParser::ParseString(&uri);
- debugs(3, 5, "ICAPService::parseConfigLine (line " << config_lineno << "): " << key << " " << service_type << " " << bypass);
+ debugs(3, 5, "ICAPService::parseConfigLine (line " << config_lineno << "): " << key.buf() << " " << service_type << " " << bypass);
method = parseMethod(service_type);
point = parseVectPoint(service_type);
debugs(3, 5, "ICAPService::parseConfigLine (line " << config_lineno << "): service is " << methodStr() << "_" << vectPointStr());
- if (strncmp(uri, "icap://", 7) != 0) {
- debugs(3, 0, "ICAPService::parseConfigLine (line " << config_lineno << "): wrong uri: " << uri);
+ if (uri.cmp("icap://", 7) != 0) {
+ debugs(3, 0, "ICAPService::parseConfigLine (line " << config_lineno << "): wrong uri: " << uri.buf());
return false;
}
- const char *s = &uri[7];
+ const char *s = uri.buf() + 7;
const char *e;
return probed() && !up();
}
-bool ICAPServiceRep::wantsUrl(const string &urlPath) const
+bool ICAPServiceRep::wantsUrl(const String &urlPath) const
{
Must(hasOptions());
return theOptions->transferKind(urlPath) != ICAPOptions::xferIgnore;
}
-bool ICAPServiceRep::wantsPreview(const string &urlPath, size_t &wantedSize) const
+bool ICAPServiceRep::wantsPreview(const String &urlPath, size_t &wantedSize) const
{
Must(hasOptions());
if (!theOptions->methods.empty()) {
bool method_found = false;
- string method_list;
+ String method_list;
Vector <ICAP::Method>::iterator iter = theOptions->methods.begin();
while (iter != theOptions->methods.end()) {
if (!method_found) {
debugs(93,1, "WARNING: Squid is configured to use ICAP method " <<
ICAP::methodStr(method) <<
- " for service " << uri <<
- " but OPTIONS response declares the methods are " << method_list);
+ " for service " << uri.buf() <<
+ " but OPTIONS response declares the methods are " << method_list.buf());
}
}
// TODO: If skew is negative, the option will be considered down
// because of stale options. We should probably change this.
debugs(93, 1, "ICAP service's clock is skewed by " << skew <<
- " seconds: " << uri);
+ " seconds: " << uri.buf());
}
}
/*
- * $Id: ICAPServiceRep.h,v 1.8 2007/05/18 06:41:30 amosjeffries Exp $
+ * $Id: ICAPServiceRep.h,v 1.9 2007/05/29 13:31:44 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
void callWhenReady(Callback *cb, void *data);
// the methods below can only be called on an up() service
- bool wantsUrl(const string &urlPath) const;
- bool wantsPreview(const string &urlPath, size_t &wantedSize) const;
+ bool wantsUrl(const String &urlPath) const;
+ bool wantsPreview(const String &urlPath, size_t &wantedSize) const;
bool allows204() const;
void noteFailure(); // called by transactions to report service failure
public:
- string key;
+ String key;
ICAP::Method method;
ICAP::VectPoint point;
- string uri; // service URI
+ String uri; // service URI
// URI components
- string host;
+ String host;
int port;
- string resource;
+ String resource;
// XXX: use it when selecting a service and handling ICAP errors!
bool bypass;
disableRetries(); // this will also safely drain pconn pool
// TODO: check whether NULL domain is appropriate here
- connection = icapPconnPool->pop(s.host.c_str(), s.port, NULL, NULL, isRetriable);
+ connection = icapPconnPool->pop(s.host.buf(), s.port, NULL, NULL, isRetriable);
if (connection >= 0) {
debugs(93,3, HERE << "reused pconn FD " << connection);
connector = &ICAPXaction_noteCommConnected; // make doneAll() false
disableRetries(); // we only retry pconn failures
connection = comm_open(SOCK_STREAM, 0, getOutgoingAddr(NULL), 0,
- COMM_NONBLOCKING, s.uri.c_str());
+ COMM_NONBLOCKING, s.uri.buf());
if (connection < 0)
dieOnConnectionFailure(); // throws
- debugs(93,3, typeName << " opens connection to " << s.host << ":" << s.port);
+ debugs(93,3, typeName << " opens connection to " << s.host.buf() << ":" << s.port);
commSetTimeout(connection, Config.Timeout.connect,
&ICAPXaction_noteCommTimedout, this);
comm_add_close_handler(connection, closer, this);
connector = &ICAPXaction_noteCommConnected;
- commConnectStart(connection, s.host.c_str(), s.port, connector, this);
+ commConnectStart(connection, s.host.buf(), s.port, connector, this);
}
/*
if (reuseConnection) {
debugs(93,3, HERE << "pushing pconn" << status());
commSetTimeout(connection, -1, NULL, NULL);
- icapPconnPool->push(connection, theService->host.c_str(), theService->port, NULL, NULL);
+ icapPconnPool->push(connection, theService->host.buf(), theService->port, NULL, NULL);
disableRetries();
} else {
debugs(93,3, HERE << "closing pconn" << status());
void ICAPXaction::handleCommTimedout()
{
- debugs(93, 0, HERE << "ICAP FD " << connection << " timeout to " << theService->methodStr() << " " << theService->uri);
+ debugs(93, 0, HERE << "ICAP FD " << connection << " timeout to " << theService->methodStr() << " " << theService->uri.buf());
reuseConnection = false;
MemBuf mb;
mb.init();
#
# Makefile for the Squid Object Cache server
#
-# $Id: Makefile.am,v 1.181 2007/05/18 06:41:22 amosjeffries Exp $
+# $Id: Makefile.am,v 1.182 2007/05/29 13:31:38 amosjeffries Exp $
#
# Uncomment and customize the following to suit your needs:
#
$(SSL_SOURCE) \
stat.cc \
StatHist.cc \
- SqString.cc \
+ String.cc \
stmem.cc \
stmem.h \
store.cc \
MemBuf.h \
Store.cci \
StoreEntryStream.h \
- SqString.cci \
- SqString.h \
+ String.cci \
SquidString.h \
SquidTime.h
StoreMetaUnpacker.cc \
StoreMetaURL.cc \
StoreMetaVary.cc \
- SqString.cc \
+ String.cc \
time.cc \
ufsdump.cc \
url.cc \
globals.cc
check_PROGRAMS+= \
- tests/testString \
tests/testAuth \
tests/testACLMaxUserIP \
tests/testBoilerplate \
tests/test_http_range \
tests/testHttpRequest \
tests/testStore \
+ tests/testString \
tests/testURL \
@STORE_TESTS@
authenticate.cc \
ConfigParser.cc \
tests/stub_acl.cc tests/stub_cache_cf.cc \
- tests/stub_helper.cc cbdata.cc SqString.cc \
+ tests/stub_helper.cc cbdata.cc String.cc \
tests/stub_store.cc HttpHeaderTools.cc HttpHeader.cc acl.cc event.cc mem.cc \
MemBuf.cc HttpHdrContRange.cc Packer.cc ACLChecklist.cc HttpHdrCc.cc HttpHdrSc.cc \
HttpHdrScTarget.cc url.cc ACLProxyAuth.cc ACLRegexData.cc ACLUserData.cc \
URLScheme.cc \
wordlist.cc
## acl.cc cache_cf.cc tools.cc \
-## helper.cc SqString.cc cbdata.cc HttpHeaderTools.cc store.cc cache_manager.cc \
+## helper.cc String.cc cbdata.cc HttpHeaderTools.cc store.cc cache_manager.cc \
## HttpHeader.cc url.cc mem.cc HttpRequest.cc Packer.cc access_log.cc \
## MemBuf.cc StatHist.cc logfile.cc
## HttpHdrScTarget.cc \
## Packer.cc \
## StatHist.cc \
-## SqString.cc \
+## String.cc \
tests_testACLMaxUserIP_SOURCES= \
acl.cc \
ACLChecklist.cc \
Parsing.cc \
StatHist.cc \
stmem.cc \
- SqString.cc \
+ String.cc \
tests/stub_cache_cf.cc \
tests/stub_comm.cc \
tests/stub_DelayId.cc \
HttpRequest.cc \
HttpRequestMethod.cc \
mem.cc \
- SqString.cc \
+ String.cc \
tests/testCacheManager.cc \
tests/testCacheManager.h \
tests/testMain.cc \
HttpRequestMethod.cc \
mem.cc \
RemovalPolicy.cc \
- SqString.cc \
+ String.cc \
tests/CapturingStoreEntry.h \
tests/testEvent.cc \
tests/testEvent.h \
HttpRequestMethod.cc \
mem.cc \
RemovalPolicy.cc \
- SqString.cc \
+ String.cc \
tests/testEventLoop.cc \
tests/testEventLoop.h \
tests/testMain.cc \
StoreMetaURL.cc \
StoreMetaVary.cc \
StoreSwapLogData.cc \
- SqString.cc \
+ String.cc \
SwapDir.cc \
time.cc \
tools.cc \
HttpRequest.cc \
HttpRequestMethod.cc \
mem.cc \
- SqString.cc \
+ String.cc \
tests/testHttpRequest.h \
tests/testHttpRequest.cc \
tests/testHttpRequestMethod.h \
SwapDir.cc \
authenticate.cc \
tests/stub_acl.cc tests/stub_cache_cf.cc \
- tests/stub_helper.cc cbdata.cc SqString.cc \
+ tests/stub_helper.cc cbdata.cc String.cc \
tests/stub_comm.cc \
tests/stub_client_side_request.cc \
tests/stub_http.cc \
tests_testStore_LDFLAGS = $(LIBADD_DL)
tests_testStore_DEPENDENCIES = $(top_builddir)/lib/libmiscutil.a \
@SQUID_CPPUNIT_LA@
+
# string needs mem.cc.
tests_testString_SOURCES= \
mem.cc \
- SqString.cc \
+ String.cc \
tests/testMain.cc \
tests/testString.cc \
tests/testString.h \
HttpRequestMethod.cc \
mem.cc \
RemovalPolicy.cc \
- SqString.cc \
+ String.cc \
tests/testURL.cc \
tests/testURL.h \
tests/testURLScheme.cc \
/*
- * $Id: MemObject.cc,v 1.27 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: MemObject.cc,v 1.28 2007/05/29 13:31:38 amosjeffries Exp $
*
* DEBUG: section 19 Store Memory Primitives
* AUTHOR: Robert Collins
debugs(20, 1, "MemObject->nclients: " << nclients);
debugs(20, 1, "MemObject->reply: " << _reply);
debugs(20, 1, "MemObject->request: " << request);
- debugs(20, 1, "MemObject->log_url: " << (log_url ? log_url : "(NULL)") );
+ debugs(20, 1, "MemObject->log_url: " << log_url << " " << checkNullString(log_url));
}
HttpReply const *
/*
- * $Id: PeerDigest.h,v 1.2 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: PeerDigest.h,v 1.3 2007/05/29 13:31:38 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
struct _peer *peer; /* pointer back to peer structure, argh */
CacheDigest *cd; /* actual digest structure */
- string host; /* copy of peer->host */
+ String host; /* copy of peer->host */
const char *req_result; /* text status of the last request */
struct
+++ /dev/null
-
-/*
- * $Id: SqString.h,v 1.3 2007/05/20 08:29:44 amosjeffries Exp $
- *
- * DEBUG: section 67 String
- * AUTHOR: Duane Wessels
- *
- * SQUID Web Proxy Cache http://www.squid-cache.org/
- * ----------------------------------------------------------
- *
- * Squid is the result of efforts by numerous individuals from
- * the Internet community; see the CONTRIBUTORS file for full
- * details. Many organizations have provided support for Squid's
- * development; see the SPONSORS file for full details. Squid is
- * Copyrighted (C) 2001 by the Regents of the University of
- * California; see the COPYRIGHT file for full details. Squid
- * incorporates software developed and/or copyrighted by other
- * sources; see the CREDITS file for full details.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111, USA.
- *
- */
-
-#ifndef SQSTRING_H
-#define SQSTRING_H
-
-/* forward decls */
-
-class CacheManager;
-
-#define DEBUGSTRINGS 0
-#if DEBUGSTRINGS
-#include "splay.h"
-
-class SqString;
-
-class SqStringRegistry
-{
-
-public:
- static StringRegistry &Instance();
-
- void add
- (SqString const *);
-
- void registerWithCacheManager(CacheManager & manager);
-
- void remove
- (SqString const *);
-
-private:
- static OBJH Stat;
-
- static StringRegistry Instance_;
-
- static SplayNode<SqString const *>::SPLAYWALKEE Stater;
-
- Splay<SqString const *> entries;
-
- bool registered;
-
-};
-
-class StoreEntry;
-#endif
-
-class SqString
-{
-
-public:
-
- /* std::string API available */
- _SQUID_INLINE_ SqString();
- SqString (char const *);
- SqString (SqString const &);
- ~SqString();
-
- SqString &operator =(char const *);
- SqString &operator =(SqString const &);
- bool operator ==(SqString const &) const;
- bool operator !=(SqString const &) const;
- bool operator >=(SqString const &) const;
- bool operator <=(SqString const &) const;
- bool operator >(SqString const &) const;
- bool operator <(SqString const &) const;
-
- _SQUID_INLINE_ int size() const;
- _SQUID_INLINE_ char const * c_str() const;
-
- const char& operator [](unsigned int) const;
- char& operator [](unsigned int);
-
- void clear();
-
- void append(char const *buf, int len);
- void append(char const *buf);
- void append(char const);
- void append(SqString const &);
-
- _SQUID_INLINE_ bool empty() const;
- _SQUID_INLINE_ int compare(char const *) const;
- _SQUID_INLINE_ int compare(char const *, size_t count) const;
- _SQUID_INLINE_ int compare(SqString const &) const;
-
-/* Custom Squid Operations available */
- /// Super-efficient string assignment. Moves internal content from one object to another.
- /// then resets the initial pobject to empty.
- _SQUID_INLINE_ void absorb(SqString &old);
- _SQUID_INLINE_ const char * pos(char const *) const;
- _SQUID_INLINE_ const char * pos(char const ch) const;
- _SQUID_INLINE_ const char * rpos(char const ch) const;
-
- _SQUID_INLINE_ void set
- (char const *loc, char const ch);
-
- _SQUID_INLINE_ void cut (size_t newLength);
-
- _SQUID_INLINE_ void cutPointer (char const *loc);
-
-#if DEBUGSTRINGS
-
- void stat (StoreEntry *) const;
-
-#endif
-
- void limitInit(const char *str, unsigned int len);
-private:
- void initBuf(size_t sz);
- void init (char const *);
-
- /* never reference these directly! */
- unsigned short int size_; /* buffer size; 64K limit */
-
- unsigned short int len_; /* current length */
-
- char *buf_;
-};
-
-#ifdef _USE_INLINE_
-#include "SqString.cci"
-#endif
-
-#endif /* SQSTRING_H */
-
/*
- * $Id: SquidString.h,v 1.9 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: SquidString.h,v 1.10 2007/05/29 13:31:38 amosjeffries Exp $
*
* DEBUG: section 67 String
- * AUTHOR: Duane Wessels, Amos Jeffries
+ * AUTHOR: Duane Wessels
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
* ----------------------------------------------------------
*
*/
-/**
- *
- * To allow for easy future updates to the string handling within squid
- * We adopt the std::string API as the basis for string operations.
- * Then we typedef string (due to namespacing actually ::Squid::string)
- * as the preferred string handling class.
- * For Release 3.0 it is intended that the old String (no SquidString)
- * Will be the default string type.
- * For Release 3.1 it is expected that either std::string of another
- * custom managed type will be defined as default.
- *
- * NP: checkout http://johnpanzer.com/tsc_cuj/ToolboxOfStrings.html
- * for possibly better and faster strings.
- *
- * This has been done for several reasons:
- *
- * The initial String implementation was incomplete and non-standard
- * std::string provides a better known API for string handling
- * std::string or a derivative may be used in future within squid
- * String is a defined alternative to std::string in some systems
- *
- * These changes:
- * - move the old String class to SquidString making the
- * internal definition explicit.
- * - provide the well-known type of 'string' for general use
- * - migrate custom functions to well-known API:
- * buf() -> c_str()
- * clean() -> clear()
- * - remove redundant functions:
- * buf(char*) -> operator=(char*)
- * initBuf(char*) -> operator=(char*)
- * reset(char*) -> operator=(char*)
- * - make init(char*) private for use by various assignment/costructor
- * - define standard string operators
- * - define debugs stream operator
- *
- */
-
#ifndef SQUID_STRING_H
#define SQUID_STRING_H
- /* Provide standard 'string' type */
- /* class defined by the #include file MUST present the basic std::string API */
- /* at least partially as not all operatios are used by squid. */
- /* API Ref: http://www.sgi.com/tech/stl/basic_string.html */
+/* forward decls */
+
+class CacheManager;
+
+#define DEBUGSTRINGS 0
+#if DEBUGSTRINGS
+#include "splay.h"
+
+class String;
+
+class StringRegistry
+{
+
+public:
+ static StringRegistry &Instance();
+
+ void add
+ (String const *);
+
+ void registerWithCacheManager(CacheManager & manager);
+
+ void remove
+ (String const *);
+
+private:
+ static OBJH Stat;
-#include "SqString.h"
-typedef SqString string;
+ static StringRegistry Instance_;
+ static SplayNode<String const *>::SPLAYWALKEE Stater;
- /* Overload standard C functions using the basic string API */
+ Splay<String const *> entries;
-inline int strncasecmp(const string &lhs, const string &rhs, size_t len) { return strncasecmp(lhs.c_str(), rhs.c_str(), len); }
-inline int strcasecmp(const string &lhs, const string &rhs) { return strcasecmp(lhs.c_str(), rhs.c_str()); }
+ bool registered;
-inline int strncmp(const string &lhs, const string &rhs, size_t len) { return strncmp(lhs.c_str(), rhs.c_str(), len); }
-inline int strcmp(const string &lhs, const string &rhs) { return strcmp(lhs.c_str(), rhs.c_str()); }
+};
-inline const char * strpbrk(const string &lhs, const string &rhs) { return strpbrk(lhs.c_str(), rhs.c_str()); }
+class StoreEntry;
+#endif
-inline const char * strstr(const string &lhs, const string &rhs) { return strstr(lhs.c_str(), rhs.c_str()); }
+class String
+{
-inline std::ostream& operator <<(std::ostream &os, const string &s) { os << s.c_str(); return os; }
+public:
+ _SQUID_INLINE_ String();
+ String (char const *);
+ String (String const &);
+ ~String();
+
+ String &operator =(char const *);
+ String &operator =(String const &);
+ bool operator ==(String const &) const;
+ bool operator !=(String const &) const;
+
+ _SQUID_INLINE_ int size() const;
+ _SQUID_INLINE_ char const * buf() const;
+ void buf(char *);
+ void init (char const *);
+ void initBuf(size_t sz);
+ void limitInit(const char *str, int len);
+ void clean();
+ void reset(char const *str);
+ void append(char const *buf, int len);
+ void append(char const *buf);
+ void append(char const);
+ void append (String const &);
+ void absorb(String &old);
+ _SQUID_INLINE_ const char * pos(char const *) const;
+ _SQUID_INLINE_ const char * pos(char const ch) const;
+ _SQUID_INLINE_ const char * rpos(char const ch) const;
+ _SQUID_INLINE_ int cmp (char const *) const;
+ _SQUID_INLINE_ int cmp (char const *, size_t count) const;
+ _SQUID_INLINE_ int cmp (String const &) const;
+ _SQUID_INLINE_ int caseCmp (char const *) const;
+ _SQUID_INLINE_ int caseCmp (char const *, size_t count) const;
+
+ _SQUID_INLINE_ void set
+ (char const *loc, char const ch);
+
+ _SQUID_INLINE_ void cut (size_t newLength);
+
+ _SQUID_INLINE_ void cutPointer (char const *loc);
+
+#if DEBUGSTRINGS
+
+ void stat (StoreEntry *) const;
+
+#endif
+
+private:
+ /* never reference these directly! */
+ unsigned short int size_; /* buffer size; 64K limit */
+
+ unsigned short int len_; /* current length */
+
+ char *buf_;
+};
+
+_SQUID_INLINE_ std::ostream & operator<<(std::ostream& os, String const &aString);
+
+#ifdef _USE_INLINE_
+#include "String.cci"
+#endif
#endif /* SQUID_STRING_H */
+
+
/*
- * $Id: Store.h,v 1.34 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: Store.h,v 1.35 2007/05/29 13:31:38 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
/* TODO: imeplement the async version */
virtual void get
- (string const key , STOREGETCLIENT callback, void *cbdata) = 0;
+ (String const key , STOREGETCLIENT callback, void *cbdata) = 0;
/* prepare the store for use. The store need not be usable immediately,
* it should respond to readable() and writable() with true as soon
virtual void unlink (StoreEntry &);
/* search in the store */
- virtual StoreSearch *search(string const url, HttpRequest *) = 0;
+ virtual StoreSearch *search(String const url, HttpRequest *) = 0;
/* pulled up from SwapDir for migration.... probably do not belong here */
virtual void reference(StoreEntry &) = 0; /* Reference this object */
/*
- * $Id: StoreHashIndex.h,v 1.3 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: StoreHashIndex.h,v 1.4 2007/05/29 13:31:38 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
(const cache_key *);
virtual void get
- (string const, STOREGETCLIENT, void * cbdata);
+ (String const, STOREGETCLIENT, void * cbdata);
virtual void init();
virtual void updateSize(size_t, int);
- virtual StoreSearch *search(string const url, HttpRequest *);
+ virtual StoreSearch *search(String const url, HttpRequest *);
private:
/* migration logic */
/*
- * $Id: SqString.cc,v 1.7 2007/05/22 01:15:55 hno Exp $
+ * $Id: String.cc,v 1.26 2007/05/29 13:31:38 amosjeffries Exp $
*
* DEBUG: section 67 String
* AUTHOR: Duane Wessels
*/
#include "squid.h"
-#include "SqString.h"
#include "Store.h"
void
-SqString::initBuf(size_t sz)
+String::initBuf(size_t sz)
{
- size_t bsz;
PROF_start(StringInitBuf);
- clear();
+ buf((char *)memAllocString(sz, &sz));
assert(sz < 65536);
- buf_ = (char *)memAllocString(sz, &bsz);
- assert(bsz < 65536);
- assert(bsz >= sz);
- size_ = bsz;
+ size_ = sz;
PROF_stop(StringInitBuf);
}
void
-SqString::limitInit(const char *str, unsigned int len)
-{
- PROF_start(StringLimitInit);
- assert(this && str);
- initBuf(len + 1);
- len_ = len;
- xmemcpy(buf_, str, len);
- buf_[len] = '\0';
- PROF_stop(StringLimitInit);
-}
-
-void
-SqString::init(char const *str)
+String::init(char const *str)
{
assert(this);
PROF_start(StringInit);
-
if (str)
limitInit(str, strlen(str));
else
- clear();
+ clean();
PROF_stop(StringInit);
}
-void
-SqString::clear()
+String::String (char const *aString) : size_(0), len_(0), buf_(NULL)
{
- PROF_start(StringClean);
- assert(this);
-
- if (buf_)
- memFreeString(size_, buf_);
-
- len_ = 0;
- size_ = 0;
- buf_ = NULL;
- PROF_stop(StringClean);
-}
-
-SqString::~SqString()
-{
- clear();
-#if DEBUGSTRINGS
-
- SqStringRegistry::Instance().remove(this);
-#endif
-}
-
-SqString::SqString (char const *aString)
-{
- memset(this, 0, sizeof(SqString));
-
- init(aString);
+ init (aString);
#if DEBUGSTRINGS
- SqStringRegistry::Instance().add(this);
+ StringRegistry::Instance().add(this);
#endif
}
-SqString &
-SqString::operator =(char const *aString)
+String &
+String::operator =(char const *aString)
{
- assert(this);
- init(aString);
+ clean();
+ init (aString);
return *this;
}
-SqString &
-SqString::operator = (SqString const &old)
+String &
+String::operator = (String const &old)
{
- if (old.size())
- limitInit(old.c_str(), old.size());
- else
- clear();
+ clean ();
+
+ if (old.len_)
+ limitInit (old.buf(), old.len_);
return *this;
}
bool
-SqString::operator == (SqString const &that) const
+String::operator == (String const &that) const
{
- return (this->compare(that) == 0);
-}
+ if (0 == this->cmp(that))
+ return true;
-bool
-SqString::operator != (SqString const &that) const
-{
- return (this->compare(that) != 0);
+ return false;
}
bool
-SqString::operator >= (SqString const &that) const
+String::operator != (String const &that) const
{
- return (this->compare(that) >= 0);
+ if (0 == this->cmp(that))
+ return false;
+
+ return true;
}
-bool
-SqString::operator <= (SqString const &that) const
+void
+String::limitInit(const char *str, int len)
{
- return (this->compare(that) <= 0);
+ PROF_start(StringLimitInit);
+ assert(this && str);
+ initBuf(len + 1);
+ len_ = len;
+ xmemcpy(buf_, str, len);
+ buf_[len] = '\0';
+ PROF_stop(StringLimitInit);
}
-bool
-SqString::operator > (SqString const &that) const
+String::String (String const &old) : size_(0), len_(0), buf_(NULL)
{
- return (this->compare(that) > 0);
+ init (old.buf());
+#if DEBUGSTRINGS
+
+ StringRegistry::Instance().add(this);
+#endif
}
-bool
-SqString::operator < (SqString const &that) const
+void
+String::clean()
{
- return (this->compare(that) < 0);
+ PROF_start(StringClean);
+ assert(this);
+
+ if (buf())
+ memFreeString(size_, buf_);
+
+ len_ = 0;
+
+ size_ = 0;
+
+ buf_ = NULL;
+ PROF_stop(StringClean);
}
-SqString::SqString (SqString const &old)
+String::~String()
{
- memset(this, 0, sizeof(SqString));
-
- operator=(old);
+ clean();
#if DEBUGSTRINGS
- SqStringRegistry::Instance().add(this);
+ StringRegistry::Instance().remove(this);
#endif
}
void
-SqString::append(const char *str, int len)
+String::reset(const char *str)
+{
+ PROF_start(StringReset);
+ clean();
+ init(str);
+ PROF_stop(StringReset);
+}
+
+void
+String::append(const char *str, int len)
{
assert(this);
+ assert(str && len >= 0);
PROF_start(StringAppend);
-
- if(len < 1 || str == NULL)
- return;
-
- if ( (len_ + len +1) < size_) {
- operator[](len_+len) = '\0';
- xmemcpy(buf_+len_, str, len);
+ if (len_ + len < size_) {
+ strncat(buf_, str, len);
len_ += len;
} else {
- size_t ssz = len_ + len;
- size_t bsz = len_ + len + 1;
- char* tmp = (char *)memAllocString(bsz, &bsz);
- assert(bsz < 65536);
- assert(bsz > ssz);
+ String snew;
+ snew.len_ = len_ + len;
+ snew.initBuf(snew.len_ + 1);
if (buf_)
- xmemcpy(tmp, buf_, len_);
+ xmemcpy(snew.buf_, buf(), len_);
if (len)
- xmemcpy(tmp + len_, str, len);
-
- tmp[ssz] = '\0';
+ xmemcpy(snew.buf_ + len_, str, len);
- clear();
+ snew.buf_[snew.len_] = '\0';
- size_ = bsz;
- len_ = ssz;
- buf_ = tmp;
- tmp = NULL;
+ absorb(snew);
}
PROF_stop(StringAppend);
}
void
-SqString::append(char const *str)
+String::append(char const *str)
{
- if(!str) return;
+ assert (str);
append (str, strlen(str));
}
void
-SqString::append (char chr)
+String::append (char chr)
{
char myString[2];
myString[0]=chr;
}
void
-SqString::append(SqString const &old)
+String::append(String const &old)
{
- append (old.c_str(), old.size());
+ append (old.buf(), old.len_);
}
-const char&
-SqString::operator [](unsigned int pos) const
+void
+String::absorb(String &old)
{
- assert(pos < size_ );
-
- return buf_[pos];
+ clean();
+ size_ = old.size_;
+ buf (old.buf_);
+ len_ = old.len_;
+ old.size_ = 0;
+ old.buf_ = NULL;
+ old.len_ = 0;
}
-char&
-SqString::operator [](unsigned int pos)
+void
+String::buf(char *newBuf)
{
- assert(pos < size_ );
-
- return buf_[pos];
+ assert (buf_ == NULL);
+ buf_ = newBuf;
}
#if DEBUGSTRINGS
void
-SqString::stat(StoreEntry *entry) const
+String::stat(StoreEntry *entry) const
{
- storeAppendPrintf(entry, "%p : %d/%d \"%s\"\n",this,len_, size_, c_str());
+ storeAppendPrintf(entry, "%p : %d/%d \"%s\"\n",this,len_, size_, buf());
}
-SqStringRegistry &
-SqStringRegistry::Instance()
+StringRegistry &
+StringRegistry::Instance()
{
return Instance_;
}
}
void
-SqStringRegistry::registerWithCacheManager(CacheManager & manager)
+StringRegistry::registerWithCacheManager(CacheManager & manager)
{
manager.registerAction("strings",
"Strings in use in squid", Stat, 0, 1);
}
void
-SqStringRegistry::add(SqString const *entry)
+
+StringRegistry::add
+ (String const *entry)
{
entries.insert(entry, ptrcmp);
}
void
-SqStringRegistry::remove(SqString const *entry)
+
+StringRegistry::remove
+ (String const *entry)
{
entries.remove(entry, ptrcmp);
}
-SqStringRegistry SqStringRegistry::Instance_;
+StringRegistry StringRegistry::Instance_;
extern size_t memStringCount();
void
-SqStringRegistry::Stat(StoreEntry *entry)
+StringRegistry::Stat(StoreEntry *entry)
{
storeAppendPrintf(entry, "%lu entries, %lu reported from MemPool\n", (unsigned long) Instance().entries.elements, (unsigned long) memStringCount());
Instance().entries.head->walk(Stater, entry);
}
void
-SqStringRegistry::Stater(SqString const * const & nodedata, void *state)
+StringRegistry::Stater(String const * const & nodedata, void *state)
{
StoreEntry *entry = (StoreEntry *) state;
nodedata->stat(entry);
#endif
+/* TODO: move onto String */
+int
+stringHasWhitespace(const char *s)
+{
+ return strpbrk(s, w_space) != NULL;
+}
+
+/* TODO: move onto String */
+int
+stringHasCntl(const char *s)
+{
+ unsigned char c;
+
+ while ((c = (unsigned char) *s++) != '\0') {
+ if (c <= 0x1f)
+ return 1;
+
+ if (c >= 0x7f && c <= 0x9f)
+ return 1;
+ }
+
+ return 0;
+}
+
/*
* Similar to strtok, but has some rudimentary knowledge
* of quoting
return (char *) word;
}
+const char *
+checkNullString(const char *p)
+{
+ return p ? p : "(NULL)";
+}
+
#ifndef _USE_INLINE_
-#include "SqString.cci"
+#include "String.cci"
#endif
/*
- * $Id: SqString.cci,v 1.2 2007/05/19 06:49:12 amosjeffries Exp $
+ * $Id: String.cci,v 1.9 2007/05/29 13:31:38 amosjeffries Exp $
*
* DEBUG: section 67 String
* AUTHOR: Duane Wessels
*
*/
-SqString::SqString() : size_(0), len_(0), buf_ (NULL)
+String::String() : size_(0), len_(0), buf_ (NULL)
{
#if DEBUGSTRINGS
- SqStringRegistry::Instance().add(this);
+ StringRegistry::Instance().add(this);
#endif
}
-void
-SqString::absorb(SqString &old)
-{
- clear();
- size_ = old.size_;
- buf_ = old.buf_;
- len_ = old.len_;
- old.size_ = 0;
- old.buf_ = NULL;
- old.len_ = 0;
-}
-
int
-SqString::size() const
+String::size() const
{
return len_;
}
char const *
-SqString::c_str() const
+String::buf() const
{
return buf_;
}
const char *
-SqString::pos(char const *aString) const
+String::pos(char const *aString) const
{
- return strstr(c_str(), aString);
+ return strstr(buf(), aString);
}
const char *
-SqString::pos(char const ch) const
+String::pos(char const ch) const
{
- return strchr(c_str(), ch);
+ return strchr(buf(), ch);
}
const char *
-SqString::rpos(char const ch) const
-{
- return strrchr(c_str(), ch);
-}
-
-bool
-SqString::empty() const
+String::rpos(char const ch) const
{
- return (buf_ == NULL || len_ == 0);
+ return strrchr(buf(), (ch));
}
int
-SqString::compare(char const *aString) const
+String::cmp (char const *aString) const
{
/* strcmp fails on NULLS */
if (aString == NULL || aString[0] == '\0')
return 1;
- return strcmp(c_str(), aString);
+ return strcmp(buf(), aString);
}
int
-SqString::compare(char const *aString, size_t count) const
+String::cmp (char const *aString, size_t count) const
{
/* always the same at length 0 */
if (aString == NULL || aString[0] == '\0')
return 1;
- return strncmp(c_str(), aString, count);
+ return strncmp(buf(), aString, count);
}
int
-SqString::compare(SqString const &aString) const
+String::cmp (String const &aString) const
{
/* strcmp fails on NULLS */
if (aString.size() == 0)
return 1;
- return strcmp(c_str(), aString.c_str());
+ return strcmp(buf(), aString.buf());
+}
+
+int
+String::caseCmp (char const *aString) const
+{
+ return strcasecmp(buf(), aString);
+}
+
+int
+String::caseCmp (char const *aString, size_t count) const
+{
+ return strncasecmp(buf(), aString, count);
}
+/* FIXME: this is can perform buffer overflows and underflows! */
void
-SqString::set (char const *loc, char const ch)
+String::set (char const *loc, char const ch)
{
- operator[](loc-buf_) = ch;
+ buf_[loc-buf_] = ch;
}
+/* FIXME: this is can perform buffer overflows and underflows! */
void
-SqString::cut (size_t newLength)
+String::cut (size_t newLength)
{
- operator[](newLength) = '\0';
len_ = newLength;
+ buf_[newLength] = '\0';
}
+/* FIXME: this is can perform buffer overflows and underflows! */
void
-SqString::cutPointer (char const *loc)
+String::cutPointer (char const *loc)
{
- operator[](loc-buf_) = '\0';
len_ = loc-buf_;
+ buf_[len_] = '\0';
}
+
+std::ostream &
+operator<<(std::ostream& os, String const &aString)
+{
+ os << aString.buf();
+ return os;
+}
+
+
/*
- * $Id: SwapDir.cc,v 1.12 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: SwapDir.cc,v 1.13 2007/05/29 13:31:38 amosjeffries Exp $
*
* DEBUG: section 20 Swap Dir base object
* AUTHOR: Robert Collins
void
SwapDir::get
- (string const key, STOREGETCLIENT callback, void *cbdata)
+ (String const key, STOREGETCLIENT callback, void *cbdata)
{
fatal("not implemented");
}
/*
- * $Id: SwapDir.h,v 1.12 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: SwapDir.h,v 1.13 2007/05/29 13:31:38 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
(const cache_key *);
virtual void get
- (string const, STOREGETCLIENT, void * cbdata);
+ (String const, STOREGETCLIENT, void * cbdata);
virtual void init();
virtual void sync(); /* Sync the store prior to shutdown */
- virtual StoreSearch *search(string const url, HttpRequest *);
+ virtual StoreSearch *search(String const url, HttpRequest *);
virtual void reference(StoreEntry &); /* Reference this object */
(const cache_key *);
virtual void get
- (string const, STOREGETCLIENT, void * cbdata);
+ (String const, STOREGETCLIENT, void * cbdata);
virtual size_t maxSize() const { return max_size;}
virtual size_t minSize() const;
virtual void stat (StoreEntry &anEntry) const;
- virtual StoreSearch *search(string const url, HttpRequest *) = 0;
+ virtual StoreSearch *search(String const url, HttpRequest *) = 0;
virtual void updateSize(size_t size, int sign);
/*
- * $Id: access_log.cc,v 1.125 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: access_log.cc,v 1.126 2007/05/29 13:31:38 amosjeffries Exp $
*
* DEBUG: section 46 Access Log
* AUTHOR: Duane Wessels
logformat_token *fmt;
static MemBuf mb;
char tmp[1024];
- string sb;
+ String sb;
mb.reset();
if (al->request)
sb = al->request->header.getByName(fmt->data.header.header);
- out = sb.c_str();
+ out = sb.buf();
quote = 1;
if (al->reply)
sb = al->reply->header.getByName(fmt->data.header.header);
- out = sb.c_str();
+ out = sb.buf();
quote = 1;
if (al->request)
sb = al->request->header.getByNameListMember(fmt->data.header.header, fmt->data.header.element, fmt->data.header.separator);
- out = sb.c_str();
+ out = sb.buf();
quote = 1;
if (al->reply)
sb = al->reply->header.getByNameListMember(fmt->data.header.header, fmt->data.header.element, fmt->data.header.separator);
- out = sb.c_str();
+ out = sb.buf();
quote = 1;
case LFT_REQUEST_URLPATH:
if (al->request) {
- out = al->request->urlpath.c_str();
+ out = al->request->urlpath.buf();
quote = 1;
}
break;
case LFT_TAG:
if (al->request)
- out = al->request->tag.c_str();
+ out = al->request->tag.buf();
quote = 1;
case LFT_EXT_LOG:
if (al->request)
- out = al->request->extacl_log.c_str();
+ out = al->request->extacl_log.buf();
quote = 1;
if (fmt->space)
mb.append(" ", 1);
- sb.clear();
+ sb.clean();
if (dofree)
safe_free(out);
static void
fvdbInit(void)
{
- via_table = hash_create((HASHCMP *) std::strcmp, 977, hash4);
- forw_table = hash_create((HASHCMP *) std::strcmp, 977, hash4);
+ via_table = hash_create((HASHCMP *) strcmp, 977, hash4);
+ forw_table = hash_create((HASHCMP *) strcmp, 977, hash4);
}
static void
{
hashFreeItems(via_table, fvdbFreeEntry);
hashFreeMemory(via_table);
- via_table = hash_create((HASHCMP *) std::strcmp, 977, hash4);
+ via_table = hash_create((HASHCMP *) strcmp, 977, hash4);
hashFreeItems(forw_table, fvdbFreeEntry);
hashFreeMemory(forw_table);
- forw_table = hash_create((HASHCMP *) std::strcmp, 977, hash4);
+ forw_table = hash_create((HASHCMP *) strcmp, 977, hash4);
}
#endif
/*
- * $Id: auth_digest.cc,v 1.56 2007/05/18 06:41:31 amosjeffries Exp $
+ * $Id: auth_digest.cc,v 1.57 2007/05/29 13:31:46 amosjeffries Exp $
*
* DEBUG: section 29 Authenticator
* AUTHOR: Robert Collins
digest_nonce_pool = memPoolCreate("Digest Scheme nonce's", sizeof(digest_nonce_h));
if (!digest_nonce_cache) {
- digest_nonce_cache = hash_create((HASHCMP *) std::strcmp, 7921, hash_string);
+ digest_nonce_cache = hash_create((HASHCMP *) strcmp, 7921, hash_string);
assert(digest_nonce_cache);
eventAdd("Digest none cache maintenance", authenticateDigestNonceCacheCleanup, NULL, digestConfig.nonceGCInterval, 1);
}
while (xisspace(*proxy_auth))
proxy_auth++;
- string temp(proxy_auth);
+ String temp(proxy_auth);
while (strListGetItem(&temp, ',', &item, &ilen, &pos)) {
if ((p = strchr(item, '=')) && (p - item < ilen))
}
}
- temp.clear();
+ temp.clean();
/* now we validate the data given to us */
/*
- * $Id: auth_negotiate.cc,v 1.19 2007/05/18 06:41:31 amosjeffries Exp $
+ * $Id: auth_negotiate.cc,v 1.20 2007/05/29 13:31:46 amosjeffries Exp $
*
* DEBUG: section 29 Negotiate Authenticator
* AUTHOR: Robert Collins, Henrik Nordstrom, Francesco Chemolli
negotiateauthenticators = helperStatefulCreate("negotiateauthenticator");
if (!proxy_auth_cache)
- proxy_auth_cache = hash_create((HASHCMP *) std::strcmp, 7921, hash_string);
+ proxy_auth_cache = hash_create((HASHCMP *) strcmp, 7921, hash_string);
assert(proxy_auth_cache);
/*
- * $Id: auth_ntlm.cc,v 1.69 2007/05/18 06:41:32 amosjeffries Exp $
+ * $Id: auth_ntlm.cc,v 1.70 2007/05/29 13:31:46 amosjeffries Exp $
*
* DEBUG: section 29 NTLM Authenticator
* AUTHOR: Robert Collins, Henrik Nordstrom, Francesco Chemolli
ntlmauthenticators = helperStatefulCreate("ntlmauthenticator");
if (!proxy_auth_cache)
- proxy_auth_cache = hash_create((HASHCMP *) std::strcmp, 7921, hash_string);
+ proxy_auth_cache = hash_create((HASHCMP *) strcmp, 7921, hash_string);
assert(proxy_auth_cache);
/*
- * $Id: cache_cf.cc,v 1.511 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: cache_cf.cc,v 1.512 2007/05/29 13:31:38 amosjeffries Exp $
*
* DEBUG: section 3 Configuration File Parsing
* AUTHOR: Harvest Derived
}
void
-ConfigParser::ParseString(string &var)
+ConfigParser::ParseString(String *var)
{
char *token = strtok(NULL, w_space);
if (token == NULL)
self_destruct();
- var = token;
+ var->reset(token);
}
static void
/*
- * $Id: client_db.cc,v 1.69 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: client_db.cc,v 1.70 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 0 Client Database
* AUTHOR: Duane Wessels
if (client_table)
return;
- client_table = hash_create((HASHCMP *) std::strcmp, CLIENT_DB_HASH_SIZE, hash_string);
+ client_table = hash_create((HASHCMP *) strcmp, CLIENT_DB_HASH_SIZE, hash_string);
}
void
/*
- * $Id: client_side.cc,v 1.755 2007/05/18 18:30:41 wessels Exp $
+ * $Id: client_side.cc,v 1.756 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 33 Client-side Routines
* AUTHOR: Duane Wessels
aLogEntry->http.version = request->http_ver;
aLogEntry->hier = request->hier;
- aLogEntry->cache.extuser = request->extacl_user.c_str();
+ aLogEntry->cache.extuser = request->extacl_user.buf();
if (request->auth_user_request) {
if (al.reply) {
al.http.code = al.reply->sline.status;
- al.http.content_type = al.reply->content_type.c_str();
+ al.http.content_type = al.reply->content_type.buf();
} else if (loggingEntry() && loggingEntry()->mem_obj) {
al.http.code = loggingEntry()->mem_obj->getReply()->sline.status;
- al.http.content_type = loggingEntry()->mem_obj->getReply()->content_type.c_str();
+ al.http.content_type = loggingEntry()->mem_obj->getReply()->content_type.buf();
}
debugs(33, 9, "clientLogRequest: http.code='" << al.http.code << "'");
safe_free(uri);
safe_free(log_uri);
safe_free(redirect.location);
- range_iter.boundary.clear();
+ range_iter.boundary.clean();
HTTPMSGUNLOCK(request);
if (client_stream.tail)
/* put terminating boundary for multiparts */
static void
-clientPackTermBound(string boundary, MemBuf * mb)
+clientPackTermBound(String boundary, MemBuf * mb)
{
- mb->Printf("\r\n--%s--\r\n", boundary.c_str());
+ mb->Printf("\r\n--%s--\r\n", boundary.buf());
debugs(33, 6, "clientPackTermBound: buf offset: " << mb->size);
}
/* appends a "part" HTTP header (as in a multi-part/range reply) to the buffer */
static void
-clientPackRangeHdr(const HttpReply * rep, const HttpHdrRangeSpec * spec, string boundary, MemBuf * mb)
+clientPackRangeHdr(const HttpReply * rep, const HttpHdrRangeSpec * spec, String boundary, MemBuf * mb)
{
HttpHeader hdr(hoReply);
Packer p;
assert(spec);
/* put boundary */
- debugs(33, 5, "clientPackRangeHdr: appending boundary: " << boundary);
+ debugs(33, 5, "clientPackRangeHdr: appending boundary: " <<
+ boundary.buf());
/* rfc2046 requires to _prepend_ boundary with <crlf>! */
- mb->Printf("\r\n--%s\r\n", boundary.c_str());
+ mb->Printf("\r\n--%s\r\n", boundary.buf());
/* stuff the header with required entries and pack it */
/* generates a "unique" boundary string for multipart responses
* the caller is responsible for cleaning the string */
-string
+String
ClientHttpRequest::rangeBoundaryStr() const
{
assert(this);
const char *key;
- string b (full_appname_string);
+ String b (full_appname_string);
b.append (":",1);
key = storeEntry()->getMD5Text();
b.append(key, strlen(key));
hdr->delById(HDR_CONTENT_TYPE);
httpHeaderPutStrf(hdr, HDR_CONTENT_TYPE,
"multipart/byteranges; boundary=\"%s\"",
- http->range_iter.boundary.c_str());
+ http->range_iter.boundary.buf());
/* Content-Length is not required in multipart responses
* but it is always nice to have one */
actual_clen = http->mRangeCLen();
request->flags.tproxy = conn->port->tproxy;
#endif
- if (internalCheck(request->urlpath.c_str())) {
+ if (internalCheck(request->urlpath.buf())) {
if (internalHostnameIs(request->host) &&
request->port == getMyPort()) {
http->flags.internal = 1;
- } else if (Config.onoff.global_internal_static && internalStaticCheck(request->urlpath.c_str())) {
+ } else if (Config.onoff.global_internal_static && internalStaticCheck(request->urlpath.buf())) {
xstrncpy(request->host, internalHostname(),
SQUIDHOSTNAMELEN);
request->port = getMyPort();
/*
- * $Id: client_side_reply.cc,v 1.128 2007/05/18 18:30:41 wessels Exp $
+ * $Id: client_side_reply.cc,v 1.129 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 88 Client-side Reply Routines
* AUTHOR: Robert Collins (Originally Duane Wessels in client_side.c)
int headers_deleted = 0;
while ((e = hdr->getEntry(&pos))) {
if (e->id == HDR_WWW_AUTHENTICATE || e->id == HDR_PROXY_AUTHENTICATE) {
- const char *value = e->value.c_str();
+ const char *value = e->value.buf();
if ((strncasecmp(value, "NTLM", 4) == 0 &&
(value[4] == '\0' || value[4] == ' '))
/* Append VIA */
{
LOCAL_ARRAY(char, bbuf, MAX_URL + 32);
- string strVia;
+ String strVia;
hdr->getList(HDR_VIA, &strVia);
snprintf(bbuf, sizeof(bbuf), "%d.%d %s",
reply->sline.version.major,
ThisCache);
strListAdd(&strVia, bbuf, ',');
hdr->delById(HDR_VIA);
- hdr->putStr(HDR_VIA, strVia.c_str());
+ hdr->putStr(HDR_VIA, strVia.buf());
}
/* Signal keep-alive if needed */
hdr->putStr(http->flags.accel ? HDR_CONNECTION : HDR_PROXY_CONNECTION,
/*
- * $Id: client_side_request.cc,v 1.88 2007/05/26 06:38:04 wessels Exp $
+ * $Id: client_side_request.cc,v 1.89 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 85 Client-side Request Routines
* AUTHOR: Robert Collins (Originally Duane Wessels in client_side.c)
#else
if (req_hdr->has(HDR_PRAGMA)) {
- string s = req_hdr->getList(HDR_PRAGMA);
+ String s = req_hdr->getList(HDR_PRAGMA);
if (strListIsMember(&s, "no-cache", ','))
no_cache++;
+
+ s.clean();
}
if (request->cache_control)
request->flags.auth = 1;
if (req_hdr->has(HDR_VIA)) {
- string s = req_hdr->getList(HDR_VIA);
+ String s = req_hdr->getList(HDR_VIA);
/*
* ThisCache cannot be a member of Via header, "1.0 ThisCache" can.
* Note ThisCache2 has a space prepended to the hostname so we don't
}
#if FORW_VIA_DB
- fvdbCountVia(s.c_str());
+ fvdbCountVia(s.buf());
#endif
- s.clear();
+ s.clean();
}
#if USE_USERAGENT_LOG
#if FORW_VIA_DB
if (req_hdr->has(HDR_X_FORWARDED_FOR)) {
- string s = req_hdr->getList(HDR_X_FORWARDED_FOR);
- fvdbCountForw(s.c_str());
- s.clear();
+ String s = req_hdr->getList(HDR_X_FORWARDED_FOR);
+ fvdbCountForw(s.buf());
+ s.clean();
}
#endif
/*
- * $Id: client_side_request.h,v 1.29 2007/05/18 06:41:23 amosjeffries Exp $
+ * $Id: client_side_request.h,v 1.30 2007/05/29 13:31:39 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
ClientHttpRequest(ClientHttpRequest const &);
ClientHttpRequest& operator=(ClientHttpRequest const &);
- string rangeBoundaryStr() const;
+ String rangeBoundaryStr() const;
void freeResources();
void updateCounters();
void logRequest();
/*
- * $Id: dns_internal.cc,v 1.99 2007/05/18 06:41:24 amosjeffries Exp $
+ * $Id: dns_internal.cc,v 1.100 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 78 DNS lookups; interacts with lib/rfc1035.c
* AUTHOR: Duane Wessels
if (!init) {
memDataInit(MEM_IDNS_QUERY, "idns_query", sizeof(idns_query), 0);
memset(RcodeMatrix, '\0', sizeof(RcodeMatrix));
- idns_lookup_hash = hash_create((HASHCMP *) std::strcmp, 103, hash_string);
+ idns_lookup_hash = hash_create((HASHCMP *) strcmp, 103, hash_string);
init++;
}
}
/*
- * $Id: errorpage.cc,v 1.226 2007/05/18 06:41:24 amosjeffries Exp $
+ * $Id: errorpage.cc,v 1.227 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 4 Error Generation
* AUTHOR: Duane Wessels
Packer p;
str.Printf("%s %s HTTP/%d.%d\n",
RequestMethodStr[r->method],
- r->urlpath.size() ? r->urlpath.c_str() : "/",
+ r->urlpath.size() ? r->urlpath.buf() : "/",
r->http_ver.major, r->http_ver.minor);
packerToMemInit(&p, &str);
r->header.packInto(&p);
Packer p;
mb.Printf("%s %s HTTP/%d.%d\n",
RequestMethodStr[r->method],
- r->urlpath.size() ? r->urlpath.c_str() : "/",
+ r->urlpath.size() ? r->urlpath.buf() : "/",
r->http_ver.major, r->http_ver.minor);
packerToMemInit(&p, &mb);
r->header.packInto(&p);
/*
- * $Id: external_acl.cc,v 1.79 2007/05/22 01:16:40 hno Exp $
+ * $Id: external_acl.cc,v 1.80 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 82 External ACL
* AUTHOR: Henrik Nordstrom, MARA Systems AB
external_acl_cache_touch(acl->def, entry);
result = entry->result;
- external_acl_message = entry->message.c_str();
+ external_acl_message = entry->message.buf();
debugs(82, 2, "aclMatchExternal: " << acl->def->name << " = " << result);
for (format = acl_data->def->format; format; format = format->next) {
const char *str = NULL;
- string sb;
+ String sb;
switch (format->type) {
break;
case _external_acl_format::EXT_ACL_PATH:
- str = request->urlpath.c_str();
+ str = request->urlpath.buf();
break;
case _external_acl_format::EXT_ACL_METHOD:
case _external_acl_format::EXT_ACL_HEADER:
sb = request->header.getByName(format->header);
- str = sb.c_str();
+ str = sb.buf();
break;
case _external_acl_format::EXT_ACL_HEADER_ID:
sb = request->header.getStrOrList(format->header_id);
- str = sb.c_str();
+ str = sb.buf();
break;
case _external_acl_format::EXT_ACL_HEADER_MEMBER:
sb = request->header.getByNameListMember(format->header, format->member, format->separator);
- str = sb.c_str();
+ str = sb.buf();
break;
case _external_acl_format::EXT_ACL_HEADER_ID_MEMBER:
sb = request->header.getListMember(format->header_id, format->member, format->separator);
- str = sb.c_str();
+ str = sb.buf();
break;
#if USE_SSL
#endif
case _external_acl_format::EXT_ACL_EXT_USER:
- str = request->extacl_user.c_str();
+ str = request->extacl_user.buf();
break;
case _external_acl_format::EXT_ACL_UNKNOWN:
strwordquote(&mb, str);
}
- sb.clear();
+ sb.clean();
first = 0;
}
if (entry != NULL) {
debugs(82, 4, "externalAclLookup: entry = { date=" <<
(long unsigned int) entry->date << ", result=" <<
- entry->result << ", user=" << entry->user << " tag=" <<
- entry->tag << " log=" << entry->log << " }");
+ entry->result << ", user=" << entry->user.buf() << " tag=" <<
+ entry->tag.buf() << " log=" << entry->log.buf() << " }");
}
for (p = Config.externalAclHelperList; p; p = p->next) {
if (!p->cache)
- p->cache = hash_create((HASHCMP *) std::strcmp, hashPrime(1024), hash4);
+ p->cache = hash_create((HASHCMP *) strcmp, hashPrime(1024), hash4);
if (!p->theHelper)
p->theHelper = helperCreate(p->name);
/*
- * $Id: fqdncache.cc,v 1.172 2007/05/18 06:41:24 amosjeffries Exp $
+ * $Id: fqdncache.cc,v 1.173 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 35 FQDN Cache
* AUTHOR: Harvest Derived
n = hashPrime(fqdncache_high / 4);
- fqdn_table = hash_create((HASHCMP *) std::strcmp, n, hash4);
+ fqdn_table = hash_create((HASHCMP *) strcmp, n, hash4);
memDataInit(MEM_FQDNCACHE_ENTRY, "fqdncache_entry",
sizeof(fqdncache_entry), 0);
virtual void create();
virtual void dump(StoreEntry &)const;
~CossSwapDir();
- virtual StoreSearch *search(string const url, HttpRequest *);
+ virtual StoreSearch *search(String const url, HttpRequest *);
virtual void unlink (StoreEntry &);
virtual void statfs (StoreEntry &)const;
virtual int canStore(StoreEntry const &)const;
/*
- * $Id: store_dir_coss.cc,v 1.74 2007/05/18 06:41:32 amosjeffries Exp $
+ * $Id: store_dir_coss.cc,v 1.75 2007/05/29 13:31:47 amosjeffries Exp $
* vim: set et :
*
* DEBUG: section 47 Store COSS Directory Routines
}
StoreSearch *
-CossSwapDir::search(string const url, HttpRequest *)
+CossSwapDir::search(String const url, HttpRequest *)
{
if (url.size())
fatal ("Cannot search by url yet\n");
CossSwapDir::stripePath() const
{
if (!stripe_path) {
- string result = path;
+ String result = path;
result.append("/stripe");
- const_cast<CossSwapDir *>(this)->stripe_path = xstrdup(result.c_str());
+ const_cast<CossSwapDir *>(this)->stripe_path = xstrdup(result.buf());
}
return stripe_path;
/*
- * $Id: store_null.cc,v 1.13 2007/05/18 06:41:32 amosjeffries Exp $
+ * $Id: store_null.cc,v 1.14 2007/05/29 13:31:47 amosjeffries Exp $
*
* DEBUG: section 47 Store Directory Routines
* AUTHOR: Duane Wessels
}
StoreSearch *
-NullSwapDir::search(string const url, HttpRequest *)
+NullSwapDir::search(String const url, HttpRequest *)
{
if (url.size())
fatal ("Cannot search by url yet\n");
/*
- * $Id: store_null.h,v 1.4 2007/05/18 06:41:33 amosjeffries Exp $
+ * $Id: store_null.h,v 1.5 2007/05/29 13:31:47 amosjeffries Exp $
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
* ----------------------------------------------------------
virtual StoreIOState::Pointer openStoreIO(StoreEntry &, StoreIOState::STFNCB *, StoreIOState::STIOCB *, void *);
virtual void parse(int, char*);
virtual void reconfigure (int, char *);
- virtual StoreSearch *search(string const url, HttpRequest *);
+ virtual StoreSearch *search(String const url, HttpRequest *);
};
class StoreSearchNull : public StoreSearch
/*
- * $Id: store_dir_ufs.cc,v 1.83 2007/05/18 06:41:33 amosjeffries Exp $
+ * $Id: store_dir_ufs.cc,v 1.84 2007/05/29 13:31:47 amosjeffries Exp $
*
* DEBUG: section 47 Store Directory Routines
* AUTHOR: Duane Wessels
}
StoreSearch *
-UFSSwapDir::search(string const url, HttpRequest *request)
+UFSSwapDir::search(String const url, HttpRequest *request)
{
if (url.size())
fatal ("Cannot search by url yet\n");
/*
- * $Id: ufscommon.h,v 1.10 2007/05/18 06:41:33 amosjeffries Exp $
+ * $Id: ufscommon.h,v 1.11 2007/05/29 13:31:47 amosjeffries Exp $
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
* ----------------------------------------------------------
virtual void create();
virtual void dump(StoreEntry &) const;
~UFSSwapDir();
- virtual StoreSearch *search(string const url, HttpRequest *);
+ virtual StoreSearch *search(String const url, HttpRequest *);
virtual bool doubleCheck(StoreEntry &);
virtual void unlink(StoreEntry &);
virtual void statfs(StoreEntry &)const;
/*
- * $Id: ftp.cc,v 1.423 2007/05/18 06:41:24 amosjeffries Exp $
+ * $Id: ftp.cc,v 1.424 2007/05/29 13:31:39 amosjeffries Exp $
*
* DEBUG: section 9 File Transfer Protocol (FTP)
* AUTHOR: Harvest Derived
int password_url;
char *reply_hdr;
int reply_hdr_state;
- string clean_url;
- string title_url;
- string base_href;
+ String clean_url;
+ String title_url;
+ String base_href;
int conn_att;
int login_att;
ftp_state_t state;
safe_free(old_filepath);
- title_url.clear();
+ title_url.clean();
- base_href.clear();
+ base_href.clean();
safe_free(filepath);
wordlist *w;
char *dirup;
int i, j, k;
- const char *title = title_url.c_str();
+ const char *title = title_url.buf();
flags.listing_started = true;
printfReplyBody("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 4.01 Transitional//EN\" \"http://www.w3.org/TR/html4/loose.dtd\">\n");
printfReplyBody("<!-- HTML listing generated by Squid %s -->\n",
if (flags.need_base_href)
printfReplyBody("<BASE HREF=\"%s\">\n",
- html_quote(base_href.c_str()));
+ html_quote(base_href.buf()));
printfReplyBody("</HEAD><BODY>\n");
if (flags.dir_slash) {
url = xstrdup("./");
} else {
- const char *title = title_url.c_str();
+ const char *title = title_url.buf();
int k = 6 + strcspn(&title[6], "/");
char *t;
url = xstrdup(title + k);
flags.isdir = 1;
flags.root_dir = 1;
flags.need_base_href = 1; /* Work around broken browsers */
- } else if (!request->urlpath.compare("/%2f/")) {
+ } else if (!request->urlpath.cmp("/%2f/")) {
/* UNIX root directory */
flags.isdir = 1;
flags.root_dir = 1;
- } else if ((l >= 1) && (*(request->urlpath.c_str() + l - 1) == '/')) {
+ } else if ((l >= 1) && (*(request->urlpath.buf() + l - 1) == '/')) {
/* Directory URL, ending in / */
flags.isdir = 1;
checkUrlpath();
buildTitleUrl();
debugs(9, 5, "ftpStart: host=" << request->host << ", path=" <<
- request->urlpath << ", user=" << user << ", passwd=" <<
+ request->urlpath.buf() << ", user=" << user << ", passwd=" <<
password);
state = BEGIN;
mode = 'A';
} else {
t = ftpState->request->urlpath.rpos('/');
- filename = t ? t + 1 : ftpState->request->urlpath.c_str();
+ filename = t ? t + 1 : ftpState->request->urlpath.buf();
mode = mimeGetTransferMode(filename);
}
debugs(9, 3, "This is ftpReadType");
if (code == 200) {
- p = path = xstrdup(ftpState->request->urlpath.c_str());
+ p = path = xstrdup(ftpState->request->urlpath.buf());
if (*p == '/')
p++;
if (ftpState->size == 0) {
debugs(9, 2, "ftpReadSize: SIZE reported " <<
ftpState->ctrl.last_reply << " on " <<
- ftpState->title_url);
+ ftpState->title_url.buf());
ftpState->size = -1;
}
safe_free(ftpState->filepath);
/* Build the new path (urlpath begins with /) */
- path = xstrdup(ftpState->request->urlpath.c_str());
+ path = xstrdup(ftpState->request->urlpath.buf());
rfc1738_unescape(path);
if (!ftpState->flags.isdir && /* Not a directory */
!ftpState->flags.try_slash_hack && /* Not in slash hack */
ftpState->mdtm <= 0 && ftpState->size < 0 && /* Not known as a file */
- strncasecmp(ftpState->request->urlpath, "/%2f", 4) != 0) { /* No slash encoded */
+ ftpState->request->urlpath.caseCmp("/%2f", 4) != 0) { /* No slash encoded */
switch (ftpState->state) {
{
const char *mime_type = NULL;
const char *mime_enc = NULL;
- string urlpath = request->urlpath;
+ String urlpath = request->urlpath;
const char *filename = NULL;
const char *t = NULL;
StoreEntry *e = entry;
e->buffer(); /* released when done processing current data payload */
- filename = (t = urlpath.rpos('/')) ? t + 1 : urlpath.c_str();
+ filename = (t = urlpath.rpos('/')) ? t + 1 : urlpath.buf();
if (flags.isdir) {
mime_type = "text/html";
request->host,
portbuf,
"/%2f",
- request->urlpath.c_str());
+ request->urlpath.buf());
if ((t = strchr(buf, '?')))
*t = '\0';
/*
- * $Id: gopher.cc,v 1.208 2007/05/23 21:10:06 hno Exp $
+ * $Id: gopher.cc,v 1.209 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 10 Gopher
* AUTHOR: Harvest Derived
static void
gopher_request_parse(const HttpRequest * req, char *type_id, char *request)
{
- const char *path = req->urlpath.c_str();
+ const char *path = req->urlpath.buf();
if (request)
request[0] = '\0';
}
inbuf[len] = '\0';
- string outbuf;
+ String outbuf;
if (!gopherState->HTML_header_added) {
if (gopherState->conversion == gopher_ds::HTML_CSO_RESULT)
} /* while loop */
if (outbuf.size() > 0) {
- entry->append(outbuf.c_str(), outbuf.size());
+ entry->append(outbuf.buf(), outbuf.size());
/* now let start sending stuff to client */
entry->flush();
}
- outbuf.clear();
+ outbuf.clean();
return;
}
/*
- * $Id: http.cc,v 1.522 2007/05/18 06:41:24 amosjeffries Exp $
+ * $Id: http.cc,v 1.523 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 11 Hypertext Transfer Protocol (HTTP)
* AUTHOR: Harvest Derived
static PF httpStateFree;
static PF httpTimeout;
static void httpMaybeRemovePublic(StoreEntry *, http_status);
-static void copyOneHeaderFromClientsideRequestToUpstreamRequest(const HttpHeaderEntry *e, string strConnection, HttpRequest * request, HttpRequest * orig_request,
+static void copyOneHeaderFromClientsideRequestToUpstreamRequest(const HttpHeaderEntry *e, String strConnection, HttpRequest * request, HttpRequest * orig_request,
HttpHeader * hdr_out, int we_do_ranges, http_state_flags);
#if ICAP_CLIENT
static void icapAclCheckDoneWrapper(ICAPServiceRep::Pointer service, void *data);
const char *url;
if (_peer->options.originserver)
- url = orig_request->urlpath.c_str();
+ url = orig_request->urlpath.buf();
else
url = entry->url();
/* Pragma: no-cache in _replies_ is not documented in HTTP,
* but servers like "Active Imaging Webcast/2.0" sure do use it */
if (hdr->has(HDR_PRAGMA)) {
- string s = hdr->getList(HDR_PRAGMA);
+ String s = hdr->getList(HDR_PRAGMA);
const int no_cache = strListIsMember(&s, "no-cache", ',');
- s.clear();
+ s.clean();
if (no_cache) {
if (!REFRESH_OVERRIDE(ignore_no_cache))
const char *
httpMakeVaryMark(HttpRequest * request, HttpReply const * reply)
{
- string vary, hdr;
+ String vary, hdr;
const char *pos = NULL;
const char *item;
const char *value;
int ilen;
- static string vstr;
+ static String vstr;
- vstr.clear();
+ vstr.clean();
vary = reply->header.getList(HDR_VARY);
while (strListGetItem(&vary, ',', &item, &ilen, &pos)) {
if (strcmp(name, "*") == 0) {
/* Can not handle "Vary: *" withtout ETag support */
safe_free(name);
- vstr.clear();
+ vstr.clean();
break;
}
strListAdd(&vstr, name, ',');
hdr = request->header.getByName(name);
safe_free(name);
- value = hdr.c_str();
+ value = hdr.buf();
if (value) {
value = rfc1738_escape_part(value);
vstr.append("\"", 1);
}
- hdr.clear();
+ hdr.clean();
}
- vary.clear();
+ vary.clean();
#if X_ACCELERATOR_VARY
pos = NULL;
strListAdd(&vstr, name, ',');
hdr = request->header.getByName(name);
safe_free(name);
- value = hdr.c_str();
+ value = hdr.buf();
if (value) {
value = rfc1738_escape_part(value);
vstr.append("\"", 1);
}
- hdr.clear();
+ hdr.clean();
}
- vary.clear();
+ vary.clean();
#endif
- debugs(11, 3, "httpMakeVaryMark: " << vstr);
- return vstr.c_str();
+ debugs(11, 3, "httpMakeVaryMark: " << vstr.buf());
+ return vstr.buf();
}
void
LOCAL_ARRAY(char, bbuf, BBUF_SZ);
const HttpHeader *hdr_in = &orig_request->header;
const HttpHeaderEntry *e;
- string strFwd;
+ String strFwd;
HttpHeaderPos pos = HttpHeaderInitPos;
assert (hdr_out->owner == hoRequest);
/* append our IMS header */
bool we_do_ranges = decideIfWeDoRanges (orig_request);
- string strConnection (hdr_in->getList(HDR_CONNECTION));
+ String strConnection (hdr_in->getList(HDR_CONNECTION));
while ((e = hdr_in->getEntry(&pos)))
copyOneHeaderFromClientsideRequestToUpstreamRequest(e, strConnection, request, orig_request, hdr_out, we_do_ranges, flags);
/* append Via */
if (Config.onoff.via) {
- string strVia;
+ String strVia;
strVia = hdr_in->getList(HDR_VIA);
snprintf(bbuf, BBUF_SZ, "%d.%d %s",
orig_request->http_ver.major,
orig_request->http_ver.minor, ThisCache);
strListAdd(&strVia, bbuf, ',');
- hdr_out->putStr(HDR_VIA, strVia.c_str());
- strVia.clear();
+ hdr_out->putStr(HDR_VIA, strVia.buf());
+ strVia.clean();
}
#if ESI
{
/* Append Surrogate-Capabilities */
- string strSurrogate (hdr_in->getList(HDR_SURROGATE_CAPABILITY));
+ String strSurrogate (hdr_in->getList(HDR_SURROGATE_CAPABILITY));
snprintf(bbuf, BBUF_SZ, "%s=\"Surrogate/1.0 ESI/1.0\"",
Config.Accel.surrogate_id);
strListAdd(&strSurrogate, bbuf, ',');
- hdr_out->putStr(HDR_SURROGATE_CAPABILITY, strSurrogate.c_str());
+ hdr_out->putStr(HDR_SURROGATE_CAPABILITY, strSurrogate.buf());
}
#endif
else
strListAdd(&strFwd, "unknown", ',');
- hdr_out->putStr(HDR_X_FORWARDED_FOR, strFwd.c_str());
+ hdr_out->putStr(HDR_X_FORWARDED_FOR, strFwd.buf());
- strFwd.clear();
+ strFwd.clean();
/* append Host if not there already */
if (!hdr_out->has(HDR_HOST)) {
if (orig_request->auth_user_request)
username = orig_request->auth_user_request->username();
else if (orig_request->extacl_user.size())
- username = orig_request->extacl_user.c_str();
+ username = orig_request->extacl_user.buf();
snprintf(loginbuf, sizeof(loginbuf), "%s%s", username, orig_request->peer_login + 1);
} else if (strcmp(orig_request->peer_login, "PASS") == 0) {
if (orig_request->extacl_user.size() && orig_request->extacl_passwd.size()) {
char loginbuf[256];
- snprintf(loginbuf, sizeof(loginbuf), "%s:%s", orig_request->extacl_user.c_str(), orig_request->extacl_passwd.c_str());
+ snprintf(loginbuf, sizeof(loginbuf), "%s:%s", orig_request->extacl_user.buf(), orig_request->extacl_passwd.buf());
httpHeaderPutStrf(hdr_out, HDR_PROXY_AUTHORIZATION, "Basic %s",
base64_encode(loginbuf));
}
hdr_out->putStr(HDR_AUTHORIZATION, auth);
} else if (orig_request->extacl_user.size() && orig_request->extacl_passwd.size()) {
char loginbuf[256];
- snprintf(loginbuf, sizeof(loginbuf), "%s:%s", orig_request->extacl_user.c_str(), orig_request->extacl_passwd.c_str());
+ snprintf(loginbuf, sizeof(loginbuf), "%s:%s", orig_request->extacl_user.buf(), orig_request->extacl_passwd.buf());
httpHeaderPutStrf(hdr_out, HDR_AUTHORIZATION, "Basic %s",
base64_encode(loginbuf));
}
if (orig_request->auth_user_request)
username = orig_request->auth_user_request->username();
else if (orig_request->extacl_user.size())
- username = orig_request->extacl_user.c_str();
+ username = orig_request->extacl_user.buf();
snprintf(loginbuf, sizeof(loginbuf), "%s%s", username, orig_request->peer_login + 1);
httpHdrCcSetMaxAge(cc, getMaxAge(url));
if (request->urlpath.size())
- assert(strstr(url, request->urlpath.c_str()));
+ assert(strstr(url, request->urlpath.buf()));
}
/* Set no-cache if determined needed but not found */
if (Config2.onoff.mangle_request_headers)
httpHdrMangleList(hdr_out, request, ROR_REQUEST);
- strConnection.clear();
+ strConnection.clean();
}
void
-copyOneHeaderFromClientsideRequestToUpstreamRequest(const HttpHeaderEntry *e, string strConnection, HttpRequest * request, HttpRequest * orig_request, HttpHeader * hdr_out, int we_do_ranges, http_state_flags flags)
+copyOneHeaderFromClientsideRequestToUpstreamRequest(const HttpHeaderEntry *e, String strConnection, HttpRequest * request, HttpRequest * orig_request, HttpHeader * hdr_out, int we_do_ranges, http_state_flags flags)
{
- debugs(11, 5, "httpBuildRequestHeader: " << e->name << ": " << e->value);
+ debugs(11, 5, "httpBuildRequestHeader: " << e->name.buf() << ": " << e->value.buf());
if (!httpRequestHdrAllowed(e, &strConnection)) {
- debugs(11, 2, "'" << e->name << "' header denied by anonymize_headers configuration");
+ debugs(11, 2, "'" << e->name.buf() << "' header denied by anonymize_headers configuration");
return;
}
HttpVersion httpver(1, 0);
mb->Printf("%s %s HTTP/%d.%d\r\n",
RequestMethodStr[request->method],
- request->urlpath.size() ? request->urlpath.c_str() : "/",
+ request->urlpath.size() ? request->urlpath.buf() : "/",
httpver.major,httpver.minor);
/* build and pack headers */
{
/*
- * $Id: ident.cc,v 1.76 2007/05/18 06:41:24 amosjeffries Exp $
+ * $Id: ident.cc,v 1.77 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 30 Ident (RFC 931)
* AUTHOR: Duane Wessels
void
identInit(void)
{
- ident_hash = hash_create((HASHCMP *) std::strcmp,
+ ident_hash = hash_create((HASHCMP *) strcmp,
hashPrime(Squid_MaxFD / 8),
hash4);
}
/*
- * $Id: internal.cc,v 1.46 2007/05/18 06:41:24 amosjeffries Exp $
+ * $Id: internal.cc,v 1.47 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 76 Internal Squid Object handling
* AUTHOR: Duane, Alex, Henrik
internalStart(HttpRequest * request, StoreEntry * entry)
{
ErrorState *err;
- const char *upath = request->urlpath.c_str();
+ const char *upath = request->urlpath.buf();
debugs(76, 3, "internalStart: " << inet_ntoa(request->client_addr) << " requesting '" << upath << "'");
if (0 == strcmp(upath, "/squid-internal-dynamic/netdb")) {
/*
- * $Id: ipcache.cc,v 1.260 2007/05/18 06:41:24 amosjeffries Exp $
+ * $Id: ipcache.cc,v 1.261 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 14 IP Cache
* AUTHOR: Harvest Derived
ipcache_low = (long) (((float) Config.ipcache.size *
(float) Config.ipcache.low) / (float) 100);
n = hashPrime(ipcache_high / 4);
- ip_table = hash_create((HASHCMP *) std::strcmp, n, hash4);
+ ip_table = hash_create((HASHCMP *) strcmp, n, hash4);
memDataInit(MEM_IPCACHE_ENTRY, "ipcache_entry", sizeof(ipcache_entry), 0);
}
/*
- * $Id: net_db.cc,v 1.195 2007/05/18 06:41:25 amosjeffries Exp $
+ * $Id: net_db.cc,v 1.196 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 38 Network Measurement Database
* AUTHOR: Duane Wessels
n = hashPrime(Config.Netdb.high / 4);
- addr_table = hash_create((HASHCMP *) std::strcmp, n, hash_string);
+ addr_table = hash_create((HASHCMP *) strcmp, n, hash_string);
n = hashPrime(3 * Config.Netdb.high / 4);
- host_table = hash_create((HASHCMP *) std::strcmp, n, hash_string);
+ host_table = hash_create((HASHCMP *) strcmp, n, hash_string);
eventAddIsh("netdbSaveState", netdbSaveState, NULL, 3600.0, 1);
/*
- * $Id: pconn.cc,v 1.52 2007/05/18 06:41:25 amosjeffries Exp $
+ * $Id: pconn.cc,v 1.53 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 48 Persistent Connections
* AUTHOR: Duane Wessels
PconnPool::PconnPool(const char *aDescr) : table(NULL), descr(aDescr)
{
int i;
- table = hash_create((HASHCMP *) std::strcmp, 229, hash_string);
+ table = hash_create((HASHCMP *) strcmp, 229, hash_string);
for (i = 0; i < PCONN_HIST_SZ; i++)
hist[i] = 0;
/*
- * $Id: peer_digest.cc,v 1.124 2007/05/18 06:41:25 amosjeffries Exp $
+ * $Id: peer_digest.cc,v 1.125 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 72 Peer Digest Routines
* AUTHOR: Alex Rousskov
if (pd->cd)
cacheDigestDestroy(pd->cd);
- pd->host.clear();
+ pd->host.clean();
}
CBDATA_CLASS_INIT(PeerDigest);
{
eventAdd("peerDigestCheck", peerDigestCheck, pd, (double) delay, 1);
pd->times.next_check = squid_curtime + delay;
- debugs(72, 3, "peerDigestSetCheck: will check peer " << pd->host << " in " << delay << " secs");
+ debugs(72, 3, "peerDigestSetCheck: will check peer " << pd->host.buf() << " in " << delay << " secs");
}
/*
peerDigestNotePeerGone(PeerDigest * pd)
{
if (pd->flags.requested) {
- debugs(72, 2, "peerDigest: peer " << pd->host << " gone, will destroy after fetch.");
+ debugs(72, 2, "peerDigest: peer " << pd->host.buf() << " gone, will destroy after fetch.");
/* do nothing now, the fetching chain will notice and take action */
} else {
- debugs(72, 2, "peerDigest: peer " << pd->host << " is gone, destroying now.");
+ debugs(72, 2, "peerDigest: peer " << pd->host.buf() << " is gone, destroying now.");
peerDigestDestroy(pd);
}
}
/* per-peer limit */
if (req_time - pd->times.received < PeerDigestReqMinGap) {
- debugs(72, 2, "peerDigestCheck: " << pd->host <<
+ debugs(72, 2, "peerDigestCheck: " << pd->host.buf() <<
", avoiding close peer requests (" <<
(int) (req_time - pd->times.received) << " < " <<
(int) PeerDigestReqMinGap << " secs).");
/* global limit */
if (req_time - pd_last_req_time < GlobDigestReqMinGap) {
- debugs(72, 2, "peerDigestCheck: " << pd->host <<
+ debugs(72, 2, "peerDigestCheck: " << pd->host.buf() <<
", avoiding close requests (" <<
(int) (req_time - pd_last_req_time) << " < " <<
(int) GlobDigestReqMinGap << " secs).");
assert(reply);
assert (reply->sline.status != 0);
status = reply->sline.status;
- debugs(72, 3, "peerDigestFetchReply: " << pd->host << " status: " << status <<
+ debugs(72, 3, "peerDigestFetchReply: " << pd->host.buf() << " status: " << status <<
", expires: " << (long int) reply->expires << " (" << std::showpos <<
(int) (reply->expires - squid_curtime) << ")");
assert (fetch->entry->getReply()->sline.status != 0);
if (fetch->entry->getReply()->sline.status != HTTP_OK) {
- debugs(72, 1, "peerDigestSwapInHeaders: " << fetch->pd->host <<
+ debugs(72, 1, "peerDigestSwapInHeaders: " << fetch->pd->host.buf() <<
" status " << fetch->entry->getReply()->sline.status <<
" got cached!");
#endif
else
- host = pd->host.c_str();
+ host = pd->host.buf();
}
debugs(72, 6, step_name << ": peer " << host << ", offset: " <<
peerDigestFetchStop(DigestFetchState * fetch, char *buf, const char *reason)
{
assert(reason);
- debugs(72, 2, "peerDigestFetchStop: peer " << fetch->pd->host << ", reason: " << reason);
+ debugs(72, 2, "peerDigestFetchStop: peer " << fetch->pd->host.buf() << ", reason: " << reason);
peerDigestReqFinish(fetch, buf, 1, 1, 1, reason, 0);
}
peerDigestFetchAbort(DigestFetchState * fetch, char *buf, const char *reason)
{
assert(reason);
- debugs(72, 2, "peerDigestFetchAbort: peer " << fetch->pd->host << ", reason: " << reason);
+ debugs(72, 2, "peerDigestFetchAbort: peer " << fetch->pd->host.buf() << ", reason: " << reason);
peerDigestReqFinish(fetch, buf, 1, 1, 1, reason, 1);
}
peerDigestPDFinish(DigestFetchState * fetch, int pcb_valid, int err)
{
PeerDigest *pd = fetch->pd;
- const char *host = pd->host.c_str();
+ const char *host = pd->host.buf();
pd->times.received = squid_curtime;
pd->times.req_delay = fetch->resp_time;
{
StoreDigestCBlock cblock;
int freed_size = 0;
- const char *host = pd->host.c_str();
+ const char *host = pd->host.buf();
xmemcpy(&cblock, buf, sizeof(cblock));
/* network -> host conversions */
const int bit_util = cacheDigestBitUtil(pd->cd);
if (bit_util > 65) {
- debugs(72, 0, "Warning: " << pd->host <<
+ debugs(72, 0, "Warning: " << pd->host.buf() <<
" peer digest has too many bits on (" << bit_util << "%%).");
return 0;
assert(pd);
- const char *host = pd->host.c_str();
+ const char *host = pd->host.buf();
storeAppendPrintf(e, "\npeer digest from %s\n", host);
cacheDigestGuessStatsReport(&pd->stats.guess, e, host);
/*
- * $Id: protos.h,v 1.546 2007/05/23 21:10:07 hno Exp $
+ * $Id: protos.h,v 1.547 2007/05/29 13:31:40 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
SQUIDCEXTERN void httpHdrCcInitModule(void);
SQUIDCEXTERN void httpHdrCcCleanModule(void);
SQUIDCEXTERN HttpHdrCc *httpHdrCcCreate(void);
-SQUIDCEXTERN HttpHdrCc *httpHdrCcParseCreate(const string * str);
+SQUIDCEXTERN HttpHdrCc *httpHdrCcParseCreate(const String * str);
SQUIDCEXTERN void httpHdrCcDestroy(HttpHdrCc * cc);
SQUIDCEXTERN HttpHdrCc *httpHdrCcDup(const HttpHdrCc * cc);
SQUIDCEXTERN void httpHdrCcPackInto(const HttpHdrCc * cc, Packer * p);
/* Http Header Tools */
SQUIDCEXTERN HttpHeaderFieldInfo *httpHeaderBuildFieldsInfo(const HttpHeaderFieldAttrs * attrs, int count);
SQUIDCEXTERN void httpHeaderDestroyFieldsInfo(HttpHeaderFieldInfo * info, int count);
-SQUIDCEXTERN http_hdr_type httpHeaderIdByName(const char *name, unsigned int name_len, const HttpHeaderFieldInfo * attrs, int end);
+SQUIDCEXTERN http_hdr_type httpHeaderIdByName(const char *name, int name_len, const HttpHeaderFieldInfo * attrs, int end);
SQUIDCEXTERN http_hdr_type httpHeaderIdByNameDef(const char *name, int name_len);
SQUIDCEXTERN const char *httpHeaderNameById(int id);
SQUIDCEXTERN int httpHeaderHasConnDir(const HttpHeader * hdr, const char *directive);
-SQUIDCEXTERN void strListAdd(string * str, const char *item, char del);
-SQUIDCEXTERN int strListIsMember(const string * str, const char *item, char del);
-SQUIDCEXTERN int strListIsSubstr(const string * list, const char *s, char del);
-SQUIDCEXTERN int strListGetItem(const string * str, char del, const char **item, int *ilen, const char **pos);
+SQUIDCEXTERN void strListAdd(String * str, const char *item, char del);
+SQUIDCEXTERN int strListIsMember(const String * str, const char *item, char del);
+SQUIDCEXTERN int strListIsSubstr(const String * list, const char *s, char del);
+SQUIDCEXTERN int strListGetItem(const String * str, char del, const char **item, int *ilen, const char **pos);
SQUIDCEXTERN const char *getStringPrefix(const char *str, const char *end);
SQUIDCEXTERN int httpHeaderParseInt(const char *start, int *val);
SQUIDCEXTERN int httpHeaderParseSize(const char *start, ssize_t * sz);
/*
- * $Id: redirect.cc,v 1.119 2007/05/18 06:41:25 amosjeffries Exp $
+ * $Id: redirect.cc,v 1.120 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 61 Redirector
* AUTHOR: Duane Wessels
if (http->request->auth_user_request)
r->client_ident = http->request->auth_user_request->username();
- else if (!http->request->extacl_user.empty()) {
- r->client_ident = http->request->extacl_user.c_str();
+ else if (http->request->extacl_user.buf() != NULL) {
+ r->client_ident = http->request->extacl_user.buf();
}
if (!r->client_ident && (conn != NULL && conn->rfc931[0]))
/*
- * $Id: stat.cc,v 1.406 2007/05/18 06:41:25 amosjeffries Exp $
+ * $Id: stat.cc,v 1.407 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 18 Cache Manager Statistics
* AUTHOR: Harvest Derived
if (http->request->auth_user_request)
p = http->request->auth_user_request->username();
- else if (!http->request->extacl_user.empty()) {
- p = http->request->extacl_user.c_str();
+ else if (http->request->extacl_user.buf() != NULL) {
+ p = http->request->extacl_user.buf();
}
if (!p && (conn != NULL && conn->rfc931[0]))
/*
- * $Id: store.cc,v 1.614 2007/05/18 06:41:25 amosjeffries Exp $
+ * $Id: store.cc,v 1.615 2007/05/29 13:31:40 amosjeffries Exp $
*
* DEBUG: section 20 Storage Manager
* AUTHOR: Harvest Derived
if (mem_obj->vary_headers && !storeGetPublic(mem_obj->url, mem_obj->method)) {
/* Create "vary" base object */
- string vary;
+ String vary;
StoreEntry *pe = storeCreateEntry(mem_obj->url, mem_obj->log_url, request->flags, request->method);
HttpVersion version(1, 0);
/* We are allowed to do this typecast */
rep->setHeaders(version, HTTP_OK, "Internal marker object", "x-squid-internal/vary", -1, -1, squid_curtime + 100000);
vary = mem_obj->getReply()->header.getList(HDR_VARY);
- if (!vary.empty()) {
+ if (vary.size()) {
/* Again, we own this structure layout */
- rep->header.putStr(HDR_VARY, vary.c_str());
- vary.clear();
+ rep->header.putStr(HDR_VARY, vary.buf());
+ vary.clean();
}
#if X_ACCELERATOR_VARY
vary = mem_obj->getReply()->header.getList(HDR_X_ACCELERATOR_VARY);
- if (!vary.empty()) {
+ if (vary.buf()) {
/* Again, we own this structure layout */
- rep->header.putStr(HDR_X_ACCELERATOR_VARY, vary.c_str());
- vary.clear();
+ rep->header.putStr(HDR_X_ACCELERATOR_VARY, vary.buf());
+ vary.clean();
}
#endif
/*
- * $Id: store_dir.cc,v 1.160 2007/05/18 06:41:25 amosjeffries Exp $
+ * $Id: store_dir.cc,v 1.161 2007/05/29 13:31:41 amosjeffries Exp $
*
* DEBUG: section 47 Store Directory Routines
* AUTHOR: Duane Wessels
}
StoreSearch *
-StoreController::search(string const url, HttpRequest *request)
+StoreController::search(String const url, HttpRequest *request)
{
/* cheat, for now you can't search the memory hot cache */
return swapDir->search(url, request);
void
StoreController::get
- (string const key, STOREGETCLIENT callback, void *cbdata)
+ (String const key, STOREGETCLIENT callback, void *cbdata)
{
fatal("not implemented");
}
void
StoreHashIndex::get
- (string const key, STOREGETCLIENT callback, void *cbdata)
+ (String const key, STOREGETCLIENT callback, void *cbdata)
{
fatal("not implemented");
}
}
StoreSearch *
-StoreHashIndex::search(string const url, HttpRequest *)
+StoreHashIndex::search(String const url, HttpRequest *)
{
if (url.size())
fatal ("Cannot search by url yet\n");
/*
- * $Id: store_log.cc,v 1.33 2007/05/18 06:41:25 amosjeffries Exp $
+ * $Id: store_log.cc,v 1.34 2007/05/29 13:31:41 amosjeffries Exp $
*
* DEBUG: section 20 Storage Manager Logging Functions
* AUTHOR: Duane Wessels
(int) reply->date,
(int) reply->last_modified,
(int) reply->expires,
- !reply->content_type.empty() ? reply->content_type.c_str() : "unknown",
+ reply->content_type.size() ? reply->content_type.buf() : "unknown",
reply->content_length,
e->contentLen(),
RequestMethodStr[mem->method],
/*
- * $Id: structs.h,v 1.557 2007/05/26 06:38:05 wessels Exp $
+ * $Id: structs.h,v 1.558 2007/05/29 13:31:41 amosjeffries Exp $
*
*
* SQUID Web Proxy Cache http://www.squid-cache.org/
class HttpHdrExtField
{
- string name; /* field-name from HTTP/1.1 (no column after name) */
- string value; /* field-value from HTTP/1.1 */
+ String name; /* field-name from HTTP/1.1 (no column after name) */
+ String value; /* field-value from HTTP/1.1 */
};
/* http cache control header field */
int max_age;
int s_maxage;
int max_stale;
- string other;
+ String other;
};
/* some fields can hold either time or etag specs (e.g. If-Range) */
HttpHeaderFieldInfo() : id (HDR_ACCEPT), type (ftInvalid){}
http_hdr_type id;
- string name;
+ String name;
field_type type;
HttpHeaderFieldStat stat;
};
CapturingStoreEntry() : _buffer_calls(0), _flush_calls(0) {}
- string _appended_text;
+ String _appended_text;
int _buffer_calls;
int _flush_calls;
{}
StoreSearch *
-TestSwapDir::search(string, HttpRequest *)
+TestSwapDir::search(String, HttpRequest *)
{
return NULL;
}
virtual StoreIOState::Pointer createStoreIO(StoreEntry &, StoreIOState::STFNCB *, StoreIOState::STIOCB *, void *);
virtual StoreIOState::Pointer openStoreIO(StoreEntry &, StoreIOState::STFNCB *, StoreIOState::STIOCB *, void *);
virtual void parse(int, char*);
- virtual StoreSearch *search(string, HttpRequest *);
+ virtual StoreSearch *search(String, HttpRequest *);
};
typedef RefCount<TestSwapDir> TestSwapDirPointer;
CacheManager manager;
manager.registerAction("sample", "my sample", &dummy_action, false, false);
CacheManagerAction *anAction = manager.findAction("sample");
- CPPUNIT_ASSERT_EQUAL( (string)"sample", (string)anAction->action );
- CPPUNIT_ASSERT_EQUAL( (string)"my sample", (string)anAction->desc );
+ CPPUNIT_ASSERT_EQUAL(String("sample"), String(anAction->action));
+ CPPUNIT_ASSERT_EQUAL(String("my sample"), String(anAction->desc));
CPPUNIT_ASSERT_EQUAL(&dummy_action, anAction->handler);
CPPUNIT_ASSERT_EQUAL(0, (int)anAction->flags.pw_req);
CPPUNIT_ASSERT_EQUAL(0, (int)anAction->flags.atomic);
scheduler.schedule("test event", CalledEvent::Handler, &event, 0, 0, false);
scheduler.schedule("test event2", CalledEvent::Handler, &event2, 0, 0, false);
scheduler.dump(anEntry);
- string expect = "Last event to run: last event\n"
+ CPPUNIT_ASSERT_EQUAL(String(
+ "Last event to run: last event\n"
"\n"
"Operation\tNext Execution\tWeight\tCallback Valid?\n"
"test event\t0.000000 seconds\t0\tN/A\n"
- "test event2\t0.000000 seconds\t0\tN/A\n";
- CPPUNIT_ASSERT_EQUAL( expect, anEntry->_appended_text);
+ "test event2\t0.000000 seconds\t0\tN/A\n"
+ ), anEntry->_appended_text);
delete anEntry;
}
HttpRequest *nullRequest = NULL;
CPPUNIT_ASSERT_EQUAL(expected_port, aRequest->port);
CPPUNIT_ASSERT_EQUAL(METHOD_GET, aRequest->method);
- CPPUNIT_ASSERT_EQUAL((string)"foo", (string)aRequest->host);
- CPPUNIT_ASSERT_EQUAL((string)"/bar", aRequest->urlpath);
+ CPPUNIT_ASSERT_EQUAL(String("foo"), String(aRequest->host));
+ CPPUNIT_ASSERT_EQUAL(String("/bar"), aRequest->urlpath);
CPPUNIT_ASSERT_EQUAL(PROTO_HTTP, aRequest->protocol);
- CPPUNIT_ASSERT_EQUAL((string)"http://foo:90/bar", (string)url);
+ CPPUNIT_ASSERT_EQUAL(String("http://foo:90/bar"), String(url));
xfree(url);
/* vanilla url, different method */
url = xstrdup("http://foo/bar");
expected_port = 80;
CPPUNIT_ASSERT_EQUAL(expected_port, aRequest->port);
CPPUNIT_ASSERT_EQUAL(METHOD_PUT, aRequest->method);
- CPPUNIT_ASSERT_EQUAL((string)"foo", (string)aRequest->host);
- CPPUNIT_ASSERT_EQUAL((string)"/bar", aRequest->urlpath);
+ CPPUNIT_ASSERT_EQUAL(String("foo"), String(aRequest->host));
+ CPPUNIT_ASSERT_EQUAL(String("/bar"), aRequest->urlpath);
CPPUNIT_ASSERT_EQUAL(PROTO_HTTP, aRequest->protocol);
- CPPUNIT_ASSERT_EQUAL((string)"http://foo/bar", (string)url);
+ CPPUNIT_ASSERT_EQUAL(String("http://foo/bar"), String(url));
/* a connect url with non-CONNECT data */
url = xstrdup(":foo/bar");
aRequest = HttpRequest::CreateFromUrlAndMethod(url, METHOD_CONNECT);
expected_port = 45;
CPPUNIT_ASSERT_EQUAL(expected_port, aRequest->port);
CPPUNIT_ASSERT_EQUAL(METHOD_CONNECT, aRequest->method);
- CPPUNIT_ASSERT_EQUAL((string)"foo", (string)aRequest->host);
- CPPUNIT_ASSERT_EQUAL((string)"", aRequest->urlpath);
+ CPPUNIT_ASSERT_EQUAL(String("foo"), String(aRequest->host));
+ CPPUNIT_ASSERT_EQUAL(String(""), aRequest->urlpath);
CPPUNIT_ASSERT_EQUAL(PROTO_NONE, aRequest->protocol);
- CPPUNIT_ASSERT_EQUAL((string)"foo:45", (string)url);
+ CPPUNIT_ASSERT_EQUAL(String("foo:45"), String(url));
xfree(url);
}
expected_port = 90;
CPPUNIT_ASSERT_EQUAL(expected_port, aRequest->port);
CPPUNIT_ASSERT_EQUAL(METHOD_GET, aRequest->method);
- CPPUNIT_ASSERT_EQUAL((string)"foo", (string)aRequest->host);
- CPPUNIT_ASSERT_EQUAL((string)"/bar", aRequest->urlpath);
+ CPPUNIT_ASSERT_EQUAL(String("foo"), String(aRequest->host));
+ CPPUNIT_ASSERT_EQUAL(String("/bar"), aRequest->urlpath);
CPPUNIT_ASSERT_EQUAL(PROTO_HTTP, aRequest->protocol);
- CPPUNIT_ASSERT_EQUAL((string)"http://foo:90/bar", (string)url);
+ CPPUNIT_ASSERT_EQUAL(String("http://foo:90/bar"), String(url));
xfree(url);
}
void
testHttpRequestMethod::testConst_str()
{
- CPPUNIT_ASSERT_EQUAL((string)"POST", (string)HttpRequestMethod("post").const_str());
+ CPPUNIT_ASSERT_EQUAL(String("POST"), String(HttpRequestMethod("post").const_str()));
}
/*
{
std::ostringstream buffer;
buffer << HttpRequestMethod("get");
- CPPUNIT_ASSERT_EQUAL((string)"GET", (string)buffer.str().c_str() );
+ CPPUNIT_ASSERT_EQUAL(String("GET"), String(buffer.str().c_str()));
}
void
TestStore::get
- (string, void (*)(StoreEntry*, void*), void*)
+ (String, void (*)(StoreEntry*, void*), void*)
{}
void
}
StoreSearch *
-TestStore::search(string const url, HttpRequest *)
+TestStore::search(String const url, HttpRequest *)
{
return NULL;
}
(const cache_key*);
virtual void get
- (string, void (*)(StoreEntry*, void*), void*);
+ (String, void (*)(StoreEntry*, void*), void*);
virtual void init();
virtual void updateSize(size_t size, int sign) {}
- virtual StoreSearch *search(string const url, HttpRequest *);
+ virtual StoreSearch *search(String const url, HttpRequest *);
};
typedef RefCount<TestStore> TestStorePointer;
static StoreEntry *
addedEntry(StorePointer hashStore,
StorePointer aStore,
- string name,
- string varySpec,
- string varyKey
+ String name,
+ String varySpec,
+ String varyKey
+
)
{
StoreEntry *e = new StoreEntry();
EBIT_CLR(e->flags, KEY_PRIVATE);
e->ping_status = PING_NONE;
EBIT_CLR(e->flags, ENTRY_VALIDATED);
- e->hashInsert((const cache_key *)name.c_str()); /* do it after we clear KEY_PRIVATE */
+ e->hashInsert((const cache_key *)name.buf()); /* do it after we clear KEY_PRIVATE */
return e;
}
stream.flush();
CPPUNIT_ASSERT_EQUAL(1, anEntry->_buffer_calls);
CPPUNIT_ASSERT_EQUAL(1, anEntry->_flush_calls);
- CPPUNIT_ASSERT_EQUAL((string)"some text !", (string)anEntry->_appended_text);
+ CPPUNIT_ASSERT_EQUAL(String("some text !"), anEntry->_appended_text);
}
delete anEntry;
StoreEntry *
addedEntry(StorePointer hashStore,
StorePointer aStore,
- string name,
- string varySpec,
- string varyKey
+ String name,
+ String varySpec,
+ String varyKey
)
{
EBIT_CLR(e->flags, KEY_PRIVATE);
e->ping_status = PING_NONE;
EBIT_CLR(e->flags, ENTRY_VALIDATED);
- e->hashInsert((const cache_key *)name.c_str()); /* do it after we clear KEY_PRIVATE */
+ e->hashInsert((const cache_key *)name.buf()); /* do it after we clear KEY_PRIVATE */
return e;
}
Mem::Init();
}
-void
-testString::testDefaults()
-{
- string aStr;
-
- /* check this reports as empty */
- CPPUNIT_ASSERT( aStr.empty() );
- CPPUNIT_ASSERT_EQUAL( (const char*)NULL, aStr.c_str() );
- CPPUNIT_ASSERT_EQUAL( 0, aStr.size() );
-
- string bStr("foo bar");
-
- /* check copy constructor */
- CPPUNIT_ASSERT( !bStr.empty() );
- CPPUNIT_ASSERT_EQUAL( 7, bStr.size() );
- CPPUNIT_ASSERT( NULL != bStr.c_str() );
- CPPUNIT_ASSERT( memcmp(bStr.c_str(), "foo bar", 8) == 0 );
-}
-
-void
-testString::testBooleans()
-{
- const string smStr("bar");
- const string bgStr("foo");
- const string eqStr("foo");
- const string nqStr("food");
-
- /* mathematical boolean operators */
- CPPUNIT_ASSERT(!(bgStr == smStr ));
- CPPUNIT_ASSERT( bgStr != smStr );
- CPPUNIT_ASSERT( bgStr > smStr );
- CPPUNIT_ASSERT(!(bgStr < smStr ));
- CPPUNIT_ASSERT( bgStr >= smStr );
- CPPUNIT_ASSERT(!(bgStr <= smStr ));
-
- /* reverse order to catch corners */
- CPPUNIT_ASSERT(!(smStr == bgStr ));
- CPPUNIT_ASSERT( smStr != bgStr );
- CPPUNIT_ASSERT(!(smStr > bgStr ));
- CPPUNIT_ASSERT( smStr < bgStr );
- CPPUNIT_ASSERT(!(smStr >= bgStr ));
- CPPUNIT_ASSERT( smStr <= bgStr );
-
- /* check identical to catch corners */
- CPPUNIT_ASSERT( bgStr == eqStr );
- CPPUNIT_ASSERT(!(bgStr != eqStr ));
- CPPUNIT_ASSERT(!(bgStr > eqStr ));
- CPPUNIT_ASSERT(!(bgStr < eqStr ));
- CPPUNIT_ASSERT( bgStr >= eqStr );
- CPPUNIT_ASSERT( bgStr <= eqStr );
-
- /* check _almost_ identical to catch corners */
- CPPUNIT_ASSERT(!(bgStr == nqStr ));
- CPPUNIT_ASSERT( bgStr != nqStr );
- CPPUNIT_ASSERT(!(bgStr > nqStr ));
- CPPUNIT_ASSERT( bgStr < nqStr );
- CPPUNIT_ASSERT(!(bgStr >= nqStr ));
- CPPUNIT_ASSERT( bgStr <= nqStr );
-}
-
-void
-testString::testAppend()
-{
- // FIXME: make tests for this.
- string aStr("hello");
-
- aStr.append(" world");
- CPPUNIT_ASSERT_EQUAL( (string)"hello world", aStr );
- aStr.append(" howsit", 7);
- CPPUNIT_ASSERT_EQUAL( (string)"hello world howsit", aStr );
-
- string bStr;
- string cStr("hello");
-
- /* corner cases */
- bStr.append(NULL, 2);
- CPPUNIT_ASSERT( bStr.empty() );
- CPPUNIT_ASSERT_EQUAL( 0, bStr.size() );
- CPPUNIT_ASSERT_EQUAL( (string)"", bStr );
-
- bStr.append("hello", 5);
- CPPUNIT_ASSERT( !bStr.empty() );
- CPPUNIT_ASSERT_EQUAL( 5, bStr.size() );
- CPPUNIT_ASSERT_EQUAL( (string)"hello", bStr );
-
- bStr.append(NULL, 2);
- CPPUNIT_ASSERT( !bStr.empty() );
- CPPUNIT_ASSERT_EQUAL( 5, bStr.size() );
- CPPUNIT_ASSERT_EQUAL( (string)"hello", bStr );
-
- bStr.append(" world untroubled by things such as null termination", 6);
- CPPUNIT_ASSERT( !bStr.empty() );
- CPPUNIT_ASSERT_EQUAL( 11, bStr.size() );
- CPPUNIT_ASSERT_EQUAL( (string)"hello world", bStr );
-
- cStr.append(" wo");
- CPPUNIT_ASSERT( !cStr.empty() );
- CPPUNIT_ASSERT_EQUAL( 8, cStr.size() );
- CPPUNIT_ASSERT_EQUAL( (string)"hello wo", cStr );
-
- cStr.append("rld\0 untroubled by things such as null termination", 10);
- CPPUNIT_ASSERT( !cStr.empty() );
- CPPUNIT_ASSERT_EQUAL( 18, cStr.size() );
- CPPUNIT_ASSERT( memcmp("hello world", cStr.c_str(), 11) == 0 );
- CPPUNIT_ASSERT( memcmp("hello world\0", cStr.c_str(), 12) == 0 );
- CPPUNIT_ASSERT( memcmp("hello world\0 untro", cStr.c_str(), 18) == 0 );
- CPPUNIT_ASSERT( memcmp("hello world\0 untro\0", cStr.c_str(), 19) == 0 );
-}
-
-void
-testString::testAccess()
-{
- string test;
- test = "123456789a"; // to get a predictable length buffer.
-
- CPPUNIT_ASSERT_EQUAL( test.size(), 10 );
-
-/* FIXME: flow checks do not seem to catch assert() sent from within code. */
- /* underflow handling test: _should_ fail with core dump. */
- /* this SHOULD be impossible due to unsigned type of parameter. */
-// CPPUNIT_ASSERT_ASSERTION_FAIL( CPPUNIT_ASSERT( test[-1] ) );
-
- /* overflow handling test: _should_ fail with core dump. */
-// CPPUNIT_ASSERT_ASSERTION_FAIL( CPPUNIT_ASSERT( test[test.size()+10] ) );
-
- /* [] access method (read and write) */
- CPPUNIT_ASSERT( test[0] == '1' );
- CPPUNIT_ASSERT( test[9] == 'a' );
- CPPUNIT_ASSERT( test[10] == '\0' );
-
- test.append('T');
- CPPUNIT_ASSERT( test[10] == 'T' );
- CPPUNIT_ASSERT( test[11] == '\0' );
- CPPUNIT_ASSERT_EQUAL((string)"123456789aT", test);
-
- /* Random access inside buffer. */
- test[5] = 't';
- CPPUNIT_ASSERT( test[5] == 't' );
- CPPUNIT_ASSERT( test[11] == '\0' );
- CPPUNIT_ASSERT_EQUAL((string)"12345t789aT", test);
-
- /* border case at last position of string */
- test[9] = 'E';
- CPPUNIT_ASSERT( test[9] == 'E' );
- CPPUNIT_ASSERT( test[11] == '\0' );
- CPPUNIT_ASSERT_EQUAL((string)"12345t789ET", test);
-
- /* border case at EOS position */
- test[11] = 'F';
- CPPUNIT_ASSERT( test[11] == 'F' );
- CPPUNIT_ASSERT( test[12] == '\0' );
- CPPUNIT_ASSERT_EQUAL((string)"12345t789ETF", test);
-}
-
-void
-testString::testAssignments()
-{
- // FIXME: make tests for this.
-}
-
-void
-testString::testCstrMethods()
-{
- // FIXME: make tests for this.
- // strcmp, strncmp, etc....
-}
-
-void
-testString::testSearch()
-{
- // FIXME: make tests for this.
-
-// pos, rpos, find, rfind, etc...
-}
-
void
testString::testCmpDefault()
{
- string left, right;
+ String left, right;
/* two default strings are equal */
- CPPUNIT_ASSERT(!left.compare(right));
- CPPUNIT_ASSERT(!left.compare(NULL));
- CPPUNIT_ASSERT(!left.compare(NULL, 1));
+ CPPUNIT_ASSERT(!left.cmp(right));
+ CPPUNIT_ASSERT(!left.cmp(NULL));
+ CPPUNIT_ASSERT(!left.cmp(NULL, 1));
}
void
testString::testCmpEmptyString()
{
- string left("");
- string right;
+ String left("");
+ String right;
/* an empty string ("") is equal to a default string */
- CPPUNIT_ASSERT(!left.compare(right));
- CPPUNIT_ASSERT(!left.compare(NULL));
- CPPUNIT_ASSERT(!left.compare(NULL, 1));
+ CPPUNIT_ASSERT(!left.cmp(right));
+ CPPUNIT_ASSERT(!left.cmp(NULL));
+ CPPUNIT_ASSERT(!left.cmp(NULL, 1));
/* reverse the order to catch corners */
- CPPUNIT_ASSERT(!right.compare(left));
- CPPUNIT_ASSERT(!right.compare(""));
- CPPUNIT_ASSERT(!right.compare("", 1));
+ CPPUNIT_ASSERT(!right.cmp(left));
+ CPPUNIT_ASSERT(!right.cmp(""));
+ CPPUNIT_ASSERT(!right.cmp("", 1));
}
void
testString::testCmpNotEmptyDefault()
{
- string left("foo");
- string right;
+ String left("foo");
+ String right;
/* empty string sorts before everything */
- CPPUNIT_ASSERT(left.compare(right) > 0);
- CPPUNIT_ASSERT(left.compare(NULL) > 0);
- CPPUNIT_ASSERT(left.compare(NULL, 1) > 0);
+ CPPUNIT_ASSERT(left.cmp(right) > 0);
+ CPPUNIT_ASSERT(left.cmp(NULL) > 0);
+ CPPUNIT_ASSERT(left.cmp(NULL, 1) > 0);
/* reverse for symmetry tests */
- CPPUNIT_ASSERT(right.compare(left) < 0);
- CPPUNIT_ASSERT(right.compare("foo") < 0);
- CPPUNIT_ASSERT(right.compare("foo", 1) < 0);
+ CPPUNIT_ASSERT(right.cmp(left) < 0);
+ CPPUNIT_ASSERT(right.cmp("foo") < 0);
+ CPPUNIT_ASSERT(right.cmp("foo", 1) < 0);
}
class testString : public CPPUNIT_NS::TestFixture
{
CPPUNIT_TEST_SUITE( testString );
- CPPUNIT_TEST( testDefaults );
- /* boolean helper tests */
CPPUNIT_TEST( testCmpDefault );
CPPUNIT_TEST( testCmpEmptyString );
CPPUNIT_TEST( testCmpNotEmptyDefault );
-
- CPPUNIT_TEST( testBooleans );
- CPPUNIT_TEST( testAppend );
- CPPUNIT_TEST( testAssignments );
- CPPUNIT_TEST( testAccess );
- CPPUNIT_TEST( testCstrMethods );
- CPPUNIT_TEST( testSearch );
CPPUNIT_TEST_SUITE_END();
public:
void setUp();
protected:
-
- /* std::string API */
- void testDefaults();
void testCmpDefault();
void testCmpEmptyString();
void testCmpNotEmptyDefault();
- void testBooleans();
- void testAppend();
- void testAssignments();
- void testAccess();
- void testCstrMethods();
- void testSearch();
};
#endif
+
void
testURLScheme::testConst_str()
{
- string lhs("wais");
+ String lhs("wais");
URLScheme wais(PROTO_WAIS);
- string rhs(wais.const_str());
+ String rhs(wais.const_str());
CPPUNIT_ASSERT_EQUAL(lhs, rhs);
}
{
std::ostringstream buffer;
buffer << URLScheme(PROTO_HTTP);
- string http_str("http");
- string from_buf(buffer.str().c_str());
+ String http_str("http");
+ String from_buf(buffer.str().c_str());
CPPUNIT_ASSERT_EQUAL(http_str, from_buf);
}
/*
- * $Id: test_http_range.cc,v 1.2 2007/05/18 06:41:33 amosjeffries Exp $
+ * $Id: test_http_range.cc,v 1.3 2007/05/29 13:31:48 amosjeffries Exp $
*
* DEBUG: section 64 HTTP Range Header
* AUTHOR: Alex Rousskov
return NULL;
}
-extern string httpHeaderGetList(const HttpHeader * hdr, http_hdr_type id)
+extern String httpHeaderGetList(const HttpHeader * hdr, http_hdr_type id)
{
fatal ("dummy function\n");
- return "";
+ return String();
}
SQUIDCEXTERN int httpHeaderHas(const HttpHeader * hdr, http_hdr_type type)
void
testRangeParser(char const *rangestring)
{
- string aString (rangestring);
+ String aString (rangestring);
HttpHdrRange *range = HttpHdrRange::ParseCreate (&aString);
if (!range)
HttpHdrRange *
rangeFromString(char const *rangestring)
{
- string aString (rangestring);
+ String aString (rangestring);
HttpHdrRange *range = HttpHdrRange::ParseCreate (&aString);
if (!range)
/*
- * $Id: url.cc,v 1.161 2007/05/23 21:10:07 hno Exp $
+ * $Id: url.cc,v 1.162 2007/05/29 13:31:41 amosjeffries Exp $
*
* DEBUG: section 23 URL Parsing
* AUTHOR: Duane Wessels
for (t = host; *t; t++)
*t = xtolower(*t);
- if (strpbrk(host, w_space) != NULL) {
+ if (stringHasWhitespace(host)) {
if (URI_WHITESPACE_STRIP == Config.uri_whitespace) {
t = q = host;
}
#endif
- if (strpbrk(urlpath, w_space) != NULL) {
+ if (stringHasWhitespace(urlpath)) {
debugs(23, 2, "urlParse: URI has whitespace: {" << url << "}");
switch (Config.uri_whitespace) {
return request->canonical;
if (request->protocol == PROTO_URN) {
- snprintf(urlbuf, MAX_URL, "urn:%s", request->urlpath.c_str());
+ snprintf(urlbuf, MAX_URL, "urn:%s", request->urlpath.buf());
} else {
switch (request->method) {
*request->login ? "@" : null_string,
request->host,
portbuf,
- request->urlpath.c_str());
+ request->urlpath.buf());
break;
}
return (request->canonical = xstrdup(urlbuf));
}
-int
-stringHasCntl(const char *s)
-{
- unsigned char c;
-
- while ((c = (unsigned char) *s++) != '\0') {
- if (c <= 0x1f)
- return 1;
-
- if (c >= 0x7f && c <= 0x9f)
- return 1;
- }
-
- return 0;
-}
-
char *
urlCanonicalClean(const HttpRequest * request)
{
char *t;
if (request->protocol == PROTO_URN) {
- snprintf(buf, MAX_URL, "urn:%s", request->urlpath.c_str());
+ snprintf(buf, MAX_URL, "urn:%s", request->urlpath.buf());
} else {
switch (request->method) {
loginbuf,
request->host,
portbuf,
- request->urlpath.c_str());
+ request->urlpath.buf());
/*
* strip arguments AFTER a question-mark
*/
/*
- * $Id: urn.cc,v 1.106 2007/05/18 06:41:26 amosjeffries Exp $
+ * $Id: urn.cc,v 1.107 2007/05/29 13:31:41 amosjeffries Exp $
*
* DEBUG: section 52 URN Parsing
* AUTHOR: Kostas Anagnostakis
void *operator new (size_t byteCount);
void operator delete (void *address);
void start (HttpRequest *, StoreEntry *);
- char *getHost (string &urlpath);
+ char *getHost (String &urlpath);
void setUriResFromRequest(HttpRequest *);
bool RequestNeedsMenu(HttpRequest *r);
void updateRequestURL(HttpRequest *r, char const *newPath);
- void createUriResRequest (string &uri);
+ void createUriResRequest (String &uri);
virtual ~UrnState();
}
char *
-UrnState::getHost (string &urlpath)
+UrnState::getHost (String &urlpath)
{
char * result;
char const *t;
if ((t = urlpath.pos(':')) != NULL) {
urlpath.set(t, '\0');
- result = xstrdup(urlpath.c_str());
+ result = xstrdup(urlpath.buf());
urlpath.set(t, ':');
} else {
- result = xstrdup(urlpath.c_str());
+ result = xstrdup(urlpath.buf());
}
return result;
bool
UrnState::RequestNeedsMenu(HttpRequest *r)
{
- return strncasecmp(r->urlpath, "menu.", 5) == 0;
+ return strncasecmp(r->urlpath.buf(), "menu.", 5) == 0;
}
void
}
void
-UrnState::createUriResRequest (string &uri)
+UrnState::createUriResRequest (String &uri)
{
LOCAL_ARRAY(char, local_urlres, 4096);
char *host = getHost (uri);
- snprintf(local_urlres, 4096, "http://%s/uri-res/N2L?urn:%s", host, uri.c_str());
+ snprintf(local_urlres, 4096, "http://%s/uri-res/N2L?urn:%s", host, uri.buf());
safe_free (host);
safe_free (urlres);
urlres = xstrdup (local_urlres);
UrnState::setUriResFromRequest(HttpRequest *r)
{
if (RequestNeedsMenu(r)) {
- updateRequestURL(r, r->urlpath.c_str() + 5);
+ updateRequestURL(r, r->urlpath.buf() + 5);
flags.force_menu = 1;
}
/*
- * $Id: whois.cc,v 1.44 2007/05/18 06:41:26 amosjeffries Exp $
+ * $Id: whois.cc,v 1.45 2007/05/29 13:31:41 amosjeffries Exp $
*
* DEBUG: section 75 WHOIS protocol
* AUTHOR: Duane Wessels, Kostas Anagnostakis
buf = (char *)xmalloc(l);
- snprintf(buf, l, "%s\r\n", p->request->urlpath.c_str() + 1);
+ snprintf(buf, l, "%s\r\n", p->request->urlpath.buf() + 1);
comm_write(fd, buf, strlen(buf), whoisWriteComplete, p, NULL);
comm_read(fd, p->buf, BUFSIZ, whoisReadReply, p);
projects / thirdparty / squid.git / commitdiff
