[PATCH 0/3] API: add notes on lifetimes
rhn
nomumali.rhn at porcupinefactory.org
Sun Dec 23 02:08:29 PST 2018
On Mon, 17 Dec 2018 22:05:45 +0200
Tomi Ollila <tomi.ollila at iki.fi> wrote:
> On Mon, Dec 17 2018, rhn wrote:
>
> > Hi,
> >
> > this patch series addresses API shortcomings that were found while working on the Rust bindings [0].
> >
> > The first two patches address the problem that the docs never clearly state when messages obtained as replies are destroyed, while the last patch fixes abroken API example.
> >
> > Thanks for Dirk Van Haerenborgh for working out how long notmuch objects live.
>
> Looks good. Have to trust that change in example is correct (why would it
> not be?) I hope tests pass :D
>
> Tomi
The tests obviously pass, you can check them yourself. They are unfortunately prone to false negatives (e.g. passes when it shouldn't), because freed memory can still be sometimes accessed.
They could be detected more reliably if valgrind's memcheck was part of the test suite, but I think this patch still adds value.
I'm eagerly hoping for a merge!
Cheers,
rhn
>
> >
> > Cheers,
> > rhn
> >
> > [0] https://github.com/vhdirk/notmuch-rs
> >
> > rhn (3):
> > lib: Explicitly state when replies will be destroyed
> > test: Check for replies obeying lifetime guarantees
> > docs: Use correct call to notmuch_query_search_threads in usage
> > example
> >
> > lib/notmuch.h | 10 ++++-
> > test/T720-lib-lifetime.sh | 83 +++++++++++++++++++++++++++++++++++++++
> > 2 files changed, 92 insertions(+), 1 deletion(-)
> > create mode 100755 test/T720-lib-lifetime.sh
> >
> > --
> > 2.17.2
> >
> > _______________________________________________
> > notmuch mailing list
> > notmuch at notmuchmail.org
> > https://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list