diff options
author | James Ross-Gowan <rossymiles@gmail.com> | 2016-11-17 23:05:17 +1100 |
---|---|---|
committer | James Ross-Gowan <rossymiles@gmail.com> | 2016-11-17 23:34:20 +1100 |
commit | 803e2c5f87c2102e67922e04fbae02247b66e790 (patch) | |
tree | ed6a3752683939fbc6e8af77d864a48c0d298390 | |
parent | 40b626fd9b58e2975cff8dec202848a7bce41a9f (diff) | |
download | mpv-803e2c5f87c2102e67922e04fbae02247b66e790.tar.bz2 mpv-803e2c5f87c2102e67922e04fbae02247b66e790.tar.xz |
win32: fix some Clang warnings
- win32-console-wrapper.c was inconsistently using the explicit Unicode
versions of some Windows API functions and structures.
- vo.c should use llabs for int64_t, since long is 32-bit on Windows.
- vo_direct3d.c had a potential use of an uninitialized variable if it
took the first goto error_exit.
-rw-r--r-- | osdep/win32-console-wrapper.c | 6 | ||||
-rw-r--r-- | video/out/vo.c | 2 | ||||
-rw-r--r-- | video/out/vo_direct3d.c | 2 |
3 files changed, 5 insertions, 5 deletions
diff --git a/osdep/win32-console-wrapper.c b/osdep/win32-console-wrapper.c index 778d699161..ff69f9aee1 100644 --- a/osdep/win32-console-wrapper.c +++ b/osdep/win32-console-wrapper.c @@ -36,8 +36,8 @@ void cr_perror(const wchar_t *prefix) int cr_runproc(wchar_t *name, wchar_t *cmdline) { - STARTUPINFO si; - STARTUPINFO our_si; + STARTUPINFOW si; + STARTUPINFOW our_si; PROCESS_INFORMATION pi; DWORD retval = 1; @@ -50,7 +50,7 @@ int cr_runproc(wchar_t *name, wchar_t *cmdline) // Copy the list of inherited CRT file descriptors to the new process our_si.cb = sizeof(our_si); - GetStartupInfo(&our_si); + GetStartupInfoW(&our_si); si.lpReserved2 = our_si.lpReserved2; si.cbReserved2 = our_si.cbReserved2; diff --git a/video/out/vo.c b/video/out/vo.c index 296be634ad..cbd2ca87c8 100644 --- a/video/out/vo.c +++ b/video/out/vo.c @@ -401,7 +401,7 @@ static void vsync_skip_detection(struct vo *vo) } int64_t desync = diff / in->num_vsync_samples; if (in->drop_point > window * 2 && - labs(desync - desync_early) >= in->vsync_interval * 3 / 4) + llabs(desync - desync_early) >= in->vsync_interval * 3 / 4) { // Assume a drop. An underflow can technically speaking not be a drop // (it's up to the driver what this is supposed to mean), but no reason diff --git a/video/out/vo_direct3d.c b/video/out/vo_direct3d.c index 38dde55af2..c99ea372f6 100644 --- a/video/out/vo_direct3d.c +++ b/video/out/vo_direct3d.c @@ -1466,13 +1466,13 @@ static mp_image_t *get_window_screenshot(d3d_priv *priv) POINT pt; D3DLOCKED_RECT locked_rect; int width, height; + IDirect3DSurface9 *surface = NULL; if (FAILED(IDirect3DDevice9_GetDisplayMode(priv->d3d_device, 0, &mode))) { MP_ERR(priv, "GetDisplayMode failed.\n"); goto error_exit; } - IDirect3DSurface9 *surface = NULL; if (FAILED(IDirect3DDevice9_CreateOffscreenPlainSurface(priv->d3d_device, mode.Width, mode.Height, D3DFMT_A8R8G8B8, D3DPOOL_SYSTEMMEM, &surface, NULL))) |