diff options
| author | David Lamparter <equinox@opensourcerouting.org> | 2017-07-14 16:48:01 +0200 |
|---|---|---|
| committer | David Lamparter <equinox@opensourcerouting.org> | 2017-07-14 16:59:43 +0200 |
| commit | acd738fc7f5a8ca637e691e45346d74a149b75c9 (patch) | |
| tree | 62d1445b34e13fdfab132fa31cfb0ff9145b73ae /lib/workqueue.c | |
| parent | 888efdbb5ec4e895f44f0307348a02fd6c61b579 (diff) | |
*: fix GCC 7 switch/case fallthrough warnings
Need a comment on these.
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
Diffstat (limited to 'lib/workqueue.c')
| -rw-r--r-- | lib/workqueue.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/lib/workqueue.c b/lib/workqueue.c index 3600df2f23..60119f1f41 100644 --- a/lib/workqueue.c +++ b/lib/workqueue.c @@ -331,14 +331,14 @@ work_queue_run (struct thread *thread) { item->ran--; work_queue_item_requeue (wq, node); - /* If a single node is being used with a meta-queue (e.g., zebra), - * update the next node as we don't want to exit the thread and - * reschedule it after every node. By definition, WQ_REQUEUE is - * meant to continue the processing; the yield logic will kick in - * to terminate the thread when time has exceeded. - */ - if (nnode == NULL) - nnode = node; + /* If a single node is being used with a meta-queue (e.g., zebra), + * update the next node as we don't want to exit the thread and + * reschedule it after every node. By definition, WQ_REQUEUE is + * meant to continue the processing; the yield logic will kick in + * to terminate the thread when time has exceeded. + */ + if (nnode == NULL) + nnode = node; break; } case WQ_RETRY_NOW: @@ -348,7 +348,7 @@ work_queue_run (struct thread *thread) if (wq->spec.errorfunc) wq->spec.errorfunc (wq, item); } - /* fall through here is deliberate */ + /* fallthru */ case WQ_SUCCESS: default: { |