Changeset: a05b340c0496 for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=a05b340c0496
Added Files:
        sql/test/BugTracker/Tests/mapi_connect_errors_vanish.SF-1432134.sql
Removed Files:
        sql/test/BugTracker/Tests/mapi_connect_errors_vanish.SF-1432134.SQL.py
Modified Files:
        clients/mapiclient/mclient.c
        gdk/gdk.h
        gdk/gdk_batop.c
        monetdb5/mal/mal_readline.c
        sql/test/BugTracker/Tests/All
Branch: default
Log Message:

Merge with Mar2018 branch.


diffs (truncated from 454 to 300 lines):

diff --git a/clients/mapiclient/mclient.c b/clients/mapiclient/mclient.c
--- a/clients/mapiclient/mclient.c
+++ b/clients/mapiclient/mclient.c
@@ -71,11 +71,11 @@ static stream *fromConsole = NULL;
 static char *language = NULL;
 static char *logfile = NULL;
 static char promptbuf[16];
-static int echoquery = 0;
+static bool echoquery = false;
 #ifdef HAVE_ICONV
 static char *encoding;
 #endif
-static int errseen = 0;
+static bool errseen = false;
 
 #define setPrompt() sprintf(promptbuf, "%.*s>", (int) sizeof(promptbuf) - 2, 
language)
 #define debugMode() (strncmp(promptbuf, "mdb", 3) == 0)
@@ -94,7 +94,7 @@ enum formatters {
 };
 static enum formatters formatter = NOformatter;
 char *separator = NULL;                /* column separator for CSV/TAB format 
*/
-int csvheader = 0;             /* include header line in CSV format */
+bool csvheader = false;                /* include header line in CSV format */
 
 #define DEFWIDTH 80
 
@@ -123,9 +123,9 @@ static char *pager = 0;             /* use external
 #endif
 static int rowsperpage = 0;    /* for SQL pagination */
 static int pagewidth = 0;      /* -1: take whatever is necessary, >0: limit */
-static int pagewidthset = 0;   /* whether the user set the width explicitly */
+static bool pagewidthset = false; /* whether the user set the width explicitly 
*/
 static int croppedfields = 0;  /* whatever got cropped/truncated */
-static char firstcrop = 1;     /* first time we see cropping/truncation */
+static bool firstcrop = true;  /* first time we see cropping/truncation */
 
 enum modifiers {
        NOmodifier,
@@ -259,13 +259,13 @@ static enum itimers {
        T_PERF          // return detailed performance
 } timermode = T_NONE;
 
-static int timerHumanCalled = 0;
+static bool timerHumanCalled = false;
 static void
 timerHuman(int64_t sqloptimizer, int64_t maloptimizer, int64_t querytime, int 
singleinstr, int total)
 {
        timertype t = th - t0;
 
-       timerHumanCalled = 1;
+       timerHumanCalled = true;
 
        /*
         * report only the times we do actually measure:
@@ -571,7 +571,8 @@ utf8skip(char *s, size_t i)
 static int
 SQLrow(int *len, int *numeric, char **rest, int fields, int trim, char wm)
 {
-       int i, more, first = 1;
+       int i;
+       bool more, first = true;
        char *t;
        int rows = 0;           /* return number of output lines printed */
        size_t ulen;
@@ -598,7 +599,7 @@ SQLrow(int *len, int *numeric, char **re
                cutafter[i] = -1;
 
        do {
-               more = 0;
+               more = false;
                for (i = 0; i < fields; i++) {
                        if (rest[i] == NULL || *rest[i] == 0) {
                                mnstr_printf(toConsole, "%c %*s ",
@@ -697,7 +698,7 @@ SQLrow(int *len, int *numeric, char **re
                                        if (cutafter[i] == 0)
                                                rest[i] = NULL;
                                        if (rest[i])
-                                               more = 1;
+                                               more = true;
                                } else {
                                        mnstr_printf(toConsole, "%c",
                                                     first ? '|' : i > 0 && 
cutafter[i - 1] == 0 ? '>' : ':');
@@ -736,7 +737,7 @@ SQLrow(int *len, int *numeric, char **re
                mnstr_printf(toConsole, "%c%s\n",
                             first ? '|' : i > 0 && cutafter[i - 1] == 0 ? '>' 
: ':',
                             wm ? ">" : "");
-               first = 0;
+               first = false;
                rows++;
        } while (more);
 
@@ -1667,8 +1668,8 @@ SQLrenderer(MapiHdl hdl)
                       croppedfields, croppedfields != 1 ? "s" : "");
        if (fields != printfields || croppedfields > 0) {
                mnstr_printf(toConsole, "!");
-               if (firstcrop == 1) {
-                       firstcrop = 0;
+               if (firstcrop) {
+                       firstcrop = false;
                        mnstr_printf(toConsole, "\nnote: to disable dropping 
columns and/or truncating fields use \\w-1");
                }
        }
@@ -1686,7 +1687,7 @@ setFormatter(const char *s)
        if (separator)
                free(separator);
        separator = NULL;
-       csvheader = 0;
+       csvheader = false;
 #ifdef _TWO_DIGIT_EXPONENT
        if (formatter == TESTformatter)
                _set_output_format(0);
@@ -1712,7 +1713,7 @@ setFormatter(const char *s)
                                separator[strlen(separator) - 1] = 0;
                } else
                        separator = strdup(s + 4);
-               csvheader = 1;
+               csvheader = true;
        } else if (strcmp(s, "tab") == 0) {
                formatter = CSVformatter;
                separator = strdup("\t");
@@ -1824,7 +1825,7 @@ format_result(Mapi mid, MapiHdl hdl, int
 
        setWidth();
 
-       timerHumanCalled = 0;
+       timerHumanCalled = false;
 
        do {
                /* handle errors first */
@@ -1836,7 +1837,7 @@ format_result(Mapi mid, MapiHdl hdl, int
                                mapi_noexplain(mid, NULL);
                        }
                        mapi_explain_result(hdl, stderr);
-                       errseen = 1;
+                       errseen = true;
                        /* don't need to print something like '0
                         * tuples' if we got an error */
                        continue;
@@ -1983,7 +1984,7 @@ format_result(Mapi mid, MapiHdl hdl, int
        if (mnstr_errnr(toConsole)) {
                mnstr_clearerr(toConsole);
                fprintf(stderr, "write error\n");
-               errseen = 1;
+               errseen = true;
        }
 #ifdef HAVE_POPEN
        end_pager(saveFD);
@@ -2008,7 +2009,7 @@ doRequest(Mapi mid, const char *buf)
                        mapi_noexplain(mid, NULL);
                }
                mapi_explain(mid, stderr);
-               errseen = 1;
+               errseen = true;
                return 1;
        }
 
@@ -2040,7 +2041,7 @@ doRequest(Mapi mid, const char *buf)
                        hdl = NULL;                             \
                } else                                          \
                        mapi_explain(mid, stderr);              \
-               errseen = 1;                                    \
+               errseen = true;                                 \
                break_or_continue;                              \
        case MTIMEOUT:                                          \
                /* lost contact with the server */              \
@@ -2055,7 +2056,7 @@ doRequest(Mapi mid, const char *buf)
                        hdl = NULL;                             \
                } else                                          \
                        mapi_explain(mid, stderr);              \
-               errseen = 1;                                    \
+               errseen = true;                                 \
                timerEnd();                                     \
                if (buf)                                        \
                        free(buf);                              \
@@ -2064,7 +2065,7 @@ doRequest(Mapi mid, const char *buf)
                return 1;                                       \
        }
 
-static int
+static bool
 doFileBulk(Mapi mid, stream *fp)
 {
        char *buf = NULL;
@@ -2079,7 +2080,7 @@ doFileBulk(Mapi mid, stream *fp)
                fprintf(stderr, "cannot allocate memory for send buffer\n");
                if (fp)
                        close_stream(fp);
-               return 1;
+               return true;
        }
 
        timerStart();
@@ -2100,7 +2101,7 @@ doFileBulk(Mapi mid, stream *fp)
                        buf[length] = 0;
                        if (strlen(buf) < (size_t) length) {
                                fprintf(stderr, "NULL byte in input\n");
-                               errseen = 1;
+                               errseen = true;
                                break;
                        }
                }
@@ -2260,7 +2261,7 @@ mydestroy(void *private)
 }
 #endif
 
-static int
+static bool
 doFile(Mapi mid, stream *fp, bool useinserts, bool interactive, int 
save_history)
 {
        char *line = NULL;
@@ -2319,12 +2320,12 @@ doFile(Mapi mid, stream *fp, bool useins
                exit(2);
        }
        do {
-               int seen_null_byte = 0;
+               bool seen_null_byte = false;
 
                if (prompt) {
                        char *p = hdl ? "more>" : prompt;
                        /* clear errors when interactive */
-                       errseen = 0;
+                       errseen = false;
 #ifdef HAVE_LIBREADLINE
                        rl.prompt = p;
 #else
@@ -2343,8 +2344,8 @@ doFile(Mapi mid, stream *fp, bool useins
                                break;
                        if (!seen_null_byte && strlen(buf + length) < (size_t) 
l) {
                                fprintf(stderr, "NULL byte in input on line %d 
of input\n", lineno);
-                               seen_null_byte = 1;
-                               errseen = 1;
+                               seen_null_byte = true;
+                               errseen = true;
                                if (hdl) {
                                        mapi_close_handle(hdl);
                                        hdl = NULL;
@@ -2359,7 +2360,7 @@ doFile(Mapi mid, stream *fp, bool useins
                        } else {
                                fprintf(stderr,"Malloc failure");
                                length = 0;
-                               errseen = 1;
+                               errseen = true;
                                if (hdl) {
                                        mapi_close_handle(hdl);
                                        hdl = NULL;
@@ -2436,12 +2437,12 @@ doFile(Mapi mid, stream *fp, bool useins
                                        rowsperpage = atoi(line + 2);
                                        continue;
                                case 'd': {
-                                       char hasWildcard = 0;
-                                       char hasSchema = 0;
-                                       char wantsSystem = 0;
+                                       bool hasWildcard = false;
+                                       bool hasSchema = false;
+                                       bool wantsSystem = false;
                                        unsigned int x = 0;
                                        char *p, *q;
-                                       char escaped = 0;
+                                       bool escaped = false;
                                        if (mode != SQL)
                                                break;
                                        while (my_isspace(line[length - 1]))
@@ -2466,7 +2467,7 @@ doFile(Mapi mid, stream *fp, bool useins
                                                        x |= MD_SCHEMA;
                                                        break;
                                                case 'S':
-                                                       wantsSystem = 1;
+                                                       wantsSystem = true;
                                                        break;
                                                default:
                                                        fprintf(stderr, 
"unknown sub-command for \\d: %c\n", *line);
@@ -2491,22 +2492,22 @@ doFile(Mapi mid, stream *fp, bool useins
                                                                        /* SQL 
escape */
                                                                        *q++ = 
*p++;
                                                                } else {
-                                                                       escaped 
= 0;
+                                                                       escaped 
= false;
                                                                }
                                                        } else {
-                                                               escaped = 1;
+                                                               escaped = true;
                                                        }
                                                } else {
                                                        if (!escaped) {
                                                                *q++ = 
tolower((int) *p);
                                                                if (*p == '*') {
                                                                        *p = 
'%';
-                                                                       
hasWildcard = 1;
+                                                                       
hasWildcard = true;
                                                                } else if (*p 
== '?') {
                                                                        *p = 
'_';
-                                                                       
hasWildcard = 1;
+                                                                       
hasWildcard = true;
                                                                } else if (*p 
== '.') {
-                                                                       
hasSchema = 1;
+                                                                       
hasSchema = true;
                                                                }
                                                        } else {
                                                                *q++ = *p;
@@ -2545,7 +2546,7 @@ doFile(Mapi mid, stream *fp, bool useins
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to