Changeset: 1fb57cae60dc for MonetDB URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=1fb57cae60dc Modified Files: clients/mapiclient/mclient.c Branch: Mar2018 Log Message:
mclient: centralize the decision which output formatter echos queries, if requested; mainly for ease of code overview and maintenance diffs (76 lines): diff --git a/clients/mapiclient/mclient.c b/clients/mapiclient/mclient.c --- a/clients/mapiclient/mclient.c +++ b/clients/mapiclient/mclient.c @@ -1131,7 +1131,6 @@ TESTrenderer(MapiHdl hdl) char *sep; int i; - SQLqueryEcho(hdl); while (!mnstr_errnr(toConsole) && (reply = fetch_line(hdl)) != 0) { if (*reply != '[') { if (*reply == '=') @@ -1271,7 +1270,6 @@ RAWrenderer(MapiHdl hdl) { char *line; - SQLqueryEcho(hdl); while ((line = fetch_line(hdl)) != 0) { if (*line == '=') line++; @@ -1359,7 +1357,6 @@ SAMrenderer(MapiHdl hdl) static void SQLheader(MapiHdl hdl, int *len, int fields, char more) { - SQLqueryEcho(hdl); SQLseparator(len, fields, '-'); if (mapi_get_name(hdl, 0)) { int i; @@ -1864,6 +1861,7 @@ format_result(Mapi mid, MapiHdl hdl, int } mnstr_printf(toConsole, "\n"); } + /* select which formatters show timing info (if requested) */ switch (formatter) { /* these formatters never produce timing output */ @@ -1887,6 +1885,7 @@ format_result(Mapi mid, MapiHdl hdl, int if (formatter == TABLEformatter) { mnstr_printf(toConsole, "operation successful\n"); } + /* select which formatters show timing info (if requested) */ switch (formatter) { /* these formatters never produce timing output */ @@ -1957,6 +1956,24 @@ format_result(Mapi mid, MapiHdl hdl, int if (debugMode()) RAWrenderer(hdl); else { + /* select which formatters echo the query (if requested) */ + switch (formatter) { + /* these formatters never echo the query */ + case NOformatter: + case CSVformatter: + case XMLformatter: + case SAMformatter: + case TRASHformatter: + case EXPANDEDformatter: + break; + /* these formatters echo the query when requested */ + case RAWformatter: + case TESTformatter: + case TABLEformatter: + SQLqueryEcho(hdl); + break; + } + switch (formatter) { case TRASHformatter: break; @@ -1989,6 +2006,7 @@ format_result(Mapi mid, MapiHdl hdl, int RAWrenderer(hdl); break; } + /* select which formatters show timing info (if requested) */ switch (formatter) { /* these formatters never produce timing output */ _______________________________________________ checkin-list mailing list checkin-list@monetdb.org https://www.monetdb.org/mailman/listinfo/checkin-list