diff src/lib-index/mail-index-transaction-sort-appends.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 69ba6977bed8
children df2e46c38c92
line wrap: on
line diff
--- a/src/lib-index/mail-index-transaction-sort-appends.c	Mon Jun 25 00:01:59 2012 +0300
+++ b/src/lib-index/mail-index-transaction-sort-appends.c	Mon Jun 25 01:14:03 2012 +0300
@@ -51,9 +51,9 @@
 
 			seq = *ext_rec < first_new_seq ? *ext_rec :
 				old_to_newseq_map[*ext_rec - first_new_seq];
-			mail_index_seq_array_add(&new_array, seq, ext_rec+1,
-						 old_array->arr.element_size -
-						 sizeof(*ext_rec), NULL);
+			(void)mail_index_seq_array_add(&new_array, seq, ext_rec+1,
+						       old_array->arr.element_size -
+						       sizeof(*ext_rec), NULL);
 		}
 		array_free(old_array);
 		ext_rec_arrays[j] = new_array;