Changeset: 7378903c7bd9 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=7378903c7bd9
Modified Files:
        clients/mapiclient/ReadlineTools.c
Branch: default
Log Message:

Merge with Feb2013 branch.


diffs (76 lines):

diff --git a/clients/mapiclient/ReadlineTools.c 
b/clients/mapiclient/ReadlineTools.c
--- a/clients/mapiclient/ReadlineTools.c
+++ b/clients/mapiclient/ReadlineTools.c
@@ -228,7 +228,7 @@ static char *mal_commands[] = {
 static int
 mal_help(int cnt, int key)
 {
-       char *name, *c, buf[BUFSIZ];
+       char *name, *c, buf[512];
        int seekpos = 0, rowcount;
        MapiHdl table_hdl;
 
@@ -240,7 +240,7 @@ mal_help(int cnt, int key)
                c--;
        while (c > rl_line_buffer && !isspace(*c))
                c--;
-       snprintf(buf, BUFSIZ, "manual.help(\"%s\");", c);
+       snprintf(buf, sizeof(buf), "manual.help(\"%s\");", c);
        if ((table_hdl = mapi_query(_mid, buf)) == NULL || mapi_error(_mid)) {
                if (table_hdl) {
                        mapi_explain_query(table_hdl, stderr);
@@ -271,7 +271,7 @@ mal_command_generator(const char *text, 
        static int idx;
        static int seekpos, len, rowcount;
        static MapiHdl table_hdl;
-       char *name, buf[BUFSIZ];
+       char *name, buf[512];
 
        /* we pick our own portion of the linebuffer */
        text = rl_line_buffer + strlen(rl_line_buffer) - 1;
@@ -295,20 +295,19 @@ mal_command_generator(const char *text, 
        }
        /* try the server to answer */
        if (!state) {
-               char cmd[BUFSIZ], *c;
+               char *c;
                c = strstr(text, ":=");
                if (c)
                        text = c + 2;
                while (isspace((int) *text))
                        text++;
-               c = strchr(text, '.');
-               if (c == NULL)
-                       snprintf(cmd, BUFSIZ, "%s.*(", text);
+               if (strchr(text, '.') == NULL)
+                       snprintf(buf, sizeof(buf),
+                                "manual.completion(\"%s.*(\");", text);
                else
-                       snprintf(cmd, BUFSIZ, "%s(", text);
+                       snprintf(buf, sizeof(buf),
+                                "manual.completion(\"%s(\");", text);
                seekpos = 0;
-               len = strlen(cmd);
-               snprintf(buf, BUFSIZ, "manual.completion(\"%s\");", cmd);
                if ((table_hdl = mapi_query(_mid, buf)) == NULL || 
mapi_error(_mid)) {
                        if (table_hdl) {
                                mapi_explain_query(table_hdl, stderr);
@@ -365,7 +364,8 @@ init_readline(Mapi mid, char *lang, int 
        _mid = mid;
        /* Allow conditional parsing of the ~/.inputrc file. */
        rl_readline_name = "MapiClient";
-       /* Tell the completer that we want to try our own completion before std 
completion (filename) kicks in. */
+       /* Tell the completer that we want to try our own completion
+        * before std completion (filename) kicks in. */
        if (strcmp(language, "sql") == 0) {
                rl_attempted_completion_function = sql_completion;
        } else if (strcmp(language, "mil") == 0) {
@@ -401,7 +401,7 @@ init_readline(Mapi mid, char *lang, int 
                        case ENOENT:
                                /* history file didn't exist, so try to create
                                 * it and then try again */
-                               if ((f = fopen(_history_file, "w")) != NULL) {
+                               if ((f = fopen(_history_file, "w")) == NULL) {
                                        /* failed to create, don't
                                         * bother saving */
                                        _save_history = 0;
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to