diff options
| author | René Scharfe <l.s.r@web.de> | 2017-07-15 20:00:45 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-07-17 21:54:56 (GMT) |
| commit | f331ab9d4cb21942dcde6d879aaca6a1784e8cb6 (patch) | |
| tree | 64203c98b322d7b468b2649df734a2a3ea443fc1 /read-cache.c | |
| parent | 578398071e45d296c3dc1de10acdbc15365e763f (diff) | |
| download | git-f331ab9d4cb21942dcde6d879aaca6a1784e8cb6.zip git-f331ab9d4cb21942dcde6d879aaca6a1784e8cb6.tar.gz git-f331ab9d4cb21942dcde6d879aaca6a1784e8cb6.tar.bz2 | |
use MOVE_ARRAY
Simplify the code for moving members inside of an array and make it more
robust by using the helper macro MOVE_ARRAY. It calculates the size
based on the specified number of elements for us and supports NULL
pointers when that number is zero. Raw memmove(3) calls with NULL can
cause the compiler to (over-eagerly) optimize out later NULL checks.
This patch was generated with contrib/coccinelle/array.cocci and spatch
(Coccinelle).
Signed-off-by: Rene Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'read-cache.c')
| -rw-r--r-- | read-cache.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/read-cache.c b/read-cache.c index 2121b6e..acfb028 100644 --- a/read-cache.c +++ b/read-cache.c @@ -515,9 +515,8 @@ int remove_index_entry_at(struct index_state *istate, int pos) istate->cache_nr--; if (pos >= istate->cache_nr) return 0; - memmove(istate->cache + pos, - istate->cache + pos + 1, - (istate->cache_nr - pos) * sizeof(struct cache_entry *)); + MOVE_ARRAY(istate->cache + pos, istate->cache + pos + 1, + istate->cache_nr - pos); return 1; } |