Grrr... yep this is a bug and the MS compiler doesn't even flag a warning. Now how to
fix
it... There are no clean options...
Bill
> 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
>
>
