Daniel Kahn Gillmor writes: > +notmuch_bool_t > +test_for_executable(const char* exename) > +{ > + char *c = NULL, *save = NULL, *tok; > + size_t n; > + int dfd = -1; c, dfd, and n could be more meaningful as variable names. % uncrustify --config devel/uncrustify.cfg --replace util/search-path.c util/search-path.h yields quite a few whitespace changes (diff attached)