[tarantool-patches] Re: [PATCH v1 16/28] sql: remove SQL_TOOBIG errcode
Vladislav Shpilevoy
v.shpilevoy at tarantool.org
Fri Jun 14 01:24:51 MSK 2019
Thanks for the patch!
Consider my review fixes below and on the branch
in a separate commit.
======================================================
diff --git a/src/box/sql/vdbeapi.c b/src/box/sql/vdbeapi.c
index ad8684dda..e41621049 100644
--- a/src/box/sql/vdbeapi.c
+++ b/src/box/sql/vdbeapi.c
@@ -274,14 +274,14 @@ setResultStrOrError(sql_context * pCtx, /* Function context */
void (*xDel) (void *) /* Destructor function */
)
{
- if (sqlVdbeMemSetStr(pCtx->pOut, z, n,1, xDel) != 0)
+ if (sqlVdbeMemSetStr(pCtx->pOut, z, n, 1, xDel) != 0)
pCtx->is_aborted = true;
}
static int
invokeValueDestructor(const void *p, /* Value to destroy */
void (*xDel) (void *), /* The destructor */
- sql_context * pCtx /* Set an error if no NULL */
+ sql_context *pCtx /* Set an error if no NULL */
)
{
assert(xDel != SQL_DYNAMIC);
@@ -306,7 +306,7 @@ sql_result_blob(sql_context * pCtx,
)
{
assert(n >= 0);
- if (sqlVdbeMemSetStr(pCtx->pOut, z, n,0, xDel) != 0)
+ if (sqlVdbeMemSetStr(pCtx->pOut, z, n, 0, xDel) != 0)
pCtx->is_aborted = true;
}
More information about the Tarantool-patches
mailing list