rsx/vk: Cleanup

- Silence some warnings, remove dprints
This commit is contained in:
kd-11 2018-02-19 15:41:52 +03:00
parent 77f2b521e1
commit 4c6ceeff11
3 changed files with 18 additions and 10 deletions

View File

@ -194,7 +194,7 @@ namespace gl
public: public:
void reset(u32 base, u32 size, bool flushable=false) void reset(u32 base, u32 size, bool /*flushable*/=false)
{ {
rsx::protection_policy policy = g_cfg.video.strict_rendering_mode ? rsx::protection_policy::protect_policy_full_range : rsx::protection_policy::protect_policy_conservative; rsx::protection_policy policy = g_cfg.video.strict_rendering_mode ? rsx::protection_policy::protect_policy_full_range : rsx::protection_policy::protect_policy_conservative;
rsx::buffered_section::reset(base, size, policy); rsx::buffered_section::reset(base, size, policy);

View File

@ -1121,6 +1121,12 @@ public:
m_width = rect.right - rect.left; m_width = rect.right - rect.left;
m_height = rect.bottom - rect.top; m_height = rect.bottom - rect.top;
if (m_width == 0 || m_height == 0)
{
LOG_ERROR(RSX, "Invalid window dimensions %d x %d", m_width, m_height);
return false;
}
BITMAPINFO bitmap = {}; BITMAPINFO bitmap = {};
bitmap.bmiHeader.biSize = sizeof(BITMAPINFOHEADER); bitmap.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);
bitmap.bmiHeader.biWidth = m_width; bitmap.bmiHeader.biWidth = m_width;
@ -1161,10 +1167,13 @@ public:
auto& src = swapchain_images[image]; auto& src = swapchain_images[image];
GdiFlush(); GdiFlush();
if (hSrcDC)
{
memcpy(hPtr, src.second->get_pixels(), src.second->get_required_memory_size()); memcpy(hPtr, src.second->get_pixels(), src.second->get_required_memory_size());
BitBlt(hDstDC, 0, 0, m_width, m_height, hSrcDC, 0, 0, SRCCOPY); BitBlt(hDstDC, 0, 0, m_width, m_height, hSrcDC, 0, 0, SRCCOPY);
src.second->free_pixels(); src.second->free_pixels();
}
src.first = false; src.first = false;
return VK_SUCCESS; return VK_SUCCESS;
} }
@ -1264,7 +1273,7 @@ public:
} }
#endif #endif
VkResult acquire_next_swapchain_image(VkSemaphore semaphore, u64 timeout, u32* result) override VkResult acquire_next_swapchain_image(VkSemaphore /*semaphore*/, u64 /*timeout*/, u32* result) override
{ {
u32 index = 0; u32 index = 0;
for (auto &p : swapchain_images) for (auto &p : swapchain_images)
@ -1301,7 +1310,6 @@ public:
void init_swapchain_images(render_device& dev, u32 preferred_count) override void init_swapchain_images(render_device& dev, u32 preferred_count) override
{ {
swapchain_images.resize(preferred_count); swapchain_images.resize(preferred_count);
printf("Preparing %d images with sizes %d x %d\n", preferred_count, m_width, m_height);
for (auto &img : swapchain_images) for (auto &img : swapchain_images)
{ {
img.second = std::make_unique<swapchain_image_RPCS3>(dev, dev.get_memory_mapping(), m_width, m_height); img.second = std::make_unique<swapchain_image_RPCS3>(dev, dev.get_memory_mapping(), m_width, m_height);
@ -1323,7 +1331,7 @@ public:
PFN_vkQueuePresentKHR queuePresentKHR = nullptr; PFN_vkQueuePresentKHR queuePresentKHR = nullptr;
protected: protected:
void init_swapchain_images(render_device& dev, u32 preferred_count) override void init_swapchain_images(render_device& dev, u32 /*preferred_count*/ = 0) override
{ {
u32 nb_swap_images = 0; u32 nb_swap_images = 0;
getSwapchainImagesKHR(dev, m_vk_swapchain, &nb_swap_images, nullptr); getSwapchainImagesKHR(dev, m_vk_swapchain, &nb_swap_images, nullptr);
@ -1507,7 +1515,7 @@ public:
destroySwapchainKHR(dev, old_swapchain, nullptr); destroySwapchainKHR(dev, old_swapchain, nullptr);
} }
init_swapchain_images(dev, nb_swap_images); init_swapchain_images(dev);
return true; return true;
} }
@ -1516,7 +1524,7 @@ public:
return vkAcquireNextImageKHR(dev, m_vk_swapchain, timeout, semaphore, VK_NULL_HANDLE, result); return vkAcquireNextImageKHR(dev, m_vk_swapchain, timeout, semaphore, VK_NULL_HANDLE, result);
} }
void end_frame(command_buffer &cmd, u32 index) override void end_frame(command_buffer& /*cmd*/, u32 /*index*/) override
{ {
} }

View File

@ -630,7 +630,7 @@ namespace vk
return upload_simple_texture(cmd.get_command_pool().get_owner(), cmd, memory_types, upload_heap, key, desc->w, desc->h, false, true, desc->data); return upload_simple_texture(cmd.get_command_pool().get_owner(), cmd, memory_types, upload_heap, key, desc->w, desc->h, false, true, desc->data);
} }
void update_uniforms(vk::glsl::program *program) override void update_uniforms(vk::glsl::program* /*program*/) override
{ {
auto dst = (f32*)m_ubo->map(first_vertex * 128, 128); auto dst = (f32*)m_ubo->map(first_vertex * 128, 128);
dst[0] = m_scale_offset.r; dst[0] = m_scale_offset.r;