Slightly refactor threading API

- Hide away threading::submit overload that is meant to only be used
  with either a task that has been allocated using "new" or using a
  templated overload. Now it's called threading::detail::submit_new
- Removed an unnecessary status check in mesher's Task::finalize impl

1 files changed, 0 insertions, 7 deletions

diff --git a/game/client/chunk_mesher.cc b/game/client/chunk_mesher.cc
index 011d6a7..7e185f1 100644
--- a/game/client/chunk_mesher.cc
+++ b/game/client/chunk_mesher.cc
@@ -161,13 +161,6 @@ void GL_MeshingTask::finalize(void)
         return;
     }
 
-    if(m_status == task_status::CANCELLED) {
-        // The task has been cancelled, the result
-        // should be invalidated, if the code even got
-        // to this point (it normally should not)
-        return;
-    }
-
     auto &component = globals::dimension->chunks.emplace_or_replace<ChunkMeshComponent>(m_entity);
 
     const std::size_t plane_count_nb = m_quads_s.size();