Re: external acl helper and url path

Hi Henrik,

> Thanks. Your patch has been applied to Squid-HEAD with a minor change to
> use strStr() rather than copying the urlpath.

Please forgive any misunderstandings that I have, but browsing CVS shows
that external_acl.c now contains the line:

  str = strStr(request->urlpath);

strStr is define in protos.h to be:

  #define strStr(s,str) ((const char*)strstr(strBuf(s), (str)))

When I try to compile external_acl.c I get:

  gcc -DHAVE_CONFIG_H -DDEFAULT_CONFIG_FILE=\"/tmp/squid/etc/squid.conf\"
  -I. -I. -I../include -I. -I. -I../include -I../include -g -Wall -c
  `test -f external_acl.c || echo './'`external_acl.c
  external_acl.c:523: macro `strStr' used with just one arg

I am guessing that there is a mistake here?

Regards

-- 
Matthew