Changeset: 251a9dca7b2c for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=251a9dca7b2c
Modified Files:
        clients/Tests/exports.stable.out
        monetdb5/optimizer/opt_pipes.c
        monetdb5/optimizer/opt_pipes.h
        sql/backends/monet5/sql.c
        sql/backends/monet5/sql_result.c
        sql/backends/monet5/sql_result.h
        sql/server/sql_env.c
        sql/server/sql_env.h
Branch: default
Log Message:

Use more const char *.


diffs (truncated from 458 to 300 lines):

diff --git a/clients/Tests/exports.stable.out b/clients/Tests/exports.stable.out
--- a/clients/Tests/exports.stable.out
+++ b/clients/Tests/exports.stable.out
@@ -1928,8 +1928,8 @@ str XMLxmltext(str *s, xml *x);
 str abortRef;
 str actionRef;
 void addMalException(MalBlkPtr mb, str msg);
-str addOptimizerPipe(Client cntxt, MalBlkPtr mb, str name);
-str addPipeDefinition(Client cntxt, str name, str pipe);
+str addOptimizerPipe(Client cntxt, MalBlkPtr mb, const char *name);
+str addPipeDefinition(Client cntxt, const char *name, const char *pipe);
 void addtoMalBlkHistory(MalBlkPtr mb);
 str affectedRowsRef;
 str aggrRef;
@@ -2016,7 +2016,7 @@ str columnBindRef;
 str columnRef;
 str commitRef;
 str compileAllOptimizers(Client cntxt);
-str compileOptimizer(Client cntxt, str name);
+str compileOptimizer(Client cntxt, const char *name);
 str compileString(Symbol *fcn, Client c, str s);
 str connectRef;
 str convertConstant(malType type, ValPtr vr);
@@ -2223,7 +2223,7 @@ int isMatLeftJoinOp(InstrPtr q);
 int isMultiplex(InstrPtr q);
 int isNotUsedIn(InstrPtr p, int start, int a);
 int isOptimizerEnabled(MalBlkPtr mb, str opt);
-int isOptimizerPipe(str name);
+int isOptimizerPipe(const char *name);
 int isOrderby(InstrPtr q);
 int isSample(InstrPtr q);
 int isSealedFunction(InstrPtr q);
diff --git a/monetdb5/optimizer/opt_pipes.c b/monetdb5/optimizer/opt_pipes.c
--- a/monetdb5/optimizer/opt_pipes.c
+++ b/monetdb5/optimizer/opt_pipes.c
@@ -241,7 +241,7 @@ optPipeInit(void)
 
 /* the session_pipe is the one defined by the user */
 str
-addPipeDefinition(Client cntxt, str name, str pipe)
+addPipeDefinition(Client cntxt, const char *name, const char *pipe)
 {
        int i;
        str msg;
@@ -292,7 +292,7 @@ addPipeDefinition(Client cntxt, str name
 }
 
 int
-isOptimizerPipe(str name)
+isOptimizerPipe(const char *name)
 {
        int i;
 
@@ -426,7 +426,7 @@ validateOptimizerPipes(void)
  * then copy the statements to the end of the MAL plan
 */
 str
-compileOptimizer(Client cntxt, str name)
+compileOptimizer(Client cntxt, const char *name)
 {
        int i, j;
        char buf[2048];
@@ -472,7 +472,7 @@ compileAllOptimizers(Client cntxt)
        return msg;
 }
 str
-addOptimizerPipe(Client cntxt, MalBlkPtr mb, str name)
+addOptimizerPipe(Client cntxt, MalBlkPtr mb, const char *name)
 {
        int i, j, k;
        InstrPtr p,q;
diff --git a/monetdb5/optimizer/opt_pipes.h b/monetdb5/optimizer/opt_pipes.h
--- a/monetdb5/optimizer/opt_pipes.h
+++ b/monetdb5/optimizer/opt_pipes.h
@@ -13,10 +13,10 @@
 
 mal_export str getPipeDefinition(str name);
 mal_export str getPipeCatalog(bat *nme, bat *def, bat *stat);
-mal_export str addPipeDefinition(Client cntxt, str name, str pipe);
-mal_export int isOptimizerPipe(str name);
-mal_export str addOptimizerPipe(Client cntxt, MalBlkPtr mb, str name);
-mal_export str compileOptimizer(Client cntxt, str name);
+mal_export str addPipeDefinition(Client cntxt, const char *name, const char 
*pipe);
+mal_export int isOptimizerPipe(const char *name);
+mal_export str addOptimizerPipe(Client cntxt, MalBlkPtr mb, const char *name);
+mal_export str compileOptimizer(Client cntxt, const char *name);
 mal_export str compileAllOptimizers(Client cntxt);
 mal_export void opt_pipes_reset(void);
 
diff --git a/sql/backends/monet5/sql.c b/sql/backends/monet5/sql.c
--- a/sql/backends/monet5/sql.c
+++ b/sql/backends/monet5/sql.c
@@ -533,7 +533,7 @@ setVariable(Client cntxt, MalBlkPtr mb, 
        int *res = getArgReference_int(stk, pci, 0);
        mvc *m = NULL;
        str msg;
-       str varname = *getArgReference_str(stk, pci, 2);
+       const char *varname = *getArgReference_str(stk, pci, 2);
        int mtype = getArgType(mb, pci, 3);
        ValRecord *src;
        char buf[BUFSIZ];
@@ -547,7 +547,7 @@ setVariable(Client cntxt, MalBlkPtr mb, 
        if (mtype < 0 || mtype >= 255)
                throw(SQL, "sql.setVariable", SQLSTATE(42100) "Variable type 
error");
        if (strcmp("optimizer", varname) == 0) {
-               str newopt = *getArgReference_str(stk, pci, 3);
+               const char *newopt = *getArgReference_str(stk, pci, 3);
                if (newopt) {
                        if (!isOptimizerPipe(newopt) && strchr(newopt, (int) 
';') == 0) {
                                throw(SQL, "sql.setVariable", SQLSTATE(42100) 
"optimizer '%s' unknown", newopt);
@@ -589,7 +589,7 @@ getVariable(Client cntxt, MalBlkPtr mb, 
        int mtype = getArgType(mb, pci, 0);
        mvc *m = NULL;
        str msg;
-       str varname = *getArgReference_str(stk, pci, 2);
+       const char *varname = *getArgReference_str(stk, pci, 2);
        atom *a;
        ValRecord *dst, *src;
 
@@ -646,7 +646,7 @@ mvc_logfile(Client cntxt, MalBlkPtr mb, 
 {
        mvc *m = NULL;
        str msg;
-       str filename = *getArgReference_str(stk, pci, 1);
+       const char *filename = *getArgReference_str(stk, pci, 1);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
                return msg;
@@ -670,8 +670,8 @@ mvc_next_value(Client cntxt, MalBlkPtr m
        str msg;
        sql_schema *s;
        lng *res = getArgReference_lng(stk, pci, 0);
-       str sname = *getArgReference_str(stk, pci, 1);
-       str seqname = *getArgReference_str(stk, pci, 2);
+       const char *sname = *getArgReference_str(stk, pci, 1);
+       const char *seqname = *getArgReference_str(stk, pci, 2);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
                return msg;
@@ -704,7 +704,7 @@ mvc_bat_next_value(Client cntxt, MalBlkP
        BATiter bi;
        bat *res = getArgReference_bat(stk, pci, 0);
        bat sid = *getArgReference_bat(stk, pci, 1);
-       str seqname = *getArgReference_str(stk, pci, 2);
+       const char *seqname = *getArgReference_str(stk, pci, 2);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
                return msg;
@@ -772,8 +772,8 @@ mvc_get_value(Client cntxt, MalBlkPtr mb
        str msg;
        sql_schema *s;
        lng *res = getArgReference_lng(stk, pci, 0);
-       str sname = *getArgReference_str(stk, pci, 1);
-       str seqname = *getArgReference_str(stk, pci, 2);
+       const char *sname = *getArgReference_str(stk, pci, 1);
+       const char *seqname = *getArgReference_str(stk, pci, 2);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
                return msg;
@@ -814,8 +814,8 @@ mvc_restart_seq(Client cntxt, MalBlkPtr 
        str msg;
        sql_schema *s;
        lng *res = getArgReference_lng(stk, pci, 0);
-       str sname = *getArgReference_str(stk, pci, 1);
-       str seqname = *getArgReference_str(stk, pci, 2);
+       const char *sname = *getArgReference_str(stk, pci, 1);
+       const char *seqname = *getArgReference_str(stk, pci, 2);
        lng start = *getArgReference_lng(stk, pci, 3);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
@@ -837,7 +837,7 @@ mvc_restart_seq(Client cntxt, MalBlkPtr 
 }
 
 static BAT *
-mvc_bind_dbat(mvc *m, char *sname, char *tname, int access)
+mvc_bind_dbat(mvc *m, const char *sname, const char *tname, int access)
 {
        sql_trans *tr = m->session->tr;
        BAT *b = NULL;
@@ -885,9 +885,9 @@ mvc_bind_wrap(Client cntxt, MalBlkPtr mb
        int coltype = getBatType(getArgType(mb, pci, 0));
        mvc *m = NULL;
        str msg;
-       str sname = *getArgReference_str(stk, pci, 2 + upd);
-       str tname = *getArgReference_str(stk, pci, 3 + upd);
-       str cname = *getArgReference_str(stk, pci, 4 + upd);
+       const char *sname = *getArgReference_str(stk, pci, 2 + upd);
+       const char *tname = *getArgReference_str(stk, pci, 3 + upd);
+       const char *cname = *getArgReference_str(stk, pci, 4 + upd);
        int access = *getArgReference_int(stk, pci, 5 + upd);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
@@ -992,9 +992,9 @@ mvc_bind_idxbat_wrap(Client cntxt, MalBl
        int coltype = getBatType(getArgType(mb, pci, 0));
        mvc *m = NULL;
        str msg;
-       str sname = *getArgReference_str(stk, pci, 2 + upd);
-       str tname = *getArgReference_str(stk, pci, 3 + upd);
-       str iname = *getArgReference_str(stk, pci, 4 + upd);
+       const char *sname = *getArgReference_str(stk, pci, 2 + upd);
+       const char *tname = *getArgReference_str(stk, pci, 3 + upd);
+       const char *iname = *getArgReference_str(stk, pci, 4 + upd);
        int access = *getArgReference_int(stk, pci, 5 + upd);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
@@ -1096,9 +1096,9 @@ mvc_append_wrap(Client cntxt, MalBlkPtr 
        int *res = getArgReference_int(stk, pci, 0);
        mvc *m = NULL;
        str msg;
-       str sname = *getArgReference_str(stk, pci, 2);
-       str tname = *getArgReference_str(stk, pci, 3);
-       str cname = *getArgReference_str(stk, pci, 4);
+       const char *sname = *getArgReference_str(stk, pci, 2);
+       const char *tname = *getArgReference_str(stk, pci, 3);
+       const char *cname = *getArgReference_str(stk, pci, 4);
        ptr ins = getArgReference(stk, pci, 5);
        int tpe = getArgType(mb, pci, 5);
        sql_schema *s;
@@ -1154,9 +1154,9 @@ mvc_update_wrap(Client cntxt, MalBlkPtr 
        int *res = getArgReference_int(stk, pci, 0);
        mvc *m = NULL;
        str msg;
-       str sname = *getArgReference_str(stk, pci, 2);
-       str tname = *getArgReference_str(stk, pci, 3);
-       str cname = *getArgReference_str(stk, pci, 4);
+       const char *sname = *getArgReference_str(stk, pci, 2);
+       const char *tname = *getArgReference_str(stk, pci, 3);
+       const char *cname = *getArgReference_str(stk, pci, 4);
        bat Tids = *getArgReference_bat(stk, pci, 5);
        bat Upd = *getArgReference_bat(stk, pci, 6);
        BAT *tids, *upd;
@@ -1222,8 +1222,8 @@ mvc_clear_table_wrap(Client cntxt, MalBl
        mvc *m = NULL;
        str msg;
        lng *res = getArgReference_lng(stk, pci, 0);
-       str sname = *getArgReference_str(stk, pci, 1);
-       str tname = *getArgReference_str(stk, pci, 2);
+       const char *sname = *getArgReference_str(stk, pci, 1);
+       const char *tname = *getArgReference_str(stk, pci, 2);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
                return msg;
@@ -1246,8 +1246,8 @@ mvc_delete_wrap(Client cntxt, MalBlkPtr 
        int *res = getArgReference_int(stk, pci, 0);
        mvc *m = NULL;
        str msg;
-       str sname = *getArgReference_str(stk, pci, 2);
-       str tname = *getArgReference_str(stk, pci, 3);
+       const char *sname = *getArgReference_str(stk, pci, 2);
+       const char *tname = *getArgReference_str(stk, pci, 3);
        ptr ins = getArgReference(stk, pci, 4);
        int tpe = getArgType(mb, pci, 4);
        BAT *b = NULL;
@@ -1755,8 +1755,8 @@ SQLtid(Client cntxt, MalBlkPtr mb, MalSt
        mvc *m = NULL;
        str msg;
        sql_trans *tr;
-       str sname = *getArgReference_str(stk, pci, 2);
-       str tname = *getArgReference_str(stk, pci, 3);
+       const char *sname = *getArgReference_str(stk, pci, 2);
+       const char *tname = *getArgReference_str(stk, pci, 3);
 
        sql_schema *s;
        sql_table *t;
@@ -1905,8 +1905,8 @@ str
 mvc_export_table_wrap( Client cntxt, MalBlkPtr mb, MalStkPtr stk, InstrPtr pci)
 {
        int *res_id =getArgReference_int(stk,pci,0);
-       str filename = *getArgReference_str(stk,pci,1);
-       str format = *getArgReference_str(stk,pci,2);
+       const char *filename = *getArgReference_str(stk,pci,1);
+       const char *format = *getArgReference_str(stk,pci,2);
        unsigned char *tsep = NULL, *rsep = NULL, *ssep = NULL, *ns = NULL;
        unsigned char *T = (unsigned char *) *getArgReference_str(stk, pci, 3);
        unsigned char *R = (unsigned char *) *getArgReference_str(stk, pci, 4);
@@ -2104,7 +2104,7 @@ mvc_export_row_wrap( Client cntxt, MalBl
 {
        int *res_id= getArgReference_int(stk, pci,0);
        str filename = * getArgReference_str(stk,pci,1);
-       str format = *getArgReference_str(stk,pci,2);
+       const char *format = *getArgReference_str(stk,pci,2);
        unsigned char *tsep = NULL, *rsep = NULL, *ssep = NULL, *ns = NULL;
        unsigned char *T = (unsigned char *) *getArgReference_str(stk, pci, 3);
        unsigned char *R = (unsigned char *) *getArgReference_str(stk, pci, 4);
@@ -2271,7 +2271,7 @@ mvc_declared_table_wrap(Client cntxt, Ma
        str msg;
        sql_schema *s = NULL;
        int *res_id = getArgReference_int(stk, pci, 0);
-       str name = *getArgReference_str(stk, pci, 1);
+       const char *name = *getArgReference_str(stk, pci, 1);
 
        if ((msg = getSQLContext(cntxt, mb, &m, NULL)) != NULL)
                return msg;
@@ -2296,9 +2296,9 @@ mvc_declared_table_column_wrap(Client cn
        sql_type *type = NULL;
        sql_subtype tpe;
        int rs = *getArgReference_int(stk, pci, 1);
-       str tname = *getArgReference_str(stk, pci, 2);
-       str name = *getArgReference_str(stk, pci, 3);
-       str typename = *getArgReference_str(stk, pci, 4);
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to