On 8 Mar 2002 [EMAIL PROTECTED] wrote:

> stoddard    02/03/07 20:25:30
>
>   Modified:    modules/experimental mod_mem_cache.c
>   Log:
>   Add file handle caching to mod_mem_cache. This is still experimental and
>   is enabled by the #define CACHE_FD, which is off by default.
>
>   @@ -102,6 +102,7 @@
>        apr_size_t refcount;
>        apr_size_t m_len;
>        void *m;
>   +    apr_os_file_t fd;
>    } mem_cache_object_t;
>
>    typedef struct {
>   @@ -163,9 +164,12 @@
>
>        /* Cleanup the mem_cache_object_t */
>        if (mobj) {
>   -        if (mobj->m) {
>   +        if (mobj->type == CACHE_TYPE_HEAP && mobj->m) {
>                free(mobj->m);
>            }
>   +        if (mobj->type == CACHE_TYPE_FILE && mobj->fd) {
>   +            apr_file_close(mobj->fd);
>   +        }
>            if (mobj->header_out) {
>                if (mobj->header_out[0].hdr)
>                    free(mobj->header_out[0].hdr);

mod_mem_cache.c: In function `cleanup_cache_object':
mod_mem_cache.c:171: warning: passing arg 1 of `apr_file_close' makes
pointer from integer without a cast


You can't apr_file_close() an apr_os_file_t.

--Cliff


--------------------------------------------------------------
   Cliff Woolley
   [EMAIL PROTECTED]
   Charlottesville, VA


Reply via email to