From 94e58057fd3b6f2ce3ce3a9c10814dc69535aafa Mon Sep 17 00:00:00 2001 From: kc5nra Date: Mon, 23 Mar 2015 22:28:08 -0500 Subject: [PATCH] deps-libff: Fix sequence-point warning Silly code that pre-incremented a value while also assigning it to the same value. --- deps/libff/libff/ff-circular-queue.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/deps/libff/libff/ff-circular-queue.c b/deps/libff/libff/ff-circular-queue.c index 027f95831..234edc485 100644 --- a/deps/libff/libff/ff-circular-queue.c +++ b/deps/libff/libff/ff-circular-queue.c @@ -116,7 +116,7 @@ void *ff_circular_queue_peek_write(struct ff_circular_queue *cq) void ff_circular_queue_advance_write(struct ff_circular_queue *cq, void *item) { cq->slots[cq->write_index] = item; - cq->write_index = ++cq->write_index % cq->capacity; + cq->write_index = (cq->write_index + 1) % cq->capacity; queue_lock(cq); ++cq->size; @@ -130,7 +130,7 @@ void *ff_circular_queue_peek_read(struct ff_circular_queue *cq) void ff_circular_queue_advance_read(struct ff_circular_queue *cq) { - cq->read_index = ++cq->read_index % cq->capacity; + cq->read_index = (cq->read_index + 1) % cq->capacity; queue_lock(cq); --cq->size; queue_signal(cq);