Changeset: 68f5a5eb77ef for MonetDB URL: https://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=68f5a5eb77ef Modified Files: sql/backends/monet5/sql_time.c Branch: clean-candidates Log Message:
Clean code to make it compile diffs (97 lines): diff --git a/sql/backends/monet5/sql_time.c b/sql/backends/monet5/sql_time.c --- a/sql/backends/monet5/sql_time.c +++ b/sql/backends/monet5/sql_time.c @@ -1093,50 +1093,49 @@ bailout: oid off = b->hseqbase; \ TPE_IN *restrict vals = Tloc(b, 0); \ if (ci.tpe == cand_dense) { \ - for (BUN i = 0 ; i < q && !msg ; i++) { \ + for (BUN i = 0; i < q; i++) { \ oid p = (canditer_next_dense(&ci) - off); \ TPE_IN next = vals[p]; \ if (is_##TPE_IN##_nil(next)) { \ ret[i] = TPE_OUT##_nil; \ nils = true; \ } else { \ - FUNC(ret[i], next, TPE_IN, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION); \ + FUNC(ret[i], TPE_IN, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION); \ } \ } \ } else { \ - for (BUN i = 0 ; i < q && !msg ; i++) { \ + for (BUN i = 0; i < q; i++) { \ oid p = (canditer_next(&ci) - off); \ TPE_IN next = vals[p]; \ if (is_##TPE_IN##_nil(next)) { \ ret[i] = TPE_OUT##_nil; \ nils = true; \ } else { \ - FUNC(ret[i], next, TPE_IN, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION); \ + FUNC(ret[i], TPE_IN, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION); \ } \ } \ } \ } else { \ - TPE_IN val = *(TPE_IN*)getArgReference(stk, pci, 1); \ - FUNC(*ret, val, TPE_IN, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION); \ + TPE_IN next = *(TPE_IN*)getArgReference(stk, pci, 1); \ + FUNC(*ret, TPE_IN, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION); \ } \ } while(0) -#define month_interval_convert(OUT, IN, TPE, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION) \ +#define month_interval_convert(OUT, TPE_IN, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION) \ do { \ - TPE next = IN; \ int cast, r; \ - CAST_VALIDATION(TPE, FUNC_NAME, MAX_VALUE); \ + CAST_VALIDATION(TPE_IN, FUNC_NAME, MAX_VALUE); \ cast = (int) next; \ r = cast * multiplier; \ - MUL_VALIDATION(TPE, FUNC_NAME, MAX_VALUE); \ + MUL_VALIDATION(TPE_IN, FUNC_NAME, MAX_VALUE); \ OUT = r; \ } while (0) -#define DO_NOTHING(TPE, FUNC_NAME, MAX_VALUE) ; +#define DO_NOTHING(TPE_IN, FUNC_NAME, MAX_VALUE) ; -#define CAST_VALIDATION(TPE, FUNC_NAME, MAX_VALUE) \ +#define CAST_VALIDATION(TPE_IN, FUNC_NAME, MAX_VALUE) \ do { \ - if (next > (TPE) MAX_VALUE) { \ + if (next > (TPE_IN) MAX_VALUE) { \ size_t len = 0; \ char *str_val = NULL; \ if (BATatoms[tpe].atomToStr(&str_val, &len, &next, false) < 0) { \ @@ -1149,7 +1148,7 @@ bailout: } \ } while (0) -#define MUL_OVERFLOW(TPE, FUNC_NAME, MAX_VALUE) /* MAX_VALUE is ignored on this macro */ \ +#define MUL_OVERFLOW(TPE_IN, FUNC_NAME, MAX_VALUE) /* TPE_IN and MAX_VALUE are ignored on this macro */ \ do { \ if (r < cast) { \ size_t len = 0; \ @@ -1252,18 +1251,17 @@ bailout: return msg; } -#define second_interval_convert(OUT, IN, TPE, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION) \ +#define second_interval_convert(OUT, TPE_IN, FUNC_NAME, MAX_VALUE, CAST_VALIDATION, MUL_VALIDATION) \ do { \ - TPE next = IN; \ lng cast, r; \ - CAST_VALIDATION(TPE, FUNC_NAME, MAX_VALUE); \ + CAST_VALIDATION(TPE_IN, FUNC_NAME, MAX_VALUE); \ cast = (lng) next; \ r = cast * multiplier; \ if (scale) { \ r += shift; \ r /= divider; \ } \ - MUL_VALIDATION(TPE, FUNC_NAME, MAX_VALUE); \ + MUL_VALIDATION(TPE_IN, FUNC_NAME, MAX_VALUE); \ OUT = r; \ } while (0) _______________________________________________ checkin-list mailing list checkin-list@monetdb.org https://www.monetdb.org/mailman/listinfo/checkin-list