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