Get rid of `BLI_task_pool_stop()`.
authorBastien Montagne <montagne29@wanadoo.fr>
Fri, 3 Mar 2017 16:16:39 +0000 (17:16 +0100)
committerBastien Montagne <montagne29@wanadoo.fr>
Fri, 3 Mar 2017 16:16:39 +0000 (17:16 +0100)
Comments said that function was supposed to 'stop worker threads', but
it absolutely did not do anything like that, was merely wiping out TODO
queue of tasks from given pool (kind of subset of what
`BLI_task_pool_cancel()` does).

Misleading, and currently useless, we can always add it back if we need
it some day, but for now we try to simplify that area.

source/blender/blenlib/intern/task.c

index 359360d59c41dc12b42202be6cff6e7efaf88602..91e821a8f1abd399f6074b93bb312393b721ccd4 100644 (file)
@@ -708,13 +708,6 @@ void BLI_task_pool_cancel(TaskPool *pool)
        pool->do_cancel = false;
 }
 
-void BLI_task_pool_stop(TaskPool *pool)
-{
-       task_scheduler_clear(pool->scheduler, pool);
-
-       BLI_assert(pool->num == 0);
-}
-
 bool BLI_task_pool_canceled(TaskPool *pool)
 {
        return pool->do_cancel;