On Sun, Jun 01, 2008 at 04:28:44PM +0100, Rob Shearman wrote: > + setup_git_directory_gently( NULL ); > git_config(git_imap_config, NULL); I know this is just about cosmetics, but the style of the rest of the code would suggest using setup_git_directory_gently(NULL); not setup_git_directory_gently( NULL );