Re: [review] [sqlite]

From: Ruben Perez (rubenperez038_at_[hidden])
Date: 2024-11-16 10:47:15

What's the current recommended way to consume the library? Installing
it with CMake seems to only install the binaries (and not the
headers/cmake find_package scripts), and I haven't found a Jamfile to
build the library as part of the Boost superproject.

Thanks,
Ruben.

On Sat, 16 Nov 2024 at 10:18, Ruben Perez <rubenperez038_at_[hidden]> wrote:
>
> I have some questions for the author:
>
> 1. Many functions are overloaded to accept an error_code and
> error_info, effectively having throwing and non-throwing overloads.
> Have you considered unifying these by using system::result?
> 2. What compilers and C++ standards are supported by the library? It
> looks like C++14, but docs don't say it.
> 3.  What's the rationale behind BOOST_SQLITE_NO_VIRTUAL? Wouldn't
> setting it cause trouble, being a compiled library?
> 4. I'm interested in the use case of exposing an in-memory data
> structure through a virtual table. I'm looking at multi_index.cpp, the
> Doxygen comments and the official sqlite docs on virtual tables. Is
> there any additional docs on the topic that can help me?
> 5. Docs for field::get_text() state "Returns the value as text, i.e. a
> string_view. Note that this value may be invalidated" - what does this
> mean?
>
> Thanks,
> Ruben.