getopt return value changed from char to int (https://github.com/clicon/clixon/issues/58)

CHANGELOG.md

@@ -12,7 +12,7 @@
   * Support of submodule, include and belongs-to.
   * Openconfig yang specs parsed: https://github.com/openconfig/public
   * Improved unknown handling
-  * `CLICON_YANG_DIR` is changed from a single directory to a path of directories
+  * Configure option `CLICON_YANG_DIR` is changed from a single directory to a path of directories
     * Note CLIXON_DATADIR (=/usr/local/share/clixon) need to be in the list
 
 ### API changes on existing features (you may need to change your code)
@@ -26,7 +26,10 @@
   * Syntactically Correct handling of '<?' (processing instructions) and '<?xml' (XML declaration)
   * XML prolog syntax for 'well-formed' XML
   * <!DOCTYPE (ie DTD) is not supported.
+
 ### Corrected Bugs
+* getopt return value changed from char to int (https://github.com/clicon/clixon/issues/58)
+
 ### Known issues
 
 ## 3.8.0 (6 Nov 2018)

apps/backend/backend_main.c

@@ -504,7 +504,7 @@ main(int    argc,
      char **argv)
 {
     int           retval = -1;
-    char          c;
+    int           c;
     int           zap;
     int           foreground;
     int           once;

apps/cli/cli_main.c

@@ -240,7 +240,7 @@ int
 main(int argc, char **argv)
 {
     int          retval = -1;
-    char         c;    
+    int          c;    
     int          once;
     char	*tmp;
     char	*argv0 = argv[0];

apps/netconf/netconf_main.c

@@ -330,7 +330,7 @@ int
 main(int    argc,
      char **argv)
 {
-    char             c;
+    int              c;
     char            *tmp;
     char            *argv0 = argv[0];
     int              quiet = 0;

apps/restconf/restconf_main.c

@@ -513,7 +513,7 @@ main(int    argc,
     char	 *argv0 = argv[0];
     FCGX_Request  request;
     FCGX_Request *r = &request;
-    char          c;
+    int           c;
     char         *sockpath;
     char         *path;
     clicon_handle h;

datastore/datastore_client.c

@@ -107,7 +107,7 @@ usage(char *argv0)
 int
 main(int argc, char **argv)
 {
-    char                c;
+    int                 c;
     clicon_handle       h;
     char               *argv0;
     char               *db = "running";

example/example_restconf.c

@@ -289,7 +289,7 @@ example_restconf_start(clicon_handle h,
 		       int           argc,
 		       char        **argv)
 {
-    char c;
+    int c;
 
     clicon_debug(1, "%s argc:%d", __FUNCTION__, argc);
     optind = 1;

lib/src/json_xpath.c

@@ -28,7 +28,7 @@ usage(char *argv0)
 int
 main(int argc, char **argv)
 {
-    int i;
+    int         i;
     cxobj     **xv;
     cxobj      *x;
     cxobj      *xn;
@@ -42,7 +42,7 @@ main(int argc, char **argv)
     char       *buf;
     char       *p;
     int         retval;
-    char        c;
+    int         c;
 
     while ((c = getopt(argc, argv, "?hbj")) != -1)
       switch (c) {

util/clixon_util_stream.c

@@ -218,7 +218,7 @@ main(int argc, char **argv)
     int    timeout = 10;
     char   start[27] = {0,}; /* strlen = 0 */
     char   stop[27] = {0,};
-    char   c;
+    int    c;
     char  *argv0 = argv[0];
     struct timeval now;
 

util/clixon_util_xml.c

@@ -85,7 +85,7 @@ main(int    argc,
     cxobj *xc;
     cbuf  *cb = cbuf_new();
     int   retval = -1;
-    char  c;
+    int   c;
     int   logdst = CLICON_LOG_STDERR;
 
     optind = 1;

util/clixon_util_xpath.c

@@ -119,7 +119,7 @@ main(int argc, char **argv)
     cxobj     **xv = NULL;
     cxobj      *x0 = NULL;
     cxobj      *x;
-    char        c;
+    int         c;
     int         len;
     char       *buf = NULL;
     int         ret;

util/clixon_util_yang.c

@@ -81,8 +81,8 @@ int
 main(int argc, char **argv)
 {
     yang_spec *yspec = NULL;
-    char      c;
+    int        c;
-    int       logdst = CLICON_LOG_STDERR;
+    int        logdst = CLICON_LOG_STDERR;
     
     optind = 1;
     opterr = 0;