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

2 files changed, 7 insertions, 3 deletions

diff --git a/game/shared/threading.cc b/game/shared/threading.cc
index 7e6f652..71c30e1 100644
--- a/game/shared/threading.cc
+++ b/game/shared/threading.cc
@@ -99,7 +99,7 @@ void threading::update(void)
     }
 }
 
-void threading::submit(Task *task)
+void threading::detail::submit_new(Task *task)
 {
     task->set_status(task_status::ENQUEUED);
 
diff --git a/game/shared/threading.hh b/game/shared/threading.hh
index e143f4f..083ccbf 100644
--- a/game/shared/threading.hh
+++ b/game/shared/threading.hh
@@ -28,9 +28,13 @@ namespace threading
 void init(void);
 void deinit(void);
 void update(void);
-void submit(Task *task);
 } // namespace threading
 
+namespace threading::detail
+{
+void submit_new(Task *task);
+} // namespace threading::detail
+
 namespace threading
 {
 template<typename T, typename... AT>
@@ -40,7 +44,7 @@ void submit(AT &&... args);
 template<typename T, typename... AT>
 inline void threading::submit(AT &&... args)
 {
-    threading::submit(new T(args...));
+    threading::detail::submit_new(new T(args...));
 }
 
 #endif /* SHARED_THREADING_HH  */