From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtpng1.m.smailru.net (smtpng1.m.smailru.net [94.100.181.251]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id C25344696C7 for ; Thu, 21 Nov 2019 00:28:25 +0300 (MSK) From: Nikita Pettik Date: Thu, 21 Nov 2019 00:28:07 +0300 Message-Id: <47387272c9d4e681beb98cbaea997d8300a17474.1574277369.git.korablev@tarantool.org> In-Reply-To: References: In-Reply-To: References: Subject: [Tarantool-patches] [PATCH v2 08/16] sql: resurrect sql_bind_parameter_count() function List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: tarantool-patches@dev.tarantool.org Cc: v.shpilevoy@tarantool.org This function is present in sql/vdbeapi.c source file, its prototype is missing in any header file. It makes impossible to use it. Let's add prototype declaration to sql/sqlInt.h (as other parameter setters/getters) and refactor a bit in accordance with our codestyle. Need for #2592 --- src/box/sql/sqlInt.h | 6 ++++++ src/box/sql/vdbeapi.c | 10 +++------- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/src/box/sql/sqlInt.h b/src/box/sql/sqlInt.h index bd0dca703..875efd6e3 100644 --- a/src/box/sql/sqlInt.h +++ b/src/box/sql/sqlInt.h @@ -688,6 +688,12 @@ int sql_bind_zeroblob64(sql_stmt *, int, sql_uint64); +/** + * Return the number of wildcards that should be bound to. + */ +int +sql_bind_parameter_count(sql_stmt *stmt); + /** * Perform pointer parameter binding for the prepared sql * statement. diff --git a/src/box/sql/vdbeapi.c b/src/box/sql/vdbeapi.c index db7936e78..11f05786c 100644 --- a/src/box/sql/vdbeapi.c +++ b/src/box/sql/vdbeapi.c @@ -1051,15 +1051,11 @@ sql_bind_zeroblob64(sql_stmt * pStmt, int i, sql_uint64 n) return sql_bind_zeroblob(pStmt, i, n); } -/* - * Return the number of wildcards that can be potentially bound to. - * This routine is added to support DBD::sql. - */ int -sql_bind_parameter_count(sql_stmt * pStmt) +sql_bind_parameter_count(sql_stmt *stmt) { - Vdbe *p = (Vdbe *) pStmt; - return p ? p->nVar : 0; + struct Vdbe *p = (struct Vdbe *) stmt; + return p->nVar; } /* -- 2.15.1