Re: squid-3 ?

2007-08-14 Thread Amos Jeffries
> How long before the next pre or release candidate of squid-3 ? > Now that both of the blocker bugs are closed. I submitted the last of the compile errors I can find today, it now builds for both make check and make install. Baring any of you others finding another I think we have reached step 2

squid-3 ?

2007-08-14 Thread Adrian Chadd
How long before the next pre or release candidate of squid-3 ? Adrian

Re: IPv6 support and Windows

2007-08-14 Thread Amos Jeffries
>> Hi Amos, >> >> Not good news here ... :-( >> >> We have a problem: >> You have used getaddrinfo() and freeaddrinfo() functions, but on >> Windows they are available only starting from Windows XP. This means >> that your code, even if compiled without IPv6 support, cannot run on >> Windows 2000.

Re: cvs commit: squid3/src ftp.cc

2007-08-14 Thread Henrik Nordstrom
On ons, 2007-08-15 at 09:28 +1200, Amos Jeffries wrote: > >> > > If you have time, it may be better to replace a virtual > >> > > FtpStateData::haveControlChannel(char*) into a static > >> > > FtpStateData::HaveControlChannel(FtpStateData*, char*) and let it > >> check > >> > > the ftpState pointer

Re: Squid 3 build error using Visual Studio

2007-08-14 Thread Amos Jeffries
> On tis, 2007-08-14 at 14:10 +0300, Tsantilas Christos wrote: > >> The major problem you may have is that c++ compilers some times return >> error messages which looks completely unrelated with the real problem. > > Or quite related but where it's impossible to grok what is related or > how... > >

Re: cvs commit: squid3/src ftp.cc

2007-08-14 Thread Amos Jeffries
> On tis, 2007-08-14 at 23:01 +1200, Amos Jeffries wrote: >> Alex Rousskov wrote: >> > On Mon, 2007-08-13 at 12:01 +0200, Guido Serassio wrote: >> > >> >> Now fixed. >> > >> > If you have time, it may be better to replace a virtual >> > FtpStateData::haveControlChannel(char*) into a static >> > Ftp

Re: cvs commit: squid3/src BodyPipe.cc BodyPipe.h

2007-08-14 Thread Amos Jeffries
> On Tue, 2007-08-14 at 09:54 +1200, Amos Jeffries wrote: >> > rousskov2007/08/13 10:48:20 MDT >> > >> >> > The new code also keeps track >> > of the number of outstanding events and skips that number if the >> consumer >> > leaves. >> > >> > TODO: when AscyncCall support is improved, shoul

Re: IPv6 support and Windows

2007-08-14 Thread Amos Jeffries
> Hi Amos, > > Not good news here ... :-( > > We have a problem: > You have used getaddrinfo() and freeaddrinfo() functions, but on > Windows they are available only starting from Windows XP. This means > that your code, even if compiled without IPv6 support, cannot run on > Windows 2000. > > I'm a

Re: cvs commit: squid3/src BodyPipe.cc BodyPipe.h

2007-08-14 Thread Alex Rousskov
On Tue, 2007-08-14 at 09:54 +1200, Amos Jeffries wrote: > > rousskov2007/08/13 10:48:20 MDT > > > > > The new code also keeps track > > of the number of outstanding events and skips that number if the consumer > > leaves. > > > > TODO: when AscyncCall support is improved, should we just sch

Re: cvs commit: squid3/src ftp.cc

2007-08-14 Thread Alex Rousskov
On Wed, 2007-08-15 at 09:28 +1200, Amos Jeffries wrote: > > On Tue, 2007-08-14 at 17:58 +0200, Henrik Nordstrom wrote: > >> On tis, 2007-08-14 at 23:01 +1200, Amos Jeffries wrote: > >> > Alex Rousskov wrote: > >> > > On Mon, 2007-08-13 at 12:01 +0200, Guido Serassio wrote: > >> > > > >> > >> Now fi

Re: cvs commit: squid3/src ftp.cc

2007-08-14 Thread Amos Jeffries
> On Tue, 2007-08-14 at 17:58 +0200, Henrik Nordstrom wrote: >> On tis, 2007-08-14 at 23:01 +1200, Amos Jeffries wrote: >> > Alex Rousskov wrote: >> > > On Mon, 2007-08-13 at 12:01 +0200, Guido Serassio wrote: >> > > >> > >> Now fixed. >> > > >> > > If you have time, it may be better to replace a v

Re: cvs commit: squid3/src ftp.cc

2007-08-14 Thread Robert Collins
On Tue, 2007-08-14 at 17:58 +0200, Henrik Nordstrom wrote: > On tis, 2007-08-14 at 23:01 +1200, Amos Jeffries wrote: > > Alex Rousskov wrote: > > > On Mon, 2007-08-13 at 12:01 +0200, Guido Serassio wrote: > > > > > >> Now fixed. > > > > > > If you have time, it may be better to replace a virtual

Re: Squid 3 build error using Visual Studio

2007-08-14 Thread Henrik Nordstrom
On tis, 2007-08-14 at 14:10 +0300, Tsantilas Christos wrote: > The major problem you may have is that c++ compilers some times return > error messages which looks completely unrelated with the real problem. Or quite related but where it's impossible to grok what is related or how... http://www.h

Re: IPv6 support and Windows

2007-08-14 Thread Henrik Nordstrom
On tis, 2007-08-14 at 13:41 +0200, Guido Serassio wrote: > I'm not sure if the licensing of the implementation coming from the > PostgreSQL allow its usage in Squid. Looks fine. PostgreSQL is licensed under the modified/modern three-clause BSD license, which is fully GPL compatible. http://www.

Re: cvs commit: squid3/src ftp.cc

2007-08-14 Thread Henrik Nordstrom
On tis, 2007-08-14 at 23:01 +1200, Amos Jeffries wrote: > Alex Rousskov wrote: > > On Mon, 2007-08-13 at 12:01 +0200, Guido Serassio wrote: > > > >> Now fixed. > > > > If you have time, it may be better to replace a virtual > > FtpStateData::haveControlChannel(char*) into a static > > FtpStateDat

Re: StoreIOBuffer field types

2007-08-14 Thread Henrik Nordstrom
On tis, 2007-08-14 at 22:40 +1200, Amos Jeffries wrote: > Or was I right about being buffer length? in which case it should be an > unsigned something? which prevents this warning showing. My understanding is that the length is the in-memory buffer length, and offset is the offset within the obj

Re: Squid 3 build error using Visual Studio

2007-08-14 Thread Alex Rousskov
On Tue, 2007-08-14 at 11:12 +0200, Guido Serassio wrote: > At 18.43 13/08/2007, Alex Rousskov wrote: > >Is it a good idea to hate something you do not understand? :-) > > Alex, let me to explain my situation. > > Principally I'm an IT consultant working (busy at 110% of my time) on > Systems Ad

Re: StoreIOBuffer field types

2007-08-14 Thread Tsantilas Christos
Hi Amos, Amos Jeffries wrote: > One of them was unsafe use of a variable typed 'size_t' setting up > StoreIOBUffer local variable > > The constructor using it is: > > StoreIOBuffer(size_t aLength, int64_t anOffset, char *someData) : > length (aLength), offset (anOffset), data (someD

Re: Squid 3 build error using Visual Studio

2007-08-14 Thread Tsantilas Christos
Hi Guido , Guido Serassio wrote: > I have learned at university (more than 15 years ago ...) Assembler, > Fortran, C and Pascal, but never C++. or any similar language. Sometimes I am feeling to old to start thinks too, but Guido if you are thinking like that now, what are we going to do after 15

Re: cvs commit: squid3/src ftp.cc

2007-08-14 Thread Amos Jeffries
Alex Rousskov wrote: On Mon, 2007-08-13 at 12:01 +0200, Guido Serassio wrote: Now fixed. If you have time, it may be better to replace a virtual FtpStateData::haveControlChannel(char*) into a static FtpStateData::HaveControlChannel(FtpStateData*, char*) and let it check the ftpState pointer i

StoreIOBuffer field types

2007-08-14 Thread Amos Jeffries
As part of the checks to find possible causes of my servers falling over I went to see if coverity had any more issues located. One of them was unsafe use of a variable typed 'size_t' setting up StoreIOBUffer local variable The constructor using it is: StoreIOBuffer(size_t aLength, int64_t an

Re: Squid 3 build error using Visual Studio

2007-08-14 Thread Guido Serassio
Hi Alex, At 18.43 13/08/2007, Alex Rousskov wrote: This is with the code recently synced with HEAD, right? Please try the attached untested patch. It follows the suggestions at the page found by Amos: http://www.codecomments.com/archive292-2005-2-396222.html Yes, it works. Thanks. > Thanks

Re: cvs commit: squid3/src HttpHdrContRange.cc Makefile.in MemObject.cc access_log.cc cache_cf.cc client_side.cc client_side_reply.cc stmem.cc

2007-08-14 Thread Tsantilas Christos
Hi , Henrik Nordstrom wrote: > hno 2007/08/13 12:25:14 MDT > > Modified files: > src HttpHdrContRange.cc Makefile.in > MemObject.cc access_log.cc cache_cf.cc > client_side.cc client_side_reply.cc >