Use only placement new on pre-allocated memory. This makes sure that we have to…

Use only placement new on pre-allocated memory. This makes sure that we have to have no assumptions on the inital state of the memory region.

lib/pls/include/pls/internal/base/aligned_stack.h

@@ -32,13 +32,13 @@ namespace pls {
 
                 template<typename T>
                 T* push(const T& object) {
-                    // Copy-Construct into desired memory location
-                    return new (push<T>())T(object);
+                    // Placement new into desired memory location
+                    return new ((void*)push<T>())T(object);
                 }
 
                 template<typename T>
-                T* push() {
-                    T* result = reinterpret_cast<T*>(head_);
+                void* push() {
+                    void* result = reinterpret_cast<T*>(head_);
 
                     // Move head to next aligned position after new object
                     head_ = next_alignment(head_ + sizeof(T));

lib/pls/include/pls/internal/scheduling/thread_state.h

@@ -32,23 +32,6 @@ namespace pls {
                     current_task_{nullptr},
                     task_stack_{task_stack},
                     id_{id} {}
-
-                thread_state(const thread_state& other):
-                    scheduler_{other.scheduler_},
-                    root_task_{other.root_task_},
-                    current_task_{other.current_task_},
-                    task_stack_{other.task_stack_},
-                    id_{other.id_} {}
-
-                thread_state& operator=(const thread_state& other) {
-                    scheduler_ = other.scheduler_;
-                    root_task_ = other.root_task_;
-                    current_task_ = other.current_task_;
-                    task_stack_ = other.task_stack_;
-                    id_ = other.id_;
-
-                    return *this;
-                }
             };
         }
     }

lib/pls/src/internal/scheduling/scheduler.cpp

@@ -14,8 +14,9 @@ namespace pls {
                 }
 
                 for (unsigned int i = 0; i < num_threads_; i++) {
-                    new(memory_->thread_state_for(i)) thread_state{this, memory_->task_stack_for(i), i};
-                    *memory_->thread_for(i) = base::start_thread(&worker_routine, memory_->thread_state_for(i));
+                    // Placement new is required, as the memory of `memory_` is not required to be initialized.
+                    new((void*)memory_->thread_state_for(i)) thread_state{this, memory_->task_stack_for(i), i};
+                    new ((void*)memory_->thread_for(i))base::thread<void(*)(), thread_state>(&worker_routine, memory_->thread_state_for(i));
                 }
             }