From a88e34b21b07f70123926fcb5c505d6afdf99807 Mon Sep 17 00:00:00 2001
From: Stanislaw Halik <sthalik@misaki.pl>
Date: Mon, 24 Dec 2018 18:54:34 +0100
Subject: style/quality only

No functional changes.

- add `override' everywhere where missing
- almost pass clang's `-Wweak-vtables'
- avoid some float/double conversions
- remove unused private members
- make signedness conversions explicit
- put stuff in right namespaces to aid analysis
---
 dinput/dinput.cpp | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

(limited to 'dinput/dinput.cpp')

diff --git a/dinput/dinput.cpp b/dinput/dinput.cpp
index 8a3d0f4a..3931255b 100644
--- a/dinput/dinput.cpp
+++ b/dinput/dinput.cpp
@@ -10,12 +10,12 @@ diptr di_t::init_di_()
     CoInitialize(nullptr);
 
     diptr di = nullptr;
-    if (HRESULT hr = DirectInput8Create(GetModuleHandle(nullptr),
-                                        DIRECTINPUT_VERSION,
-                                        IID_IDirectInput8,
-                                        (void**)&di,
-                                        nullptr);
-        !SUCCEEDED(hr))
+    HRESULT hr = DirectInput8Create(GetModuleHandle(nullptr),
+                                    DIRECTINPUT_VERSION,
+                                    IID_IDirectInput8,
+                                    (void**)&di,
+                                    nullptr);
+    if (!SUCCEEDED(hr))
     {
         qDebug() << "can't make dinput:" << (void*)(LONG_PTR)hr;
         qDebug() << "crashing!";
@@ -32,7 +32,8 @@ di_t::di_t()
 
 void di_t::ref_di()
 {
-    while (init_lock.test_and_set()) { /* busy loop */ }
+    while (init_lock.test_and_set())
+        (void)0;
 
     if (!handle)
         handle = init_di_();
@@ -48,7 +49,8 @@ void di_t::unref_di()
 
     if (refcnt_ == 0)
     {
-        while (init_lock.test_and_set()) { /* busy loop */ }
+        while (init_lock.test_and_set())
+            (void)0;
 
         qDebug() << "exit: di handle";
         handle->Release();
-- 
cgit v1.2.3