From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <tarantool-patches-bounce@freelists.org>
Received: from localhost (localhost [127.0.0.1])
	by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id 4D2062682C
	for <tarantool-patches@freelists.org>; Tue, 26 Feb 2019 13:12:52 -0500 (EST)
Received: from turing.freelists.org ([127.0.0.1])
	by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id ke8rCfEqH5u3 for <tarantool-patches@freelists.org>;
	Tue, 26 Feb 2019 13:12:52 -0500 (EST)
Received: from smtpng2.m.smailru.net (smtpng2.m.smailru.net [94.100.179.3])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(No client certificate requested)
	by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id 01E83267F9
	for <tarantool-patches@freelists.org>; Tue, 26 Feb 2019 13:12:51 -0500 (EST)
Content-Type: text/plain;
	charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 12.2 \(3445.102.3\))
Subject: [tarantool-patches] Re: [PATCH v1 1/1] sql: remove SQL_*_BKPT macros
From: "n.pettik" <korablev@tarantool.org>
In-Reply-To: <efdf9612620d7f1741bbe0a5c3ff585dbcb269dc.1551192665.git.imeevma@gmail.com>
Date: Tue, 26 Feb 2019 21:12:49 +0300
Content-Transfer-Encoding: quoted-printable
Message-Id: <BA6EB4E5-B6C7-4628-8D07-96945A6E98E3@tarantool.org>
References: <efdf9612620d7f1741bbe0a5c3ff585dbcb269dc.1551192665.git.imeevma@gmail.com>
Sender: tarantool-patches-bounce@freelists.org
Errors-to: tarantool-patches-bounce@freelists.org
Reply-To: tarantool-patches@freelists.org
List-help: <mailto:ecartis@freelists.org?Subject=help>
List-unsubscribe: <tarantool-patches-request@freelists.org?Subject=unsubscribe>
List-software: Ecartis version 1.0.0
List-Id: tarantool-patches <tarantool-patches.freelists.org>
List-subscribe: <tarantool-patches-request@freelists.org?Subject=subscribe>
List-owner: <mailto:>
List-post: <mailto:tarantool-patches@freelists.org>
List-archive: <http://www.freelists.org/archives/tarantool-patches>
To: tarantool-patches@freelists.org
Cc: Imeev Mergen <imeevma@tarantool.org>



> On 26 Feb 2019, at 17:52, imeevma@tarantool.org wrote:
>=20
> Macros SQL_*_BKPT were used in debug mode to log errors that
> occurred during the execution of VDBE. They are not used now.
> This patch removes them.
>=20
> Part of #3965
> ---
> https://github.com/tarantool/tarantool/issues/3965
> =
https://github.com/tarantool/tarantool/tree/imeevma/gh-3965-rework-errors-=
in-vdbe
>=20
>=20
> @@ -652,7 +652,7 @@ sql_progress_handler(sql * db,
> {
> #ifdef SQL_ENABLE_API_ARMOR
> 	if (!sqlSafetyCheckOk(db)) {
> -		(void)SQL_MISUSE_BKPT;
> +		(void)SQL_MISUSE;

I=E2=80=99ve removed things like this and pushed updated branch.
LGTM.