Unbreak some synchronization edge cases

src/Vulkan/VkFence.hpp

@@ -60,14 +60,6 @@ public:
 		}
 	}
 
-	void signal()
-	{
-		std::unique_lock<std::mutex> lock(mutex);
-		status = VK_SUCCESS;
-		lock.unlock();
-		condition.notify_all();
-	}
-
 	void reset()
 	{
 		std::unique_lock<std::mutex> lock(mutex);

src/Vulkan/VkQueue.cpp

@@ -17,6 +17,7 @@
 #include "VkQueue.hpp"
 #include "VkSemaphore.hpp"
 #include "WSI/VkSwapchainKHR.hpp"
+#include "Device/Renderer.hpp"
 
 #include <cstring>
 
@@ -151,6 +152,9 @@ void Queue::submitQueue(const Task& task)
 
 	if(task.fence)
 	{
+		// TODO: fix renderer signaling so that work submitted separately from (but before) a fence
+		// is guaranteed complete by the time the fence signals.
+		renderer->synchronize();
 		task.fence->done();
 	}
 }
@@ -188,9 +192,6 @@ VkResult Queue::waitIdle()
 
 	fence.wait();
 
-	// Wait for all draw operations to complete, if any
-	renderer->synchronize();
-
 	garbageCollect();
 
 	return VK_SUCCESS;

src/WSI/VkSwapchainKHR.cpp

@@ -193,7 +193,8 @@ VkResult SwapchainKHR::getNextImage(uint64_t timeout, VkSemaphore semaphore, VkF
 
 			if(fence)
 			{
-				vk::Cast(fence)->signal();
+				vk::Cast(fence)->add();
+				vk::Cast(fence)->done();
 			}
 
 			return VK_SUCCESS;