diff src/lib-storage/index/dbox-common/dbox-sync-rebuild.c @ 14682:d0d7b810646b

Make sure we check all the functions' return values. Minor API changes to simplify this. Checked using a patched clang that adds attribute(warn_unused_result) to all functions. This commit fixes several error handling mistakes.
author Timo Sirainen <tss@iki.fi>
date Mon, 25 Jun 2012 01:14:03 +0300
parents c93ca5e46a8a
children 89c319980584
line wrap: on
line diff
--- a/src/lib-storage/index/dbox-common/dbox-sync-rebuild.c	Mon Jun 25 00:01:59 2012 +0300
+++ b/src/lib-storage/index/dbox-common/dbox-sync-rebuild.c	Mon Jun 25 01:14:03 2012 +0300
@@ -158,7 +158,7 @@
 	ctx->trans = trans;
 	mail_index_reset(ctx->trans);
 	index_mailbox_reset_uidvalidity(box);
-	mail_index_ext_lookup(box->index, "cache", &ctx->cache_ext_id);
+	(void)mail_index_ext_lookup(box->index, "cache", &ctx->cache_ext_id);
 
 	/* open cache and read the caching decisions. we'll reset the cache in
 	   case it contains any invalid data, but we want to preserve the