Re: refresh.cc - refreshIsCachable()

> The return value of refreshIsCachable() can be calculated without making a
> call to refreshCheck().
>
> I.e. you can remove the call to refreshCheck() from refreshIsCachable(), and
> refreshIsCachable() will still return the correct result.

Sorry, still not following you. refreshIsCachable() uses the 'reason' value
in an early if statement:

     if (reason < 200)
         /* Does not need refresh. This is certainly cachable */
         return 1;

And there are numerous cases where refreshCheck() would return FRESH_*
(i.e. < 200) values.

DW
Received on Thu Apr 27 2006 - 11:41:50 MDT