Commit two fixes while I'm considering Jeff's comments.

  apr_getopt_t->cont is not really necessary, but I don't yet see a reason
  to kill it outright.  Might come in handy later.

PR:
Obtained from:
Submitted by:
Reviewed by:


git-svn-id: http://svn.apache.org/repos/asf/apr/apr/trunk@60494 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 2 insertions, 2 deletions

diff --git a/include/apr_getopt.h b/include/apr_getopt.h
index a06980698..ad86ef37e 100644
--- a/include/apr_getopt.h
+++ b/include/apr_getopt.h
@@ -56,7 +56,7 @@
 #define APR_GETOPT_H
 
 typedef struct apr_getopt_t {
-    apr_pool_t cont;         /* context for processing */
+    apr_pool_t *cont;        /* context for processing */
     int err;                 /* if error message should be printed */
     int ind;                 /* index into parent argv vector */
     int opt;                 /* character checked for validity */
@@ -73,7 +73,7 @@ typedef struct apr_getopt_t {
  * @param argc The number of arguments to parse
  * @param argv The array of arguments to parse
  * @tip Arguments 2 and 3 are most commonly argc and argv from main(argc, argv)
- * @deffunc apr_status_t apr_initopt(apr_getopt_t **os, apr_pool_t *cont, int argc, char const* const* argv)
+ * @deffunc apr_status_t apr_initopt(apr_pool_t *cont, apr_getopt_t **os, int argc, char const* const* argv)
  */
 APR_EXPORT(apr_status_t) apr_initopt(apr_getopt_t **os, apr_pool_t *cont,
                                      int argc, char const* const* argv);