Do not pass data which is already contained in wrapper::Device

[IAmNotHanni]

Don't pass redundant data

Instead of doing this:

m_imgui_overlay = std::make_unique<imgui::ImGUIOverlay>(*m_vkdevice, *m_swapchain,
 m_vkdevice->physical_device(),
 m_vkdevice->allocator(), m_vkdevice->graphics_queue(),
 m_vkdevice->graphics_queue_family_index());

Do this:

m_imgui_overlay = std::make_unique<imgui::ImGUIOverlay>(*m_vkdevice, *m_swapchain);

Because the other data is already available through m_vkdevice .

Rethink use of queues

• Currently we're using graphics queue even in cases where transfer queue would be appropriate!