X-Git-Url: https://git.notmuchmail.org/git?p=notmuch;a=blobdiff_plain;f=query.cc;h=c669fb9110a6fed7347ae454ddafa12fac3edbaa;hp=d3614dc3ee8d012ab5a868b03b096bda001c82a5;hb=f6c7810945f1bc25b15dee72257c3b68bd0e8a40;hpb=466a7bbf620e4bf1b57097a6d3c474159c475b6d

diff --git a/query.cc b/query.cc
index d3614dc3..c669fb91 100644
--- a/query.cc
+++ b/query.cc
@@ -66,8 +66,14 @@ notmuch_query_set_sort (notmuch_query_t *query, notmuch_sort_t sort)
     query->sort = sort;
 }
 
+/* We end up having to call the destructors explicitly because we had
+ * to use "placement new" in order to initialize C++ objects within a
+ * block that we allocated with talloc. So C++ is making talloc
+ * slightly less simple to use, (we wouldn't need
+ * talloc_set_destructor at all otherwise).
+ */
 static int
-_notmuch_results_destroy (notmuch_results_t *results)
+_notmuch_results_destructor (notmuch_results_t *results)
 {
     results->iterator.~PostingIterator ();
     results->iterator_end.~PostingIterator ();
@@ -94,7 +100,7 @@ notmuch_query_search (notmuch_query_t *query)
 	new (&results->iterator) Xapian::PostingIterator ();
 	new (&results->iterator_end) Xapian::PostingIterator ();
 
-	talloc_set_destructor (results, _notmuch_results_destroy);
+	talloc_set_destructor (results, _notmuch_results_destructor);
 
 	results->iterator = query->notmuch->xapian_db->postlist_begin ("");
 	results->iterator_end = query->notmuch->xapian_db->postlist_end ("");