Scrive Arno Garrels <[EMAIL PROTECTED]>:

> Maurizio Lotauro wrote:
>  
> > I added a new internal variable FDelaySetReady (boolean) as flag that
> > the SetReady must be called when the whole SendStream is sent.
> > Some places changed so:
> > 
> > =>            else if ((FStatusCode = 401) or (FStatusCode = 407)) and
> > =>                    FAllowedToSend then
> > =>                FDelaySetReady := TRUE
> >            else
> >                SetReady;
> 
> I think your code works fine. But since the above check is done at 7 places

7 places? I count 6.

> I suggest to call a little procedure instead, what do you think? 

Initially they are only two, then they are proliferated :-)
So yes, I think that it is a good idea.

> Is there a reason I'm not aware of why this check is also required when 
> FRequestType = httpHEAD?

Do you refer to the code marked with "AG 05/27/08"? It is a mistake since
FAllowedToSend is always false on this case.


Bye, Maurizio.

----------------------------------------------------
This mail has been sent using Alpikom webmail system
http://www.alpikom.it

-- 
To unsubscribe or change your settings for TWSocket mailing list
please goto http://lists.elists.org/cgi-bin/mailman/listinfo/twsocket
Visit our website at http://www.overbyte.be

Reply via email to