[Tarantool-patches] [PATCH v3 06/20] sql: rename sql_reset() to sql_stmt_reset()

Sergey Ostanevich sergos at tarantool.org
Tue Dec 24 15:09:03 MSK 2019


Hi!

10x for the patch, LGTM.

Sergos


On 20 Dec 15:47, Nikita Pettik wrote:
> ---
>  src/box/ck_constraint.c | 2 +-
>  src/box/sql/sqlInt.h    | 2 +-
>  src/box/sql/vdbe.c      | 2 +-
>  src/box/sql/vdbeapi.c   | 4 ++--
>  4 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/src/box/ck_constraint.c b/src/box/ck_constraint.c
> index 551bdd397..bc2a5e8f4 100644
> --- a/src/box/ck_constraint.c
> +++ b/src/box/ck_constraint.c
> @@ -173,7 +173,7 @@ ck_constraint_program_run(struct ck_constraint *ck_constraint,
>  	 * Get VDBE execution state and reset VM to run it
>  	 * next time.
>  	 */
> -	return sql_reset(ck_constraint->stmt);
> +	return sql_stmt_reset(ck_constraint->stmt);
>  }
>  
>  int
> diff --git a/src/box/sql/sqlInt.h b/src/box/sql/sqlInt.h
> index cf0b946f1..b1e4ac2fa 100644
> --- a/src/box/sql/sqlInt.h
> +++ b/src/box/sql/sqlInt.h
> @@ -532,7 +532,7 @@ sql_stmt_finalize(sql_stmt * pStmt);
>   * @retval sql_ret_code Error code on error.
>   */
>  int
> -sql_reset(struct sql_stmt *stmt);
> +sql_stmt_reset(struct sql_stmt *stmt);
>  
>  int
>  sql_exec(sql *,	/* An open database */
> diff --git a/src/box/sql/vdbe.c b/src/box/sql/vdbe.c
> index 336fd4a52..8da007b4d 100644
> --- a/src/box/sql/vdbe.c
> +++ b/src/box/sql/vdbe.c
> @@ -1084,7 +1084,7 @@ case OP_Yield: {            /* in1, jump */
>   * automatically.
>   *
>   * P1 is the result code returned by sql_exec(),
> - * sql_reset(), or sql_stmt_finalize().  For a normal halt,
> + * sql_stmt_reset(), or sql_stmt_finalize().  For a normal halt,
>   * this should be 0.
>   * For errors, it can be some other value.  If P1!=0 then P2 will
>   * determine whether or not to rollback the current transaction.
> diff --git a/src/box/sql/vdbeapi.c b/src/box/sql/vdbeapi.c
> index 7463fb9d7..b6bf9aa81 100644
> --- a/src/box/sql/vdbeapi.c
> +++ b/src/box/sql/vdbeapi.c
> @@ -88,7 +88,7 @@ sql_stmt_finalize(sql_stmt * pStmt)
>  }
>  
>  int
> -sql_reset(sql_stmt * pStmt)
> +sql_stmt_reset(sql_stmt *pStmt)
>  {
>  	assert(pStmt != NULL);
>  	struct Vdbe *v = (Vdbe *) pStmt;
> @@ -414,7 +414,7 @@ sqlStep(Vdbe * p)
>  
>  	assert(p);
>  	if (p->magic != VDBE_MAGIC_RUN)
> -		sql_reset((sql_stmt *) p);
> +		sql_stmt_reset((sql_stmt *) p);
>  
>  	/* Check that malloc() has not failed. If it has, return early. */
>  	db = p->db;
> -- 
> 2.15.1
> 


More information about the Tarantool-patches mailing list