Philip M. Gollucci wrote:
Philip M. Gollucci wrote:

Philip M. Gollucci wrote:


Cool.  I just posted rc2 in the usual place

   http://people.apache.org/~joes/libapreq2-2.06-dev.rc2.tar.gz
Next up, I'll try to fix some of the 10million warnings during the genration of docs.


With the attached patch, all the remaining doxygen warnings/errors come from externally used projects or "not documented" (I take it we should fix those). For reference a list of these is below the patch. I'll look at documenting them later in the week. This patch(also attached) fixes the following errors/warnings:

/usr/local/dev/repos/ASF/httpd/apreq/trunk/CHANGES:2: Warning: Found unknown command `\brief'

/usr/local/dev/repos/ASF/httpd/apreq/trunk/CHANGES:366: Warning: End of list marker found has invalid indent level

/usr/local/dev/repos/ASF/httpd/apreq/trunk/include/apreq_module.h:421:Warning: argument `key' of command @param is not found in the argument list of apreq_cookie(req, name)

svn diff
Index: CHANGES
===================================================================
--- CHANGES     (revision 216120)
+++ CHANGES     (working copy)
@@ -1,5 +1,5 @@
 /** @page apreq_changes CHANGES
[EMAIL PROTECTED] List of major changes.
+//! brief List of major changes.


 @section v2_06 Changes with libapreq2-2.06
@@ -362,7 +362,7 @@
   apreq_jar_t and added args_status field to apreq_request_t. Parsers
   also must return their public status code when a NULL brigade is passed.
   apreq_hook_disable_uploads() is also added.
-  .
+
This is an ABI change affecting all versions of libapreq2 prior to 2.0.12.

 - Perl API [joes]

Index: /usr/local/dev/repos/ASF/httpd/apreq/trunk/include/apreq_module.h
===================================================================
--- /usr/local/dev/repos/ASF/httpd/apreq/trunk/include/apreq_module.h (revision 216120) +++ /usr/local/dev/repos/ASF/httpd/apreq/trunk/include/apreq_module.h (working copy)
@@ -423,7 +423,7 @@
  * The match is case-insensitive.
  *
  * @param req request handle.
- * @param key desired cookie name
+ * @param name desired cookie name
  *
* @return The first matching parameter (with args searched first) or NULL.
  */


/include/apreq.h:41: Warning: Member APREQ_DECLARE_NONSTD(d) of file apreq.h is not documented. /include/apreq.h:42: Warning: Member APREQ_DECLARE_DATA of file apreq.h is not documented. /include/apreq.h:49: Warning: Member APREQ_DEFAULT_READ_BLOCK_SIZE of file apreq.h is not documented. /include/apreq.h:50: Warning: Member APREQ_DEFAULT_READ_LIMIT of file apreq.h is not documented. /include/apreq.h:51: Warning: Member APREQ_DEFAULT_BRIGADE_LIMIT of file apreq.h is not documented. /include/apreq.h:52: Warning: Member APREQ_DEFAULT_NELTS of file apreq.h is not documented. /include/apreq.h:56: Warning: Member APREQ_FLAGS_OFF(f, name) of file apreq.h is not documented. /include/apreq.h:57: Warning: Member APREQ_FLAGS_ON(f, name) of file apreq.h is not documented. /include/apreq.h:58: Warning: Member APREQ_FLAGS_GET(f, name) of file apreq.h is not documented. /include/apreq.h:61: Warning: Member APREQ_FLAGS_SET(f, name, value) of file apreq.h is not documented. /include/apreq.h:64: Warning: Member APREQ_CHARSET_BIT of file apreq.h is not documented. /include/apreq.h:65: Warning: Member APREQ_CHARSET_MASK of file apreq.h is not documented. /include/apreq.h:67: Warning: Member APREQ_TAINTED_BIT of file apreq.h is not documented. /include/apreq.h:68: Warning: Member APREQ_TAINTED_MASK of file apreq.h is not documented. /include/apreq.h:70: Warning: Member APREQ_COOKIE_VERSION_BIT of file apreq.h is not documented. /include/apreq.h:71: Warning: Member APREQ_COOKIE_VERSION_MASK of file apreq.h is not documented. /include/apreq.h:73: Warning: Member APREQ_COOKIE_SECURE_BIT of file apreq.h is not documented. /include/apreq.h:74: Warning: Member APREQ_COOKIE_SECURE_MASK of file apreq.h is not documented. /include/apreq.h:119: Warning: Member apreq_attr_to_type(T, A, P) of file apreq.h is not documented. /include/apreq.h:115: Warning: Member apreq_value_table_add(const apreq_value_t *v, apr_table_t *t) of file apreq.h is not documented. /include/apreq_error.h:43: Warning: Member APR_EBADARG of file apreq_error.h is not documented. /module/apache2/apreq_module_apache2.h:142: Warning: Member APREQ_FILTER_NAME of group mod_apreq2 is not documented. /module/apache2/apreq_module_apache2.h:143: Warning: Member APREQ_APACHE2_MMN of group mod_apreq2 is not documented

~

--
END
------------------------------------------------------------
    What doesn't kill us can only make us stronger.
                Nothing is impossible.
                                
Philip M. Gollucci ([EMAIL PROTECTED]) 301.254.5198
Consultant / http://p6m7g8.net/Resume/
Senior Developer / Liquidity Services, Inc.
  http://www.liquidityservicesinc.com
       http://www.liquidation.com
       http://www.uksurplus.com
       http://www.govliquidation.com
       http://www.gowholesale.com

/usr/local/dev/repos/ASF/httpd/apreq/trunk/CHANGES:2: Warning: Found unknown 
command `\brief'
/usr/local/dev/repos/ASF/httpd/apreq/trunk/CHANGES:366: Warning: End of list 
marker found has invalid indent level
/usr/local/dev/repos/ASF/httpd/apreq/trunk/include/apreq_module.h:421:Warning: 
argument `key' of command @param is not found in the argument list of 
apreq_cookie(req, name)
/usr/local/dev/src/httpd-2.0.54/srclib/apr/include/apr_poll.h:110: Warning: end 
of comment block while expecting command </pre>

svn diff
Index: CHANGES
===================================================================
--- CHANGES     (revision 216120)
+++ CHANGES     (working copy)
@@ -1,5 +1,5 @@
 /** @page apreq_changes CHANGES
[EMAIL PROTECTED] List of major changes.
+//! brief List of major changes.
 
 
 @section v2_06 Changes with libapreq2-2.06
@@ -362,7 +362,7 @@
   apreq_jar_t and added args_status field to apreq_request_t. Parsers
   also must return their public status code when a NULL brigade is passed.
   apreq_hook_disable_uploads() is also added.
-  .
+
   This is an ABI change affecting all versions of libapreq2 prior to 2.0.12.
 
 - Perl API [joes]

Index: /usr/local/dev/repos/ASF/httpd/apreq/trunk/include/apreq_module.h
===================================================================
--- /usr/local/dev/repos/ASF/httpd/apreq/trunk/include/apreq_module.h   
(revision 216120)
+++ /usr/local/dev/repos/ASF/httpd/apreq/trunk/include/apreq_module.h   
(working copy)
@@ -423,7 +423,7 @@
  * The match is case-insensitive.
  *
  * @param req request handle.
- * @param key desired cookie name
+ * @param name desired cookie name
  *
  * @return The first matching parameter (with args searched first) or NULL.
  */

Reply via email to