[Tarantool-patches] [PATCH v4 23/53] sql: introduce mem_set_null()
imeevma at tarantool.org
imeevma at tarantool.org
Tue Mar 23 12:35:46 MSK 2021
This patch introduces mem_set_null(). Function mem_set_null() clears MEM
and sets it to NULL.
Part of #5818
---
src/box/sql/func.c | 4 ++--
src/box/sql/mem.c | 36 +++++++++---------------------------
src/box/sql/mem.h | 6 +++---
src/box/sql/vdbe.c | 14 ++++++--------
src/box/sql/vdbeapi.c | 4 ++--
5 files changed, 22 insertions(+), 42 deletions(-)
diff --git a/src/box/sql/func.c b/src/box/sql/func.c
index 6b6081150..b61de18d8 100644
--- a/src/box/sql/func.c
+++ b/src/box/sql/func.c
@@ -290,7 +290,7 @@ port_lua_get_vdbemem(struct port *base, uint32_t *size)
goto error;
break;
case MP_NIL:
- sqlVdbeMemSetNull(&val[i]);
+ mem_set_null(&val[i]);
break;
default:
diag_set(ClientError, ER_SQL_EXECUTE,
@@ -361,7 +361,7 @@ port_c_get_vdbemem(struct port *base, uint32_t *size)
goto error;
break;
case MP_NIL:
- sqlVdbeMemSetNull(&val[i]);
+ mem_set_null(&val[i]);
break;
default:
diag_set(ClientError, ER_SQL_EXECUTE,
diff --git a/src/box/sql/mem.c b/src/box/sql/mem.c
index c84b3775d..da2aa5c94 100644
--- a/src/box/sql/mem.c
+++ b/src/box/sql/mem.c
@@ -257,6 +257,12 @@ mem_destroy(struct Mem *mem)
mem->zMalloc = NULL;
}
+void
+mem_set_null(struct Mem *mem)
+{
+ mem_clear(mem);
+}
+
int
mem_copy(struct Mem *to, const struct Mem *from)
{
@@ -323,7 +329,7 @@ int
mem_concat(struct Mem *left, struct Mem *right, struct Mem *result)
{
assert(result != right);
- sqlVdbeMemSetNull(result);
+ mem_clear(result);
result->field_type = field_type_MAX;
if (((left->flags | right->flags) & MEM_Null) != 0) {
@@ -374,7 +380,7 @@ int
mem_arithmetic(const struct Mem *left, const struct Mem *right,
struct Mem *result, int op)
{
- sqlVdbeMemSetNull(result);
+ mem_clear(result);
result->field_type = FIELD_TYPE_NUMBER;
if (((left->flags | right->flags) & MEM_Null) != 0)
return 0;
@@ -559,7 +565,7 @@ mem_arithmetic(const struct Mem *left, const struct Mem *right,
int
mem_bitwise(struct Mem *left, struct Mem *right, struct Mem *result, int op)
{
- sqlVdbeMemSetNull(result);
+ mem_clear(result);
result->field_type = FIELD_TYPE_INTEGER;
if (((left->flags | right->flags) & MEM_Null) != 0)
return 0;
@@ -2033,24 +2039,6 @@ sqlVdbeMemSetStr(Mem * pMem, /* Memory cell to set to string value */
return 0;
}
-/*
- * Delete any previous value and set the value stored in *pMem to NULL.
- *
- * This routine calls the Mem.xDel destructor to dispose of values that
- * require the destructor. But it preserves the Mem.zMalloc memory allocation.
- * To free all resources, use mem_destroy(), which both calls this
- * routine to invoke the destructor and deallocates Mem.zMalloc.
- *
- * Use this routine to reset the Mem prior to insert a new value.
- *
- * Use mem_destroy() to complete erase the Mem prior to abandoning it.
- */
-void
-sqlVdbeMemSetNull(Mem * pMem)
-{
- mem_clear(pMem);
-}
-
/*
* Delete any previous value and set the value to be a BLOB of length
* n containing all zeros.
@@ -2081,12 +2069,6 @@ sqlValueSetStr(sql_value * v, /* Value to be set */
sqlVdbeMemSetStr((Mem *) v, z, n, 1, xDel);
}
-void
-sqlValueSetNull(sql_value * p)
-{
- sqlVdbeMemSetNull((Mem *) p);
-}
-
/*
* Free an sql_value object
*/
diff --git a/src/box/sql/mem.h b/src/box/sql/mem.h
index 3561b49b1..dc0d4a284 100644
--- a/src/box/sql/mem.h
+++ b/src/box/sql/mem.h
@@ -163,6 +163,9 @@ mem_create(struct Mem *mem);
void
mem_destroy(struct Mem *mem);
+void
+mem_set_null(struct Mem *mem);
+
/**
* Copy content of MEM from one MEM to another. In case source MEM contains
* string or binary and allocation type is not STATIC, this value is copied to
@@ -431,12 +434,9 @@ mem_set_double(struct Mem *mem, double value);
int
sqlVdbeMemSetStr(struct Mem *, const char *, int, u8, void (*)(void *));
void
-sqlVdbeMemSetNull(struct Mem *);
-void
sqlVdbeMemSetZeroBlob(struct Mem *, int);
void sqlValueSetStr(struct Mem *, int, const void *,
void (*)(void *));
-void sqlValueSetNull(struct Mem *);
void sqlValueFree(struct Mem *);
struct Mem *sqlValueNew(struct sql *);
diff --git a/src/box/sql/vdbe.c b/src/box/sql/vdbe.c
index 80497b7b2..f0b56033a 100644
--- a/src/box/sql/vdbe.c
+++ b/src/box/sql/vdbe.c
@@ -244,7 +244,7 @@ vdbe_prepare_null_out(struct Vdbe *v, int n)
assert(n <= (v->nMem + 1 - v->nCursor));
struct Mem *out = &v->aMem[n];
memAboutToChange(v, out);
- sqlVdbeMemSetNull(out);
+ mem_set_null(out);
out->field_type = field_type_MAX;
return out;
}
@@ -830,19 +830,17 @@ case OP_String: { /* out2 */
*/
case OP_Null: { /* out2 */
int cnt;
- u16 nullFlag;
pOut = vdbe_prepare_null_out(p, pOp->p2);
cnt = pOp->p3-pOp->p2;
assert(pOp->p3<=(p->nMem+1 - p->nCursor));
- pOut->flags = nullFlag = pOp->p1 ? (MEM_Null|MEM_Cleared) : MEM_Null;
- pOut->n = 0;
+ if (pOp->p1 != 0)
+ pOut->flags = MEM_Null|MEM_Cleared;
while( cnt>0) {
pOut++;
memAboutToChange(p, pOut);
- sqlVdbeMemSetNull(pOut);
- pOut->flags = nullFlag;
- pOut->field_type = field_type_MAX;
- pOut->n = 0;
+ mem_set_null(pOut);
+ if (pOp->p1 != 0)
+ pOut->flags = MEM_Null|MEM_Cleared;
cnt--;
}
break;
diff --git a/src/box/sql/vdbeapi.c b/src/box/sql/vdbeapi.c
index 04016ea54..76af333e8 100644
--- a/src/box/sql/vdbeapi.c
+++ b/src/box/sql/vdbeapi.c
@@ -201,7 +201,7 @@ sql_result_bool(struct sql_context *ctx, bool value)
void
sql_result_null(sql_context * pCtx)
{
- sqlVdbeMemSetNull(pCtx->pOut);
+ mem_set_null(pCtx->pOut);
}
void
@@ -382,7 +382,7 @@ createAggContext(sql_context * p, int nByte)
Mem *pMem = p->pMem;
assert(!mem_is_aggregate(pMem));
if (nByte <= 0) {
- sqlVdbeMemSetNull(pMem);
+ mem_set_null(pMem);
pMem->z = 0;
} else {
sqlVdbeMemClearAndResize(pMem, nByte);
--
2.25.1
More information about the Tarantool-patches
mailing list