apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <wr...@rowe-clan.net>
Subject Re: svn commit: r106663 - /apr/apr/trunk/CHANGES /apr/apr/trunk/include/apr_file_io.h
Date Mon, 29 Nov 2004 15:44:18 GMT
At 03:26 PM 11/26/2004, you wrote:
>Author: stas
>Date: Fri Nov 26 13:26:37 2004
>New Revision: 106663
>
>URL: http://svn.apache.org/viewcvs?view=rev&rev=106663
>Log:
>rename the fopen defines (APR_READ, APR_WRITE, etc.) to have prefix 
>APR_FOPEN_ (keeping the old defines)

Stas, you forgot doxygen @deprecate tags!  For example, with;

>+#define APR_FOPEN_READ       0x00001  /**< Open the file for reading */

we need a matching doxygen entry redirecting the user;

>+#define APR_READ  APR_FOPEN_READ  /**< @deprecated @see APR_FOPEN_READ */

This provides documentation that devs *need* to track the changes.

Bill


Mime
View raw message