From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtpng3.m.smailru.net (smtpng3.m.smailru.net [94.100.177.149]) (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 1635246971A for ; Fri, 13 Dec 2019 16:55:56 +0300 (MSK) Date: Fri, 13 Dec 2019 16:55:55 +0300 From: Nikita Pettik Message-ID: <20191213135555.GA80915@tarantool.org> References: <20191204115517.GK6592@atlas> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20191204115517.GK6592@atlas> Subject: Re: [Tarantool-patches] [PATCH v2 09/16] sql: resurrect sql_bind_parameter_name() List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Konstantin Osipov , tarantool-patches@dev.tarantool.org, v.shpilevoy@tarantool.org On 04 Dec 14:55, Konstantin Osipov wrote: > * Nikita Pettik [19/11/21 10:00]: > > +/** > > + * Return the name of a wildcard parameter. Return NULL if the index > > + * is out of range or if the wildcard is unnamed. > > + */ > > Please document the valid range of the index (0-based I assume) > > +const char * > > +sql_bind_parameter_name(sql_stmt *stmt, int i); > > Fixed: diff --git a/src/box/sql/sqlInt.h b/src/box/sql/sqlInt.h index e5092b7a4..a5c564b8a 100644 --- a/src/box/sql/sqlInt.h +++ b/src/box/sql/sqlInt.h @@ -697,7 +697,8 @@ sql_bind_parameter_count(sql_stmt *stmt); /** * Return the name of a wildcard parameter. Return NULL if the index - * is out of range or if the wildcard is unnamed. + * is out of range or if the wildcard is unnamed. Parameter's index + * is 0-based. */ const char * sql_bind_parameter_name(sql_stmt *stmt, int i); > -- > Konstantin Osipov, Moscow, Russia