<p dir="ltr">The point was, we can do this *now* without soname bumps, because we haven't released this yet.<br>
</p>
<div class="gmail_quote">On Nov 2, 2013 1:16 AM,  <<a href="mailto:david@tethera.net">david@tethera.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
From: David Bremner <<a href="mailto:bremner@debian.org">bremner@debian.org</a>><br>
<br>
Compact was missing callback documentation entirely, and upgrade did not discuss the<br>
closure parameter.<br>
---<br>
This patch depends on<br>
<br>
     <a href="mailto:id%3A2a58adbdc1257f16579692544b4bcbadca3d3045.1383315568.git.jani@nikula.org">id:2a58adbdc1257f16579692544b4bcbadca3d3045.1383315568.git.jani@nikula.org</a><br>
<br>
BTW, I didn't completely understand the remark about SONAME bumps;<br>
since we're providing new symbols, it doesn't really matter what the<br>
signature is?<br>
<br>
 lib/notmuch.h | 6 +++++-<br>
 1 file changed, 5 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/lib/notmuch.h b/lib/notmuch.h<br>
index cd301a4..82fd599 100644<br>
--- a/lib/notmuch.h<br>
+++ b/lib/notmuch.h<br>
@@ -227,6 +227,9 @@ typedef void (*notmuch_compact_status_cb_t)(const char *message, void *closure);<br>
  * The database will be opened with NOTMUCH_DATABASE_MODE_READ_WRITE<br>
  * during the compaction process to ensure no writes are made.<br>
  *<br>
+ * If the optional callback function 'status_cb' is non-NULL, it will<br>
+ * be called with diagnostic and informational messages. The argument<br>
+ * 'closure' is passed verbatim to any callback invoked.<br>
  */<br>
 notmuch_status_t<br>
 notmuch_database_compact (const char* path,<br>
@@ -270,7 +273,8 @@ notmuch_database_needs_upgrade (notmuch_database_t *database);<br>
  * provide progress indication to the user. If non-NULL it will be<br>
  * called periodically with 'progress' as a floating-point value in<br>
  * the range of [0.0 .. 1.0] indicating the progress made so far in<br>
- * the upgrade process.<br>
+ * the upgrade process.  The argument 'closure' is passed verbatim to<br>
+ * any callback invoked.<br>
  */<br>
 notmuch_status_t<br>
 notmuch_database_upgrade (notmuch_database_t *database,<br>
--<br>
1.8.4.rc3<br>
<br>
_______________________________________________<br>
notmuch mailing list<br>
<a href="mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.org</a><br>
<a href="http://notmuchmail.org/mailman/listinfo/notmuch" target="_blank">http://notmuchmail.org/mailman/listinfo/notmuch</a><br>
</blockquote></div>