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, 6 insertions, 2 deletions

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  */