Mercurial > dovecot > core-2.2
comparison 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 |
comparison
equal
deleted
inserted
replaced
14681:ca37d1577291 | 14682:d0d7b810646b |
---|---|
49 for (i = 0; i < ext_count; i++) { | 49 for (i = 0; i < ext_count; i++) { |
50 ext_rec = array_idx(old_array, i); | 50 ext_rec = array_idx(old_array, i); |
51 | 51 |
52 seq = *ext_rec < first_new_seq ? *ext_rec : | 52 seq = *ext_rec < first_new_seq ? *ext_rec : |
53 old_to_newseq_map[*ext_rec - first_new_seq]; | 53 old_to_newseq_map[*ext_rec - first_new_seq]; |
54 mail_index_seq_array_add(&new_array, seq, ext_rec+1, | 54 (void)mail_index_seq_array_add(&new_array, seq, ext_rec+1, |
55 old_array->arr.element_size - | 55 old_array->arr.element_size - |
56 sizeof(*ext_rec), NULL); | 56 sizeof(*ext_rec), NULL); |
57 } | 57 } |
58 array_free(old_array); | 58 array_free(old_array); |
59 ext_rec_arrays[j] = new_array; | 59 ext_rec_arrays[j] = new_array; |
60 } | 60 } |
61 } | 61 } |