joystickstatuswindow: defer deletion of window upon close

Defer deleting the window when it's closed until after the main loop has
run. This prevents a segfault when the window is closed for
controllers with accelerometers and gyroscopes, such as DualSense
controllers.

Signed-off-by: Sean Cross <sean@xobs.io>

Author: xobs

src/gui/joystickstatuswindow.cpp

@@ -302,6 +302,8 @@ JoystickStatusWindow::JoystickStatusWindow(InputDevice *joystick, QWidget *paren
     connect(this, &JoystickStatusWindow::finished, this, &JoystickStatusWindow::restoreButtonStates);
 }
 
+void JoystickStatusWindow::reject() { this->deleteLater(); }
+
 JoystickStatusWindow::~JoystickStatusWindow() { delete ui; }
 
 void JoystickStatusWindow::restoreButtonStates(int code)
@@ -336,9 +338,9 @@ void JoystickStatusWindow::updateAccelerometerValues(float valueX, float valueY,
  */
 void JoystickStatusWindow::updateGyroscopeValues(float valueX, float valueY, float valueZ)
 {
-    m_gyro_axes[0]->setValue(JoySensor::radToDeg(valueX) * 1000);
-    m_gyro_axes[1]->setValue(JoySensor::radToDeg(valueY) * 1000);
-    m_gyro_axes[2]->setValue(JoySensor::radToDeg(valueZ) * 1000);
+    m_gyro_axes[0]->setValue(valueX * 1000);
+    m_gyro_axes[1]->setValue(valueY * 1000);
+    m_gyro_axes[2]->setValue(valueZ * 1000);
 }
 
 InputDevice *JoystickStatusWindow::getJoystick() const { return joystick; }

src/gui/joystickstatuswindow.h

@@ -44,6 +44,7 @@ class JoystickStatusWindow : public QDialog
     InputDevice *getJoystick() const;
 
   private:
+    void reject();
     Ui::JoystickStatusWindow *ui;
 
     InputDevice *joystick;