From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp33.i.mail.ru (smtp33.i.mail.ru [94.100.177.93]) (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 8EBC44696C4 for ; Fri, 22 Nov 2019 05:47:13 +0300 (MSK) From: Ilya Kosarev Date: Fri, 22 Nov 2019 05:46:54 +0300 Message-Id: <0a91d03b085e9850329480321c857ba628bda915.1574390065.git.i.kosarev@tarantool.org> In-Reply-To: References: In-Reply-To: References: Subject: [Tarantool-patches] [PATCH v5 6/8] refactoring: update comment for index_def_check_tuple List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: tarantool-patches@dev.tarantool.org Originally index_def_check_tuple comment said that it throws a nice error. Since refactoring: remove exceptions from index_def_new_from_tuple (90ac0037f6cae587d8ca589dbe45c58e67f05657) it returns an error. Now it is clearly specified in the comment. Part of #4247 --- src/box/alter.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/box/alter.cc b/src/box/alter.cc index 4b3bbca31..684b93798 100644 --- a/src/box/alter.cc +++ b/src/box/alter.cc @@ -159,7 +159,7 @@ index_def_check_sequence(struct index_def *index_def, uint32_t sequence_fieldno, /** * Support function for index_def_new_from_tuple(..) - * Checks tuple (of _index space) and sets a nice diag if it is invalid + * Checks tuple (of _index space) and returns an error if it is invalid * Checks only types of fields and their count! */ static int -- 2.17.1