libobs-d3d11: Consistent exception catch parameters
Fixed missing references, and stray usages of const references instead of reference to const types. Apply const correctly where applicable.master
parent
6943d9a973
commit
fe02a8286e
|
@ -87,7 +87,7 @@ EXPORT bool device_get_duplicator_monitor_info(gs_device_t *device,
|
|||
if (FAILED(hr))
|
||||
throw HRError("GetDesc failed", hr);
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR,
|
||||
"device_get_duplicator_monitor_info: "
|
||||
"%s (%08lX)",
|
||||
|
@ -151,7 +151,7 @@ EXPORT gs_duplicator_t *device_duplicator_create(gs_device_t *device,
|
|||
blog(LOG_DEBUG, "device_duplicator_create: %s", error);
|
||||
return nullptr;
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_DEBUG, "device_duplicator_create: %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
return nullptr;
|
||||
|
|
|
@ -440,6 +440,6 @@ try {
|
|||
} catch (const char *error) {
|
||||
bcrash("Failed to recreate D3D11: %s", error);
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
bcrash("Failed to recreate D3D11: %s (%08lX)", error.str, error.hr);
|
||||
}
|
||||
|
|
|
@ -342,7 +342,7 @@ try {
|
|||
}
|
||||
return bad_driver;
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_WARNING, "HasBadNV12Output failed: %s (%08lX)", error.str,
|
||||
error.hr);
|
||||
return false;
|
||||
|
@ -717,7 +717,7 @@ bool device_enum_adapters(bool (*callback)(void *param, const char *name,
|
|||
EnumD3DAdapters(callback, param);
|
||||
return true;
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_WARNING, "Failed enumerating devices: %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
return false;
|
||||
|
@ -797,12 +797,12 @@ int device_create(gs_device_t **p_device, uint32_t adapter)
|
|||
|
||||
device = new gs_device(adapter);
|
||||
|
||||
} catch (UnsupportedHWError &error) {
|
||||
} catch (const UnsupportedHWError &error) {
|
||||
blog(LOG_ERROR, "device_create (D3D11): %s (%08lX)", error.str,
|
||||
error.hr);
|
||||
errorcode = GS_ERROR_NOT_SUPPORTED;
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_create (D3D11): %s (%08lX)", error.str,
|
||||
error.hr);
|
||||
errorcode = GS_ERROR_FAIL;
|
||||
|
@ -836,7 +836,7 @@ gs_swapchain_t *device_swapchain_create(gs_device_t *device,
|
|||
|
||||
try {
|
||||
swap = new gs_swap_chain(device, data);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_swapchain_create (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -866,7 +866,7 @@ void device_resize(gs_device_t *device, uint32_t cx, uint32_t cy)
|
|||
depthView = device->curZStencilBuffer->view;
|
||||
device->context->OMSetRenderTargets(1, &renderView, depthView);
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_resize (D3D11): %s (%08lX)", error.str,
|
||||
error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -916,7 +916,7 @@ gs_texture_t *device_texture_create(gs_device_t *device, uint32_t width,
|
|||
texture = new gs_texture_2d(device, width, height, color_format,
|
||||
levels, data, flags, GS_TEXTURE_2D,
|
||||
false);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_texture_create (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -937,7 +937,7 @@ gs_texture_t *device_cubetexture_create(gs_device_t *device, uint32_t size,
|
|||
texture = new gs_texture_2d(device, size, size, color_format,
|
||||
levels, data, flags,
|
||||
GS_TEXTURE_CUBE, false);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR,
|
||||
"device_cubetexture_create (D3D11): %s "
|
||||
"(%08lX)",
|
||||
|
@ -976,7 +976,7 @@ gs_zstencil_t *device_zstencil_create(gs_device_t *device, uint32_t width,
|
|||
try {
|
||||
zstencil =
|
||||
new gs_zstencil_buffer(device, width, height, format);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_zstencil_create (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -993,7 +993,7 @@ gs_stagesurf_t *device_stagesurface_create(gs_device_t *device, uint32_t width,
|
|||
try {
|
||||
surf = new gs_stage_surface(device, width, height,
|
||||
color_format);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR,
|
||||
"device_stagesurface_create (D3D11): %s "
|
||||
"(%08lX)",
|
||||
|
@ -1011,7 +1011,7 @@ device_samplerstate_create(gs_device_t *device,
|
|||
gs_sampler_state *ss = NULL;
|
||||
try {
|
||||
ss = new gs_sampler_state(device, info);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR,
|
||||
"device_samplerstate_create (D3D11): %s "
|
||||
"(%08lX)",
|
||||
|
@ -1030,14 +1030,14 @@ gs_shader_t *device_vertexshader_create(gs_device_t *device,
|
|||
try {
|
||||
shader = new gs_vertex_shader(device, file, shader_string);
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR,
|
||||
"device_vertexshader_create (D3D11): %s "
|
||||
"(%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
||||
} catch (ShaderError &const error) {
|
||||
} catch (const ShaderError &error) {
|
||||
const char *buf =
|
||||
(const char *)error.errors->GetBufferPointer();
|
||||
if (error_string)
|
||||
|
@ -1063,14 +1063,14 @@ gs_shader_t *device_pixelshader_create(gs_device_t *device,
|
|||
try {
|
||||
shader = new gs_pixel_shader(device, file, shader_string);
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR,
|
||||
"device_pixelshader_create (D3D11): %s "
|
||||
"(%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
||||
} catch (ShaderError &const error) {
|
||||
} catch (const ShaderError &error) {
|
||||
const char *buf =
|
||||
(const char *)error.errors->GetBufferPointer();
|
||||
if (error_string)
|
||||
|
@ -1094,7 +1094,7 @@ gs_vertbuffer_t *device_vertexbuffer_create(gs_device_t *device,
|
|||
gs_vertex_buffer *buffer = NULL;
|
||||
try {
|
||||
buffer = new gs_vertex_buffer(device, data, flags);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR,
|
||||
"device_vertexbuffer_create (D3D11): %s "
|
||||
"(%08lX)",
|
||||
|
@ -1116,7 +1116,7 @@ gs_indexbuffer_t *device_indexbuffer_create(gs_device_t *device,
|
|||
gs_index_buffer *buffer = NULL;
|
||||
try {
|
||||
buffer = new gs_index_buffer(device, type, indices, num, flags);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_indexbuffer_create (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -1130,7 +1130,7 @@ gs_timer_t *device_timer_create(gs_device_t *device)
|
|||
gs_timer *timer = NULL;
|
||||
try {
|
||||
timer = new gs_timer(device);
|
||||
} catch (HRError error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_timer_create (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -1144,7 +1144,7 @@ gs_timer_range_t *device_timer_range_create(gs_device_t *device)
|
|||
gs_timer_range *range = NULL;
|
||||
try {
|
||||
range = new gs_timer_range(device);
|
||||
} catch (HRError error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_timer_range_create (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -1591,7 +1591,7 @@ void device_draw(gs_device_t *device, enum gs_draw_mode draw_mode,
|
|||
blog(LOG_ERROR, "device_draw (D3D11): %s", error);
|
||||
return;
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_draw (D3D11): %s (%08lX)", error.str,
|
||||
error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -2390,7 +2390,7 @@ device_texture_create_gdi(gs_device_t *device, uint32_t width, uint32_t height)
|
|||
texture = new gs_texture_2d(device, width, height, GS_BGRA, 1,
|
||||
nullptr, GS_RENDER_TARGET,
|
||||
GS_TEXTURE_2D, true);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "device_texture_create_gdi (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -2448,7 +2448,7 @@ extern "C" EXPORT gs_texture_t *device_texture_open_shared(gs_device_t *device,
|
|||
gs_texture *texture = nullptr;
|
||||
try {
|
||||
texture = new gs_texture_2d(device, handle);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "gs_texture_open_shared (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -2535,7 +2535,7 @@ device_texture_create_nv12(gs_device_t *device, gs_texture_t **p_tex_y,
|
|||
true);
|
||||
tex_uv = new gs_texture_2d(device, tex_y->texture, flags);
|
||||
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR, "gs_texture_create_nv12 (D3D11): %s (%08lX)",
|
||||
error.str, error.hr);
|
||||
LogD3D11ErrorDetails(error, device);
|
||||
|
@ -2561,7 +2561,7 @@ device_stagesurface_create_nv12(gs_device_t *device, uint32_t width,
|
|||
gs_stage_surface *surf = NULL;
|
||||
try {
|
||||
surf = new gs_stage_surface(device, width, height);
|
||||
} catch (HRError &error) {
|
||||
} catch (const HRError &error) {
|
||||
blog(LOG_ERROR,
|
||||
"device_stagesurface_create (D3D11): %s "
|
||||
"(%08lX)",
|
||||
|
|
Loading…
Reference in New Issue