On Fri, Jun 10 2016, David Bremner <david@tethera.net> wrote: > David Bremner <david@tethera.net> writes: > >> +#ifdef HAVE_XAPIAN_FIELD_PROCESSOR >> + delete notmuch->from_re_field_processor; >> + notmuch->from_re_field_processor = NULL; >> + delete notmuch->subject_re_field_processor; >> + notmuch->subject_re_field_processor = NULL; >> +#endif > > and of course everywhere it says #ifdef HAVE_XAPIAN_FIELD_PROCESSOR, is > should say #if. ... is there a static code analyzer which notices such a mistakes... ? i did marked that version trivial, which completes the thought that there are no such thing as trivial changes ;/ > d