You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@httpd.apache.org by Cliff Woolley <jw...@virginia.edu> on 2003/12/10 04:35:10 UTC

Re: cvs commit: httpd-2.0/modules/experimental cache_cache.c mod_mem_cache.c

On Tue, 10 Dec 2003 jwoolley@apache.org wrote:

>   --- mod_mem_cache.c	4 Dec 2003 00:16:13 -0000	1.98
>   +++ mod_mem_cache.c	10 Dec 2003 03:22:32 -0000	1.99
>   @@ -240,7 +240,7 @@
>        cache_object_t *obj = (cache_object_t *)a;
>        mem_cache_object_t *mobj = obj->vobj;
>        if (mobj->priority == 0)
>   -        mobj->priority = -((long)(queue_clock + mobj->total_refs));
>   +        mobj->priority = queue_clock - mobj->total_refs;

The only thing I don't understand is why this went from:

- queue_clock - mobj->total_refs

to:

queue_clock - mobj->total_refs

When the other case negated the whole statement.  Is this intentional or
did you forget to account for the - in this case being distributed across
the ()'s?

Just checking,
Cliff