Patch: Flush and Reopen
Martin Owens
doctormo at gmail.com
Thu Sep 8 19:54:21 PDT 2011
Hey Austin,
Thanks for the review :-) I'll attempt to fix some of the style
guidelines, but see below.
On Thu, 2011-09-08 at 22:42 -0400, Austin Clements wrote:
> Did you mean to change this?
This fails to build on Ubuntu maverick with the extra .3 and I see no
reason to have that sub-minor version. Pushing it in would probably be
useful unless there is a real reason.
> This cast will fail. Shouldn't this just be a wrapper around
> notmuch->xapian_db->reopen?
I'm a python programmer, c wrappers are beyond me unfortunately. Your
help implementing that would be really good.
Best regards, Martin Owens
More information about the notmuch
mailing list