On Mon, Aug 25, 2025 at 4:54 AM Dominique Devienne via Boost < boost@lists.boost.org> wrote:
On Mon, Aug 25, 2025 at 6:53 AM Mohammad Nejati [ashtum] via Boost <boost@lists.boost.org> wrote:
This is the second review attempt. The first concluded without a result due to too few submitted reviews.
Hi. Could we please have a summary a changes, if any, compared to the 1st review? Any review feedback incorporated?
For the first change I made a mistake: I replaced the doxygen docs with sqlite and I forgot to update the link: it's currently at http://klemens.dev/sqlite-new/ . Mohammad, can I update the link during the review? Otherwise - bunch of fixes, e.g. missing default-constructor. - json support is header only, so it is not a dependency for compiling sqlite. - static_result_set has a strict mode, where the columns must match exactly. In non-strict mode the struct can have extra fields. - a sqlite::connection can be constructed from a filesystem::path - transaction destructors are only not noexcept for very old sqlite versions - added a delete_function function - the aggregate and window functions are stateful (i.e. the type gets constructed when the function is used) ( https://github.com/klemens-morgenstern/sqlite/pull/4/commits/c29ec35717fcc61... )
_______________________________________________ Boost mailing list -- boost@lists.boost.org To unsubscribe send an email to boost-leave@lists.boost.org https://lists.boost.org/mailman3/lists/boost.lists.boost.org/ Archived at: https://lists.boost.org/archives/list/boost@lists.boost.org/message/TDMWGPUX...