Changeset 300 for pjproject/trunk

Timestamp:

Mar 6, 2006 3:14:59 PM (19 years ago)

Author:

bennylp

Message:

Renamed getopt() to pj_getopt(), and fixed misc Linux warnings

Location:

pjproject/trunk/pjsip

Files:

• build/Makefile (modified) (1 diff)
• include/pjsua-lib/getopt.h (modified) (7 diffs)
• include/pjsua-lib/pjsua.h (modified) (1 diff)
• src/pjsua-lib/getopt.c (modified) (39 diffs)
• src/pjsua-lib/pjsua_im.c (modified) (1 diff)
• src/pjsua-lib/pjsua_settings.c (modified) (20 diffs)

pjproject/trunk/pjsip/build/Makefile

@@ -71 +71 @@
 export PJSUA_LIB_SRCDIR = ../src/pjsua-lib
 export PJSUA_LIB_OBJS += $(OS_OBJS) $(M_OBJS) $(CC_OBJS) $(HOST_OBJS) \
-                        pjsua_call.o pjsua_core.o pjsua_im.o pjsua_pres.o pjsua_reg.o \
-                        pjsua_settings.o \
+                        getopt.o pjsua_call.o pjsua_core.o pjsua_im.o \
+                        pjsua_pres.o pjsua_reg.o pjsua_settings.o \
                         
 export PJSUA_LIB_CFLAGS += $(_CFLAGS)

pjproject/trunk/pjsip/include/pjsua-lib/getopt.h

@@ -1 +1 @@
 /* $Id$ */
 /* This file has now become GPL. */
-/* Declarations for getopt.
+/* Declarations for pj_getopt.
    Copyright (C) 1989,90,91,92,93,94,96,97,98 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
@@ -20 +20 @@
    Boston, MA 02111-1307, USA.  */
 
-#ifndef _GETOPT_H
-#define _GETOPT_H 1
+#ifndef __PJ_GETOPT_H__
+#define __PJ_GETOPT_H__ 1
 
 #ifdef  __cplusplus
@@ -27 +27 @@
 #endif
 
-/* For communication from `getopt' to the caller.
-   When `getopt' finds an option that takes an argument,
+/* For communication from `pj_getopt' to the caller.
+   When `pj_getopt' finds an option that takes an argument,
    the argument value is returned here.
    Also, when `ordering' is RETURN_IN_ORDER,
    each non-option ARGV-element is returned here.  */
 
-extern char *optarg;
+extern char *pj_optarg;
 
 /* Index in ARGV of the next element to be scanned.
    This is used for communication to and from the caller
-   and for communication between successive calls to `getopt'.
+   and for communication between successive calls to `pj_getopt'.
 
-   On entry to `getopt', zero means this is the first call; initialize.
+   On entry to `pj_getopt', zero means this is the first call; initialize.
 
-   When `getopt' returns -1, this is the index of the first of the
+   When `pj_getopt' returns -1, this is the index of the first of the
    non-option elements that the caller should itself scan.
 
-   Otherwise, `optind' communicates from one call to the next
+   Otherwise, `pj_optind' communicates from one call to the next
    how much of ARGV has been scanned so far.  */
 
-extern int optind;
+extern int pj_optind;
 
-/* Callers store zero here to inhibit the error message `getopt' prints
+/* Callers store zero here to inhibit the error message `pj_getopt' prints
    for unrecognized options.  */
 
-extern int opterr;
+extern int pj_opterr;
 
 /* Set to an option character which was unrecognized.  */
 
-extern int optopt;
+extern int pj_optopt;
 
 /* Describe the long-named options requested by the application.
-   The LONG_OPTIONS argument to getopt_long or getopt_long_only is a vector
-   of `struct option' terminated by an element containing a name which is
+   The LONG_OPTIONS argument to pj_getopt_long or pj_getopt_long_only is a vector
+   of `struct pj_getopt_option' terminated by an element containing a name which is
    zero.
 
@@ -73 +73 @@
 
    To have a long-named option do something other than set an `int' to
-   a compiled-in constant, such as set a value from `optarg', set the
+   a compiled-in constant, such as set a value from `pj_optarg', set the
    option's `flag' field to zero and its `val' field to a nonzero
    value (the equivalent single-letter option character, if there is
-   one).  For long options that have a zero `flag' field, `getopt'
+   one).  For long options that have a zero `flag' field, `pj_getopt'
    returns the contents of the `val' field.  */
 
-struct option
+struct pj_getopt_option
 {
   const char *name;
@@ -89 +89 @@
 };
 
-/* Names for the values of the `has_arg' field of `struct option'.  */
+/* Names for the values of the `has_arg' field of `struct pj_getopt_option'.  */
 
 # define no_argument            0
@@ -102 +102 @@
    Return the option character from OPTS just read.  Return -1 when
    there are no more options.  For unrecognized options, or options
-   missing arguments, `optopt' is set to the option letter, and '?' is
+   missing arguments, `pj_optopt' is set to the option letter, and '?' is
    returned.
 
    The OPTS string is a list of characters which are recognized option
    letters, optionally followed by colons, specifying that that letter
-   takes an argument, to be placed in `optarg'.
+   takes an argument, to be placed in `pj_optarg'.
 
    If a letter in OPTS is followed by two colons, its argument is
-   optional.  This behavior is specific to the GNU `getopt'.
+   optional.  This behavior is specific to the GNU `pj_getopt'.
 
    The argument `--' causes premature termination of argument
-   scanning, explicitly telling `getopt' that there are no more
+   scanning, explicitly telling `pj_getopt' that there are no more
    options.
 
    If OPTS begins with `--', then non-option arguments are treated as
    arguments to the option '\0'.  This behavior is specific to the GNU
-   `getopt'.  */
+   `pj_getopt'.  */
 
-int getopt (int argc, char *const *argv, const char *shortopts);
+int pj_getopt (int argc, char *const *argv, const char *shortopts);
 
-int getopt_long (int argc, char *const *argv, const char *options,
-                        const struct option *longopts, int *longind);
-int getopt_long_only (int argc, char *const *argv,
+int pj_getopt_long (int argc, char *const *argv, const char *options,
+                        const struct pj_getopt_option *longopts, int *longind);
+int pj_getopt_long_only (int argc, char *const *argv,
                              const char *shortopts,
-                             const struct option *longopts, int *longind);
+                             const struct pj_getopt_option *longopts, int *longind);
 
 
@@ -133 +133 @@
 #endif
 
-#endif /* getopt.h */
+#endif /* pj_getopt.h */
 

pjproject/trunk/pjsip/include/pjsua-lib/pjsua.h

@@ -466 +466 @@
  */
 pj_bool_t pjsua_im_accept_pager(pjsip_rx_data *rdata,
-                                const pjsip_accept_hdr **p_accept_hdr);
+                                pjsip_accept_hdr **p_accept_hdr);
 
 /**

pjproject/trunk/pjsip/src/pjsua-lib/getopt.c

@@ -19 +19 @@
 
 /* 
- * getopt entry points
+ * pj_getopt entry points
  *
  * modified by Mike Borella <mike_borella@mw.3com.com>
@@ -33 +33 @@
 int _getopt_internal (int argc, char *const *argv,
                       const char *shortopts,
-                      const struct option *longopts, int *longind,
+                      const struct pj_getopt_option *longopts, int *longind,
                       int long_only);
 
-/* getopt_long and getopt_long_only entry points for GNU getopt.
+/* pj_getopt_long and pj_getopt_long_only entry points for GNU pj_getopt.
    Copyright (C) 1987,88,89,90,91,92,93,94,96,97 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
@@ -68 +68 @@
 
 int
-getopt_long (int argc, char *const *argv, const char *options, 
-             const struct option *long_options, int *opt_index)
+pj_getopt_long (int argc, char *const *argv, const char *options, 
+             const struct pj_getopt_option *long_options, int *opt_index)
 {
   return _getopt_internal (argc, argv, options, long_options, opt_index, 0);
 }
 
-/* Like getopt_long, but '-' as well as '--' can indicate a long option.
+/* Like pj_getopt_long, but '-' as well as '--' can indicate a long option.
    If an option that starts with '-' (not '--') doesn't match a long option,
    but does match a short option, it is parsed as a short option
@@ -80 +80 @@
  
 int
-getopt (int argc, char * const * argv, const char * optstring)
+pj_getopt (int argc, char * const * argv, const char * optstring)
 {
   return _getopt_internal (argc, argv, optstring,
-                           (const struct option *) 0,
+                           (const struct pj_getopt_option *) 0,
                            (int *) 0,
                            0);
@@ -91 +91 @@
 #define _(msgid)        (msgid)
 
-/* This version of `getopt' appears to the caller like standard Unix `getopt'
+/* This version of `pj_getopt' appears to the caller like standard Unix `pj_getopt'
    but it behaves differently for the user, since it allows the user
    to intersperse the options with the other arguments.
 
-   As `getopt' works, it permutes the elements of ARGV so that,
+   As `pj_getopt' works, it permutes the elements of ARGV so that,
    when it is done, all the options precede everything else.  Thus
    all application programs are extended to handle flexible argument order.
@@ -105 +105 @@
    they can distinguish the relative order of options and other arguments.  */
 
-/* For communication from `getopt' to the caller.
-   When `getopt' finds an option that takes an argument,
+/* For communication from `pj_getopt' to the caller.
+   When `pj_getopt' finds an option that takes an argument,
    the argument value is returned here.
    Also, when `ordering' is RETURN_IN_ORDER,
    each non-option ARGV-element is returned here.  */
 
-char *optarg = NULL;
+char *pj_optarg = NULL;
 
 /* Index in ARGV of the next element to be scanned.
    This is used for communication to and from the caller
-   and for communication between successive calls to `getopt'.
-
-   On entry to `getopt', zero means this is the first call; initialize.
-
-   When `getopt' returns -1, this is the index of the first of the
+   and for communication between successive calls to `pj_getopt'.
+
+   On entry to `pj_getopt', zero means this is the first call; initialize.
+
+   When `pj_getopt' returns -1, this is the index of the first of the
    non-option elements that the caller should itself scan.
 
-   Otherwise, `optind' communicates from one call to the next
+   Otherwise, `pj_optind' communicates from one call to the next
    how much of ARGV has been scanned so far.  */
 
 /* 1003.2 says this must be 1 before any call.  */
-int optind = 1;
-
-/* Formerly, initialization of getopt depended on optind==0, which
-   causes problems with re-calling getopt as programs generally don't
+int pj_optind = 1;
+
+/* Formerly, initialization of pj_getopt depended on pj_optind==0, which
+   causes problems with re-calling pj_getopt as programs generally don't
    know that. */
 
@@ -145 +145 @@
 /* Set to an option character which was unrecognized.
    This must be initialized on some systems to avoid linking in the
-   system's own getopt implementation.  */
-
-int optopt = '?';
+   system's own pj_getopt implementation.  */
+
+int pj_optopt = '?';
 
 /* Describe how to deal with options that follow non-option ARGV-elements.
@@ -176 +176 @@
    The special argument `--' forces an end of option-scanning regardless
    of the value of `ordering'.  In the case of RETURN_IN_ORDER, only
-   `--' can cause `getopt' to return -1 with `optind' != ARGC.  */
+   `--' can cause `pj_getopt' to return -1 with `pj_optind' != ARGC.  */
 
 static enum
@@ -213 +213 @@
    One subsequence is elements [first_nonopt,last_nonopt)
    which contains all the non-options that have been skipped so far.
-   The other is elements [last_nonopt,optind), which contains all
+   The other is elements [last_nonopt,pj_optind), which contains all
    the options processed since those non-options were skipped.
 
@@ -224 +224 @@
   int bottom = first_nonopt;
   int middle = last_nonopt;
-  int top = optind;
+  int top = pj_optind;
   char *tem;
 
@@ -272 +272 @@
   /* Update records for the slots the non-options now occupy.  */
 
-  first_nonopt += (optind - last_nonopt);
-  last_nonopt = optind;
+  first_nonopt += (pj_optind - last_nonopt);
+  last_nonopt = pj_optind;
 }
 
@@ -288 +288 @@
      non-option ARGV-elements is empty.  */
 
-  first_nonopt = last_nonopt = optind;
+  first_nonopt = last_nonopt = pj_optind;
 
   nextchar = NULL;
@@ -320 +320 @@
    If an element of ARGV starts with '-', and is not exactly "-" or "--",
    then it is an option element.  The characters of this element
-   (aside from the initial '-') are option characters.  If `getopt'
+   (aside from the initial '-') are option characters.  If `pj_getopt'
    is called repeatedly, it returns successively each of the option characters
    from each of the option elements.
 
-   If `getopt' finds another option character, it returns that character,
-   updating `optind' and `nextchar' so that the next call to `getopt' can
+   If `pj_getopt' finds another option character, it returns that character,
+   updating `pj_optind' and `nextchar' so that the next call to `pj_getopt' can
    resume the scan with the following option character or ARGV-element.
 
-   If there are no more option characters, `getopt' returns -1.
-   Then `optind' is the index in ARGV of the first ARGV-element
+   If there are no more option characters, `pj_getopt' returns -1.
+   Then `pj_optind' is the index in ARGV of the first ARGV-element
    that is not an option.  (The ARGV-elements have been permuted
    so that those that are not options now come last.)
@@ -335 +335 @@
    OPTSTRING is a string containing the legitimate option characters.
    If an option character is seen that is not listed in OPTSTRING,
-   return '?' after printing an error message.  If you set `opterr' to
+   return '?' after printing an error message.  If you set `pj_opterr' to
    zero, the error message is suppressed but we still return '?'.
 
    If a char in OPTSTRING is followed by a colon, that means it wants an arg,
    so the following text in the same ARGV-element, or the text of the following
-   ARGV-element, is returned in `optarg'.  Two colons mean an option that
+   ARGV-element, is returned in `pj_optarg'.  Two colons mean an option that
    wants an optional arg; if there is text in the current ARGV-element,
-   it is returned in `optarg', otherwise `optarg' is set to zero.
+   it is returned in `pj_optarg', otherwise `pj_optarg' is set to zero.
 
    If OPTSTRING starts with `-' or `+', it requests different methods of
@@ -353 +353 @@
    argument, it follows the option name in the same ARGV-element, separated
    from the option name by a `=', or else the in next ARGV-element.
-   When `getopt' finds a long-named option, it returns 0 if that option's
+   When `pj_getopt' finds a long-named option, it returns 0 if that option's
    `flag' field is nonzero, the value of the option's `val' field
    if the `flag' field is zero.
@@ -361 +361 @@
    with other systems.
 
-   LONGOPTS is a vector of `struct option' terminated by an
+   LONGOPTS is a vector of `struct pj_getopt_option' terminated by an
    element containing a name which is zero.
 
@@ -373 +373 @@
 static int
 _getopt_internal (int argc, char *const *argv, const char *optstring, 
-                  const struct option *longopts, int *longind, 
+                  const struct pj_getopt_option *longopts, int *longind, 
                   int long_only)
 {
-  optarg = NULL;
-
-  if (optind == 0 || !__getopt_initialized)
+  pj_optarg = NULL;
+
+  if (pj_optind == 0 || !__getopt_initialized)
     {
-      if (optind == 0)
-        optind = 1;     /* Don't scan ARGV[0], the program name.  */
+      if (pj_optind == 0)
+        pj_optind = 1;  /* Don't scan ARGV[0], the program name.  */
       optstring = _getopt_initialize (argc, argv, optstring);
       __getopt_initialized = 1;
     }
 
-  /* Test whether ARGV[optind] points to a non-option argument.
+  /* Test whether ARGV[pj_optind] points to a non-option argument.
      Either it does not have option syntax, or there is an environment flag
      from the shell indicating it is not an option.  The later information
      is only used when the used in the GNU libc.  */
-#define NONOPTION_P (argv[optind][0] != '-' || argv[optind][1] == '\0')
+#define NONOPTION_P (argv[pj_optind][0] != '-' || argv[pj_optind][1] == '\0')
 
   if (nextchar == NULL || *nextchar == '\0')
@@ -398 +398 @@
       /* Give FIRST_NONOPT & LAST_NONOPT rational values if OPTIND has been
          moved back by the user (who may also have changed the arguments).  */
-      if (last_nonopt > optind)
-        last_nonopt = optind;
-      if (first_nonopt > optind)
-        first_nonopt = optind;
+      if (last_nonopt > pj_optind)
+        last_nonopt = pj_optind;
+      if (first_nonopt > pj_optind)
+        first_nonopt = pj_optind;
 
       if (ordering == PERMUTE)
@@ -408 +408 @@
              exchange them so that the options come first.  */
 
-          if (first_nonopt != last_nonopt && last_nonopt != optind)
+          if (first_nonopt != last_nonopt && last_nonopt != pj_optind)
             exchange ((char **) argv);
-          else if (last_nonopt != optind)
-            first_nonopt = optind;
+          else if (last_nonopt != pj_optind)
+            first_nonopt = pj_optind;
 
           /* Skip any additional non-options
              and extend the range of non-options previously skipped.  */
 
-          while (optind < argc && NONOPTION_P)
-            optind++;
-          last_nonopt = optind;
+          while (pj_optind < argc && NONOPTION_P)
+            pj_optind++;
+          last_nonopt = pj_optind;
         }
 
@@ -426 +426 @@
          then skip everything else like a non-option.  */
 
-      if (optind != argc && !pj_ansi_strcmp(argv[optind], "--"))
-        {
-          optind++;
-
-          if (first_nonopt != last_nonopt && last_nonopt != optind)
+      if (pj_optind != argc && !pj_ansi_strcmp(argv[pj_optind], "--"))
+        {
+          pj_optind++;
+
+          if (first_nonopt != last_nonopt && last_nonopt != pj_optind)
             exchange ((char **) argv);
           else if (first_nonopt == last_nonopt)
-            first_nonopt = optind;
+            first_nonopt = pj_optind;
           last_nonopt = argc;
 
-          optind = argc;
+          pj_optind = argc;
         }
 
@@ -442 +442 @@
          and back over any non-options that we skipped and permuted.  */
 
-      if (optind == argc)
+      if (pj_optind == argc)
         {
           /* Set the next-arg-index to point at the non-options
              that we previously skipped, so the caller will digest them.  */
           if (first_nonopt != last_nonopt)
-            optind = first_nonopt;
+            pj_optind = first_nonopt;
           return -1;
         }
@@ -458 +458 @@
           if (ordering == REQUIRE_ORDER)
             return -1;
-          optarg = argv[optind++];
+          pj_optarg = argv[pj_optind++];
           return 1;
         }
@@ -465 +465 @@
          Skip the initial punctuation.  */
 
-      nextchar = (argv[optind] + 1
-                  + (longopts != NULL && argv[optind][1] == '-'));
+      nextchar = (argv[pj_optind] + 1
+                  + (longopts != NULL && argv[pj_optind][1] == '-'));
     }
 
@@ -485 +485 @@
 
   if (longopts != NULL
-      && (argv[optind][1] == '-'
-          || (long_only && (argv[optind][2] || !my_index (optstring, argv[optind][1])))))
+      && (argv[pj_optind][1] == '-'
+          || (long_only && (argv[pj_optind][2] || !my_index (optstring, argv[pj_optind][1])))))
     {
       char *nameend;
-      const struct option *p;
-      const struct option *pfound = NULL;
+      const struct pj_getopt_option *p;
+      const struct pj_getopt_option *pfound = NULL;
       int exact = 0;
       int ambig = 0;
@@ -527 +527 @@
         {
           nextchar += strlen (nextchar);
-          optind++;
-          optopt = 0;
+          pj_optind++;
+          pj_optopt = 0;
           return '?';
         }
@@ -535 +535 @@
         {
           option_index = indfound;
-          optind++;
+          pj_optind++;
           if (*nameend)
             {
@@ -541 +541 @@
                  allow it to be used on enums.  */
               if (pfound->has_arg)
-                optarg = nameend + 1;
+                pj_optarg = nameend + 1;
               else
                 {
                   nextchar += strlen (nextchar);
 
-                  optopt = pfound->val;
+                  pj_optopt = pfound->val;
                   return '?';
                 }
@@ -552 +552 @@
           else if (pfound->has_arg == 1)
             {
-              if (optind < argc)
-                optarg = argv[optind++];
+              if (pj_optind < argc)
+                pj_optarg = argv[pj_optind++];
               else
                 {
                   nextchar += strlen (nextchar);
-                  optopt = pfound->val;
+                  pj_optopt = pfound->val;
                   return optstring[0] == ':' ? ':' : '?';
                 }
@@ -572 +572 @@
         }
 
-      /* Can't find it as a long option.  If this is not getopt_long_only,
+      /* Can't find it as a long option.  If this is not pj_getopt_long_only,
          or the option starts with '--' or is not a valid short
          option, then it's an error.
          Otherwise interpret it as a short option.  */
-      if (!long_only || argv[optind][1] == '-'
+      if (!long_only || argv[pj_optind][1] == '-'
           || my_index (optstring, *nextchar) == NULL)
         {
           nextchar = (char *) "";
-          optind++;
-          optopt = 0;
+          pj_optind++;
+          pj_optopt = 0;
           return '?';
         }
@@ -592 +592 @@
     char *temp = my_index (optstring, c);
 
-    /* Increment `optind' when we start to process its last character.  */
+    /* Increment `pj_optind' when we start to process its last character.  */
     if (*nextchar == '\0')
-      ++optind;
+      ++pj_optind;
 
     if (temp == NULL || c == ':')
       {
-        optopt = c;
+        pj_optopt = c;
         return '?';
       }
@@ -605 +605 @@
       {
         char *nameend;
-        const struct option *p;
-        const struct option *pfound = NULL;
+        const struct pj_getopt_option *p;
+        const struct pj_getopt_option *pfound = NULL;
         int exact = 0;
         int ambig = 0;
@@ -615 +615 @@
         if (*nextchar != '\0')
           {
-            optarg = nextchar;
+            pj_optarg = nextchar;
             /* If we end this ARGV-element by taking the rest as an arg,
                we must advance to the next element now.  */
-            optind++;
+            pj_optind++;
           }
-        else if (optind == argc)
+        else if (pj_optind == argc)
           {
-            optopt = c;
+            pj_optopt = c;
             if (optstring[0] == ':')
               c = ':';
@@ -630 +630 @@
           }
         else
-          /* We already incremented `optind' once;
+          /* We already incremented `pj_optind' once;
              increment it again when taking next ARGV-elt as argument.  */
-          optarg = argv[optind++];
-
-        /* optarg is now the argument, see if it's in the
+          pj_optarg = argv[pj_optind++];
+
+        /* pj_optarg is now the argument, see if it's in the
            table of longopts.  */
 
-        for (nextchar = nameend = optarg; *nameend && *nameend != '='; nameend++)
+        for (nextchar = nameend = pj_optarg; *nameend && *nameend != '='; nameend++)
           /* Do nothing.  */ ;
 
@@ -666 +666 @@
           {
             nextchar += strlen (nextchar);
-            optind++;
+            pj_optind++;
             return '?';
           }
@@ -677 +677 @@
                    allow it to be used on enums.  */
                 if (pfound->has_arg)
-                  optarg = nameend + 1;
+                  pj_optarg = nameend + 1;
                 else
                   {
@@ -686 +686 @@
             else if (pfound->has_arg == 1)
               {
-                if (optind < argc)
-                  optarg = argv[optind++];
+                if (pj_optind < argc)
+                  pj_optarg = argv[pj_optind++];
                 else
                   {
@@ -714 +714 @@
             if (*nextchar != '\0')
               {
-                optarg = nextchar;
-                optind++;
+                pj_optarg = nextchar;
+                pj_optind++;
               }
             else
-              optarg = NULL;
+              pj_optarg = NULL;
             nextchar = NULL;
           }
@@ -726 +726 @@
             if (*nextchar != '\0')
               {
-                optarg = nextchar;
+                pj_optarg = nextchar;
                 /* If we end this ARGV-element by taking the rest as an arg,
                    we must advance to the next element now.  */
-                optind++;
+                pj_optind++;
               }
-            else if (optind == argc)
+            else if (pj_optind == argc)
               {
-                optopt = c;
+                pj_optopt = c;
                 if (optstring[0] == ':')
                   c = ':';
@@ -740 +740 @@
               }
             else
-              /* We already incremented `optind' once;
+              /* We already incremented `pj_optind' once;
                  increment it again when taking next ARGV-elt as argument.  */
-              optarg = argv[optind++];
+              pj_optarg = argv[pj_optind++];
             nextchar = NULL;
           }

pjproject/trunk/pjsip/src/pjsua-lib/pjsua_im.c

@@ -108 +108 @@
  */
 pj_bool_t pjsua_im_accept_pager(pjsip_rx_data *rdata,
-                                const pjsip_accept_hdr **p_accept_hdr)
+                                pjsip_accept_hdr **p_accept_hdr)
 {
     pjsip_ctype_hdr *ctype;

pjproject/trunk/pjsip/src/pjsua-lib/pjsua_settings.c

@@ -228 +228 @@
            OPT_NEXT_ACCOUNT, OPT_NEXT_CRED, OPT_MAX_CALLS,
     };
-    struct option long_options[] = {
+    struct pj_getopt_option long_options[] = {
         { "config-file",1, 0, OPT_CONFIG_FILE},
         { "log-file",   1, 0, OPT_LOG_FILE},
@@ -273 +273 @@
     char *config_file = NULL;
 
-    /* Run getopt once to see if user specifies config file to read. */
-    while ((c=getopt_long(argc, argv, "", long_options, &option_index)) != -1) {
+    /* Run pj_getopt once to see if user specifies config file to read. */ 
+    while ((c=pj_getopt_long(argc, argv, "", long_options, &option_index)) != -1) {
         switch (c) {
         case OPT_CONFIG_FILE:
-            config_file = optarg;
+            config_file = pj_optarg;
             break;
         }
@@ -295 +295 @@
 
 
-    /* Reinitialize and re-run getopt again, possibly with new arguments
+    /* Reinitialize and re-run pj_getopt again, possibly with new arguments
      * read from config file.
      */
-    optind = 0;
-    while ((c=getopt_long(argc, argv, "", long_options, &option_index)) != -1) {
+    pj_optind = 0;
+    while((c=pj_getopt_long(argc, argv, "", long_options, &option_index))!=-1) {
         char *p;
         pj_str_t tmp;
@@ -307 +307 @@
 
         case OPT_LOG_FILE:
-            pjsua.log_filename = optarg;
+            pjsua.log_filename = pj_optarg;
             break;
 
         case OPT_LOG_LEVEL:
-            c = pj_strtoul(pj_cstr(&tmp, optarg));
+            c = pj_strtoul(pj_cstr(&tmp, pj_optarg));
             if (c < 0 || c > 6) {
                 printf("Error: expecting integer value 0-6 for --log-level\n");
@@ -320 +320 @@
 
         case OPT_APP_LOG_LEVEL:
-            pjsua.app_log_level = pj_strtoul(pj_cstr(&tmp, optarg));
+            pjsua.app_log_level = pj_strtoul(pj_cstr(&tmp, pj_optarg));
             if (pjsua.app_log_level < 0 || pjsua.app_log_level > 6) {
                 printf("Error: expecting integer value 0-6 for --app-log-level\n");
@@ -348 +348 @@
 
         case OPT_LOCAL_PORT:   /* local-port */
-            lval = pj_strtoul(pj_cstr(&tmp, optarg));
+            lval = pj_strtoul(pj_cstr(&tmp, pj_optarg));
             if (lval < 1 || lval > 65535) {
                 printf("Error: expecting integer value for --local-port\n");
@@ -357 +357 @@
 
         case OPT_PROXY:   /* proxy */
-            if (pjsua_verify_sip_url(optarg) != 0) {
-                printf("Error: invalid SIP URL '%s' in proxy argument\n", optarg);
+            if (pjsua_verify_sip_url(pj_optarg) != 0) {
+                printf("Error: invalid SIP URL '%s' in proxy argument\n", pj_optarg);
                 return PJ_EINVAL;
             }
-            cur_acc->proxy = pj_str(optarg);
+            cur_acc->proxy = pj_str(pj_optarg);
             break;
 
         case OPT_OUTBOUND_PROXY:   /* outbound proxy */
-            if (pjsua_verify_sip_url(optarg) != 0) {
-                printf("Error: invalid SIP URL '%s' in outbound proxy argument\n", optarg);
+            if (pjsua_verify_sip_url(pj_optarg) != 0) {
+                printf("Error: invalid SIP URL '%s' in outbound proxy argument\n", pj_optarg);
                 return PJ_EINVAL;
             }
-            pjsua.outbound_proxy = pj_str(optarg);
+            pjsua.outbound_proxy = pj_str(pj_optarg);
             break;
 
         case OPT_REGISTRAR:   /* registrar */
-            if (pjsua_verify_sip_url(optarg) != 0) {
-                printf("Error: invalid SIP URL '%s' in registrar argument\n", optarg);
+            if (pjsua_verify_sip_url(pj_optarg) != 0) {
+                printf("Error: invalid SIP URL '%s' in registrar argument\n", pj_optarg);
                 return PJ_EINVAL;
             }
-            cur_acc->reg_uri = pj_str(optarg);
+            cur_acc->reg_uri = pj_str(pj_optarg);
             break;
 
         case OPT_REG_TIMEOUT:   /* reg-timeout */
-            cur_acc->reg_timeout = pj_strtoul(pj_cstr(&tmp,optarg));
+            cur_acc->reg_timeout = pj_strtoul(pj_cstr(&tmp,pj_optarg));
             if (cur_acc->reg_timeout < 1 || cur_acc->reg_timeout > 3600) {
                 printf("Error: invalid value for --reg-timeout (expecting 1-3600)\n");
@@ -389 +389 @@
 
         case OPT_ID:   /* id */
-            if (pjsua_verify_sip_url(optarg) != 0) {
-                printf("Error: invalid SIP URL '%s' in local id argument\n", optarg);
+            if (pjsua_verify_sip_url(pj_optarg) != 0) {
+                printf("Error: invalid SIP URL '%s' in local id argument\n", pj_optarg);
                 return PJ_EINVAL;
             }
-            cur_acc->local_uri = pj_str(optarg);
+            cur_acc->local_uri = pj_str(pj_optarg);
             pjsua.has_acc = 1;
             break;
 
         case OPT_CONTACT:   /* contact */
-            if (pjsua_verify_sip_url(optarg) != 0) {
-                printf("Error: invalid SIP URL '%s' in contact argument\n", optarg);
+            if (pjsua_verify_sip_url(pj_optarg) != 0) {
+                printf("Error: invalid SIP URL '%s' in contact argument\n", pj_optarg);
                 return PJ_EINVAL;
             }
-            cur_acc->contact_uri = pj_str(optarg);
+            cur_acc->contact_uri = pj_str(pj_optarg);
             break;
 
@@ -412 +412 @@
         case OPT_USERNAME:   /* Default authentication user */
             if (pjsua.cred_count==0) pjsua.cred_count=1;
-            cur_cred->username = pj_str(optarg);
+            cur_cred->username = pj_str(pj_optarg);
             break;
 
         case OPT_REALM:     /* Default authentication realm. */
             if (pjsua.cred_count==0) pjsua.cred_count=1;
-            cur_cred->realm = pj_str(optarg);
+            cur_cred->realm = pj_str(pj_optarg);
             break;
 
@@ -423 +423 @@
             if (pjsua.cred_count==0) pjsua.cred_count=1;
             cur_cred->data_type = 0;
-            cur_cred->data = pj_str(optarg);
+            cur_cred->data = pj_str(pj_optarg);
             break;
 
@@ -432 +432 @@
 
         case OPT_USE_STUN1:   /* STUN server 1 */
-            p = pj_ansi_strchr(optarg, ':');
+            p = pj_ansi_strchr(pj_optarg, ':');
             if (p) {
                 *p = '\0';
-                pjsua.stun_srv1 = pj_str(optarg);
+                pjsua.stun_srv1 = pj_str(pj_optarg);
                 pjsua.stun_port1 = pj_strtoul(pj_cstr(&tmp, p+1));
                 if (pjsua.stun_port1 < 1 || pjsua.stun_port1 > 65535) {
@@ -443 +443 @@
             } else {
                 pjsua.stun_port1 = 3478;
-                pjsua.stun_srv1 = pj_str(optarg);
+                pjsua.stun_srv1 = pj_str(pj_optarg);
             }
             break;
 
         case OPT_USE_STUN2:   /* STUN server 2 */
-            p = pj_ansi_strchr(optarg, ':');
+            p = pj_ansi_strchr(pj_optarg, ':');
             if (p) {
                 *p = '\0';
-                pjsua.stun_srv2 = pj_str(optarg);
+                pjsua.stun_srv2 = pj_str(pj_optarg);
                 pjsua.stun_port2 = pj_strtoul(pj_cstr(&tmp,p+1));
                 if (pjsua.stun_port2 < 1 || pjsua.stun_port2 > 65535) {
@@ -459 +459 @@
             } else {
                 pjsua.stun_port2 = 3478;
-                pjsua.stun_srv2 = pj_str(optarg);
+                pjsua.stun_srv2 = pj_str(pj_optarg);
             }
             break;
 
         case OPT_ADD_BUDDY: /* Add to buddy list. */
-            if (pjsua_verify_sip_url(optarg) != 0) {
-                printf("Error: invalid URL '%s' in --add-buddy option\n", optarg);
+            if (pjsua_verify_sip_url(pj_optarg) != 0) {
+                printf("Error: invalid URL '%s' in --add-buddy option\n", pj_optarg);
                 return -1;
             }
@@ -472 +472 @@
                 return -1;
             }
-            pjsua.buddies[pjsua.buddy_cnt++].uri = pj_str(optarg);
+            pjsua.buddies[pjsua.buddy_cnt++].uri = pj_str(pj_optarg);
             break;
 
@@ -488 +488 @@
 
         case OPT_PLAY_FILE:
-            pjsua.wav_file = optarg;
+            pjsua.wav_file = pj_optarg;
             break;
 
         case OPT_RTP_PORT:
-            pjsua.start_rtp_port = my_atoi(optarg);
+            pjsua.start_rtp_port = my_atoi(pj_optarg);
             if (pjsua.start_rtp_port < 1 || pjsua.start_rtp_port > 65535) {
                 PJ_LOG(1,(THIS_FILE,
@@ -500 +500 @@
 
         case OPT_ADD_CODEC:
-            pjsua.codec_arg[pjsua.codec_cnt++] = pj_str(optarg);
+            pjsua.codec_arg[pjsua.codec_cnt++] = pj_str(pj_optarg);
             break;
 
         case OPT_COMPLEXITY:
-            pjsua.complexity = my_atoi(optarg);
+            pjsua.complexity = my_atoi(pj_optarg);
             if (pjsua.complexity < 0 || pjsua.complexity > 10) {
                 PJ_LOG(1,(THIS_FILE,
@@ -513 +513 @@
 
         case OPT_QUALITY:
-            pjsua.quality = my_atoi(optarg);
+            pjsua.quality = my_atoi(pj_optarg);
             if (pjsua.quality < 0 || pjsua.quality > 10) {
                 PJ_LOG(1,(THIS_FILE,
@@ -522 +522 @@
 
         case OPT_AUTO_ANSWER:
-            pjsua.auto_answer = my_atoi(optarg);
+            pjsua.auto_answer = my_atoi(pj_optarg);
             if (pjsua.auto_answer < 100 || pjsua.auto_answer > 699) {
                 PJ_LOG(1,(THIS_FILE,
@@ -531 +531 @@
 
         case OPT_MAX_CALLS:
-            pjsua.max_calls = my_atoi(optarg);
+            pjsua.max_calls = my_atoi(pj_optarg);
             if (pjsua.max_calls < 1 || pjsua.max_calls > 255) {
                 PJ_LOG(1,(THIS_FILE,"Too many calls for max-calls (1-255)"));
@@ -540 +540 @@
     }
 
-    if (optind != argc) {
-        printf("Error: unknown options %s\n", argv[optind]);
+    if (pj_optind != argc) {
+        printf("Error: unknown options %s\n", argv[pj_optind]);
         return PJ_EINVAL;
     }