From: "n.pettik" <korablev@tarantool.org> To: tarantool-patches@freelists.org Cc: Kirill Shcherbatov <kshcherbatov@tarantool.org> Subject: [tarantool-patches] Re: [PATCH v1 1/1] sql: drop useless legacy Cursor hints Date: Tue, 19 Jun 2018 21:31:20 +0300 [thread overview] Message-ID: <0C053368-F010-48B0-9505-D57B9E7ABCA4@tarantool.org> (raw) In-Reply-To: <295e3ffa-2b83-916a-9741-9156c2108ece@tarantool.org> > On 19 Jun 2018, at 17:31, Kirill Shcherbatov <kshcherbatov@tarantool.org> wrote: > >> Could you please resend fresh diff after fixes, so that I review >> updated version. >> > > Deleted BTREE_BULKLOAD and OPFLAG_BULKCSR as there become Typo: there —> they. > useless since btree.c was dropped as a part of #2419. > Removed SQLITE_ENABLE_CURSOR_HINTS sections as this code > has never been in use. > Start use OPFLAG_SEEKEQ instead of equal BTREE_SEEK_EQ as > this macro name has no sense since no btree present in SQL. > > Resolves #3121. Except for your removals, I also suggest you to remove: SQLITE_CursorHints macros; sqlite3CursorHintFlags() function — it is used only once and can be inlined; sqlite3CursorHasHint() function — it is used only once and can be inlined;
next prev parent reply other threads:[~2018-06-19 18:31 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-06-09 13:09 [tarantool-patches] " Kirill Shcherbatov 2018-06-09 21:41 ` [tarantool-patches] " Vladislav Shpilevoy 2018-06-13 6:43 ` [tarantool-patches] " Kirill Shcherbatov 2018-06-13 8:48 ` Vladislav Shpilevoy 2018-06-19 9:41 ` Kirill Shcherbatov 2018-06-19 14:03 ` n.pettik 2018-06-19 14:31 ` Kirill Shcherbatov 2018-06-19 18:31 ` n.pettik [this message] 2018-06-20 6:58 ` Kirill Shcherbatov 2018-06-20 9:05 ` n.pettik 2018-06-22 12:11 ` Kirill Yukhin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=0C053368-F010-48B0-9505-D57B9E7ABCA4@tarantool.org \ --to=korablev@tarantool.org \ --cc=kshcherbatov@tarantool.org \ --cc=tarantool-patches@freelists.org \ --subject='[tarantool-patches] Re: [PATCH v1 1/1] sql: drop useless legacy Cursor hints' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox