summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohn Helmert III <jchelmert3@posteo.net>2021-01-22 20:54:53 -0600
committerJames Le Cuirot <chewi@gentoo.org>2021-01-24 23:05:49 +0000
commit1d17bdd5ca9abdc95eaf9c132a6706d70c10ae7c (patch)
treed5986e3af8bd015a9cf0e9752a34bf42bff5ca5e /games-emulation/openmsx/files
parentmedia-libs/qt-gstreamer: Remove last-rited package (diff)
downloadgentoo-1d17bdd5ca9abdc95eaf9c132a6706d70c10ae7c.tar.gz
gentoo-1d17bdd5ca9abdc95eaf9c132a6706d70c10ae7c.tar.bz2
gentoo-1d17bdd5ca9abdc95eaf9c132a6706d70c10ae7c.zip
games-emulation/openmsx: fix build failure with libsdl2[-joystick]
Closes: https://bugs.gentoo.org/766552 Package-Manager: Portage-3.0.14, Repoman-3.0.2 Signed-off-by: John Helmert III <jchelmert3@posteo.net> Closes: https://github.com/gentoo/gentoo/pull/19187 Signed-off-by: James Le Cuirot <chewi@gentoo.org>
Diffstat (limited to 'games-emulation/openmsx/files')
-rw-r--r--games-emulation/openmsx/files/openmsx-16.0-libsdl-joystick-fix.patch66
1 files changed, 66 insertions, 0 deletions
diff --git a/games-emulation/openmsx/files/openmsx-16.0-libsdl-joystick-fix.patch b/games-emulation/openmsx/files/openmsx-16.0-libsdl-joystick-fix.patch
new file mode 100644
index 000000000000..ff4e4c31e0cd
--- /dev/null
+++ b/games-emulation/openmsx/files/openmsx-16.0-libsdl-joystick-fix.patch
@@ -0,0 +1,66 @@
+From eb5ddae80bdc6793de42ee67dd72e2da9d632ba8 Mon Sep 17 00:00:00 2001
+From: Maarten ter Huurne <maarten@treewalker.org>
+Date: Sat, 23 Jan 2021 03:25:38 +0100
+Subject: [PATCH] Fix compile errors and warnings when SDL_JOYSTICK_DISABLED is
+ defined
+
+If there is no base class, it's impossible to override the destructor.
+
+There were warning suppressors for 3 out of 5 arguments that are unused
+when SDL_JOYSTICK_DISABLED is defined; I added the othe two.
+---
+ src/input/JoyMega.hh | 6 +++++-
+ src/input/Joystick.cc | 2 ++
+ src/input/Joystick.hh | 8 ++++++--
+ 3 files changed, 13 insertions(+), 3 deletions(-)
+
+diff --git a/src/input/JoyMega.hh b/src/input/JoyMega.hh
+index 8422bcd2a..05371e27d 100644
+--- a/src/input/JoyMega.hh
++++ b/src/input/JoyMega.hh
+@@ -25,7 +25,11 @@ public:
+ JoyMega(MSXEventDistributor& eventDistributor,
+ StateChangeDistributor& stateChangeDistributor,
+ SDL_Joystick* joystick);
+- ~JoyMega() override;
++ ~JoyMega()
++#ifndef SDL_JOYSTICK_DISABLED
++ override
++#endif
++ ;
+
+ #ifndef SDL_JOYSTICK_DISABLED
+ // Pluggable
+diff --git a/src/input/Joystick.cc b/src/input/Joystick.cc
+index 04b4826cd..90b344c42 100644
+--- a/src/input/Joystick.cc
++++ b/src/input/Joystick.cc
+@@ -32,6 +32,8 @@ void Joystick::registerAll(MSXEventDistributor& eventDistributor,
+ #ifdef SDL_JOYSTICK_DISABLED
+ (void)eventDistributor;
+ (void)stateChangeDistributor;
++ (void)commandController;
++ (void)globalSettings;
+ (void)controller;
+ #else
+ for (auto i : xrange(SDL_NumJoysticks())) {
+diff --git a/src/input/Joystick.hh b/src/input/Joystick.hh
+index 18dd3c1c0..6a0f38d9d 100644
+--- a/src/input/Joystick.hh
++++ b/src/input/Joystick.hh
+@@ -38,9 +38,13 @@ public:
+ Joystick(MSXEventDistributor& eventDistributor,
+ StateChangeDistributor& stateChangeDistributor,
+ CommandController& commandController,
+- GlobalSettings& globalSettings,
++ GlobalSettings& globalSettings,
+ SDL_Joystick* joystick);
+- ~Joystick() override;
++ ~Joystick()
++#ifndef SDL_JOYSTICK_DISABLED
++ override
++#endif
++ ;
+
+ #ifndef SDL_JOYSTICK_DISABLED
+ // Pluggable