apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Garrett Rooney <roo...@electricjellyfish.net>
Subject Re: svn commit: r368769 - /apr/apr/trunk/include/apr_file_io.h
Date Fri, 13 Jan 2006 17:42:36 GMT
On 1/13/06, mturk@apache.org <mturk@apache.org> wrote:
> Author: mturk
> Date: Fri Jan 13 08:17:06 2006
> New Revision: 368769
>
> URL: http://svn.apache.org/viewcvs?rev=368769&view=rev
> Log:
> Mark pool param for apr_file_remove and apr_file_rename
> as unused (because it is).
> Perhaps some day it will be removed from the API.

I'm not sure how I feel about this.  The pool could potentially be
used for temporary allocation some day, if we port to a platform that
happens to require it.  Encouraging people to just pass random things
in place of the pool argument seems like a bad idea.

-garrett

Mime
View raw message