# SDL: Re-add SDL\_HAPTIC\_SQUARE for SDL 3.0

**URL:** <https://discourse.libsdl.org/t/sdl-re-add-sdl-haptic-square-for-sdl-3-0/48475>\
**Category:** SDL Commits\
**Created:** [January 18, 2024, 5:26pm UTC](https://discourse.libsdl.org/t/sdl-re-add-sdl-haptic-square-for-sdl-3-0/48475 "2024-01-18T17:26:02Z")\
**Posts on this page:** 1\
**Page:** 1

<div class="post-metadata">

**Author:** ![SDLPushPostBot](https://discourse.libsdl.org/user_avatar/discourse.libsdl.org/sdlpushpostbot/32/4370_2.png) [@SDLPushPostBot](https://discourse.libsdl.org/u/SDLPushPostBot)\
**Post date:** [January 18, 2024, 5:26pm UTC](https://discourse.libsdl.org/t/sdl-re-add-sdl-haptic-square-for-sdl-3-0/48475/1 "2024-01-18T17:26:03Z")

</div>

> <https://github.com/libsdl-org/SDL/commit/cda2bffeeea18a44c1ab559bb1612dbc4691b075>
>
> Fixes https://github.com/libsdl-org/SDL/issues/8753
> Closes https://github.com/li…bsdl-org/SDL/pull/8831
> Closes https://github.com/libsdl-org/SDL/pull/8783

```diff
From cda2bffeeea18a44c1ab559bb1612dbc4691b075 Mon Sep 17 00:00:00 2001
From: Sam Lantinga <[EMAIL REDACTED]>
Date: Thu, 18 Jan 2024 06:55:36 -0800
Subject: [PATCH] Re-add SDL_HAPTIC_SQUARE for SDL 3.0

Fixes https://github.com/libsdl-org/SDL/issues/8753
Closes https://github.com/libsdl-org/SDL/pull/8831
Closes https://github.com/libsdl-org/SDL/pull/8783
---
 include/SDL3/SDL_haptic.h | 65 ++++++++++++++++-----------
 src/dynapi/SDL_dynapi_procs.h | 2 +-
 src/haptic/SDL_haptic.c | 2 +-
 src/haptic/SDL_syshaptic.h | 2 +-
 src/haptic/darwin/SDL_syshaptic.c | 11 ++---
 src/haptic/linux/SDL_syshaptic.c | 11 ++---
 src/haptic/windows/SDL_dinputhaptic.c | 11 ++---
 test/testhaptic.c | 5 +--
 8 files changed, 55 insertions(+), 54 deletions(-)

diff --git a/include/SDL3/SDL_haptic.h b/include/SDL3/SDL_haptic.h
index 1cd044acef3f..40582e771c7d 100644
--- a/include/SDL3/SDL_haptic.h
+++ b/include/SDL3/SDL_haptic.h
@@ -166,7 +166,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticCondition
  */
-#define SDL_HAPTIC_CONSTANT (1u<<0)
+#define SDL_HAPTIC_CONSTANT (1u<<0)
 
 /**
  * Sine wave effect supported.
@@ -175,21 +175,16 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticPeriodic
  */
-#define SDL_HAPTIC_SINE (1u<<1)
+#define SDL_HAPTIC_SINE (1u<<1)
 
 /**
- * Left/Right effect supported.
+ * Square wave effect supported.
  *
- * Haptic effect for direct control over high/low frequency motors.
+ * Periodic haptic effect that simulates square waves.
  *
- * \sa SDL_HapticLeftRight
- * \warning this value was SDL_HAPTIC_SQUARE right before 2.0.0 shipped. Sorry,
- * we ran out of bits, and this is important for XInput devices.
+ * \sa SDL_HapticPeriodic
  */
-#define SDL_HAPTIC_LEFTRIGHT (1u<<2)
-
-/* !!! FIXME: put this back when we have more bits in 2.1 */
-/* #define SDL_HAPTIC_SQUARE (1<<2) */
+#define SDL_HAPTIC_SQUARE (1<<2)
 
 /**
  * Triangle wave effect supported.
@@ -198,7 +193,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticPeriodic
  */
-#define SDL_HAPTIC_TRIANGLE (1u<<3)
+#define SDL_HAPTIC_TRIANGLE (1u<<3)
 
 /**
  * Sawtoothup wave effect supported.
@@ -207,7 +202,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticPeriodic
  */
-#define SDL_HAPTIC_SAWTOOTHUP (1u<<4)
+#define SDL_HAPTIC_SAWTOOTHUP (1u<<4)
 
 /**
  * Sawtoothdown wave effect supported.
@@ -225,7 +220,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticRamp
  */
-#define SDL_HAPTIC_RAMP (1u<<6)
+#define SDL_HAPTIC_RAMP (1u<<6)
 
 /**
  * Spring effect supported - uses axes position.
@@ -235,7 +230,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticCondition
  */
-#define SDL_HAPTIC_SPRING (1u<<7)
+#define SDL_HAPTIC_SPRING (1u<<7)
 
 /**
  * Damper effect supported - uses axes velocity.
@@ -245,7 +240,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticCondition
  */
-#define SDL_HAPTIC_DAMPER (1u<<8)
+#define SDL_HAPTIC_DAMPER (1u<<8)
 
 /**
  * Inertia effect supported - uses axes acceleration.
@@ -255,7 +250,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticCondition
  */
-#define SDL_HAPTIC_INERTIA (1u<<9)
+#define SDL_HAPTIC_INERTIA (1u<<9)
 
 /**
  * Friction effect supported - uses axes movement.
@@ -265,14 +260,30 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_HapticCondition
  */
-#define SDL_HAPTIC_FRICTION (1u<<10)
+#define SDL_HAPTIC_FRICTION (1u<<10)
+
+/**
+ * Left/Right effect supported.
+ *
+ * Haptic effect for direct control over high/low frequency motors.
+ *
+ * \sa SDL_HapticLeftRight
+ */
+#define SDL_HAPTIC_LEFTRIGHT (1u<<11)
+
+/**
+ * Reserved for future use
+ */
+#define SDL_HAPTIC_RESERVED1 (1u<<12)
+#define SDL_HAPTIC_RESERVED2 (1u<<13)
+#define SDL_HAPTIC_RESERVED3 (1u<<14)
 
 /**
  * Custom effect is supported.
  *
  * User defined custom haptic effect.
  */
-#define SDL_HAPTIC_CUSTOM (1u<<11)
+#define SDL_HAPTIC_CUSTOM (1u<<15)
 
 /* @} *//* Haptic effects */
 
@@ -285,7 +296,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_SetHapticGain
  */
-#define SDL_HAPTIC_GAIN (1u<<12)
+#define SDL_HAPTIC_GAIN (1u<<16)
 
 /**
  * Device can set autocenter.
@@ -294,7 +305,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_SetHapticAutocenter
  */
-#define SDL_HAPTIC_AUTOCENTER (1u<<13)
+#define SDL_HAPTIC_AUTOCENTER (1u<<17)
 
 /**
  * Device can be queried for effect status.
@@ -303,7 +314,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  *
  * \sa SDL_GetHapticEffectStatus
  */
-#define SDL_HAPTIC_STATUS (1u<<14)
+#define SDL_HAPTIC_STATUS (1u<<18)
 
 /**
  * Device can be paused.
@@ -313,7 +324,7 @@ typedef struct SDL_Haptic SDL_Haptic;
  * \sa SDL_PauseHaptic
  * \sa SDL_ResumeHaptic
  */
-#define SDL_HAPTIC_PAUSE (1u<<15)
+#define SDL_HAPTIC_PAUSE (1u<<19)
 
 
 /**
@@ -511,7 +522,7 @@ typedef struct SDL_HapticConstant
  *
  * The struct handles the following effects:
  * - ::SDL_HAPTIC_SINE
- * - ::SDL_HAPTIC_LEFTRIGHT
+ * - ::SDL_HAPTIC_SQUARE
  * - ::SDL_HAPTIC_TRIANGLE
  * - ::SDL_HAPTIC_SAWTOOTHUP
  * - ::SDL_HAPTIC_SAWTOOTHDOWN
@@ -557,7 +568,7 @@ typedef struct SDL_HapticConstant
     \endverbatim
  *
  * \sa SDL_HAPTIC_SINE
- * \sa SDL_HAPTIC_LEFTRIGHT
+ * \sa SDL_HAPTIC_SQUARE
  * \sa SDL_HAPTIC_TRIANGLE
  * \sa SDL_HAPTIC_SAWTOOTHUP
  * \sa SDL_HAPTIC_SAWTOOTHDOWN
@@ -566,7 +577,7 @@ typedef struct SDL_HapticConstant
 typedef struct SDL_HapticPeriodic
 {
     /* Header */
- Uint16 type; /**< ::SDL_HAPTIC_SINE, ::SDL_HAPTIC_LEFTRIGHT,
+ Uint16 type; /**< ::SDL_HAPTIC_SINE, ::SDL_HAPTIC_SQUARE
                              ::SDL_HAPTIC_TRIANGLE, ::SDL_HAPTIC_SAWTOOTHUP or
                              ::SDL_HAPTIC_SAWTOOTHDOWN */
     SDL_HapticDirection direction; /**< Direction of the effect. */
@@ -1050,7 +1061,7 @@ extern DECLSPEC int SDLCALL SDL_GetMaxHapticEffectsPlaying(SDL_Haptic *haptic);
  * \sa SDL_HapticEffectSupported
  * \sa SDL_GetMaxHapticEffects
  */
-extern DECLSPEC unsigned int SDLCALL SDL_GetHapticFeatures(SDL_Haptic *haptic);
+extern DECLSPEC Uint32 SDLCALL SDL_GetHapticFeatures(SDL_Haptic *haptic);
 
 
 /**
diff --git a/src/dynapi/SDL_dynapi_procs.h b/src/dynapi/SDL_dynapi_procs.h
index 9030305e8827..dc9291c45250 100644
--- a/src/dynapi/SDL_dynapi_procs.h
+++ b/src/dynapi/SDL_dynapi_procs.h
@@ -447,7 +447,7 @@ SDL_DYNAPI_PROC(SDL_Haptic*,SDL_OpenHaptic,(SDL_HapticID a),(a),return)
 SDL_DYNAPI_PROC(SDL_Haptic*,SDL_OpenHapticFromJoystick,(SDL_Joystick *a),(a),return)
 SDL_DYNAPI_PROC(SDL_Haptic*,SDL_OpenHapticFromMouse,(void),(),return)
 SDL_DYNAPI_PROC(int,SDL_PauseHaptic,(SDL_Haptic *a),(a),return)
-SDL_DYNAPI_PROC(unsigned int,SDL_GetHapticFeatures,(SDL_Haptic *a),(a),return)
+SDL_DYNAPI_PROC(Uint32,SDL_GetHapticFeatures,(SDL_Haptic *a),(a),return)
 SDL_DYNAPI_PROC(int,SDL_InitHapticRumble,(SDL_Haptic *a),(a),return)
 SDL_DYNAPI_PROC(int,SDL_PlayHapticRumble,(SDL_Haptic *a, float b, Uint32 c),(a,b,c),return)
 SDL_DYNAPI_PROC(int,SDL_StopHapticRumble,(SDL_Haptic *a),(a),return)
diff --git a/src/haptic/SDL_haptic.c b/src/haptic/SDL_haptic.c
index e12db0980da8..1fc666c379f3 100644
--- a/src/haptic/SDL_haptic.c
+++ b/src/haptic/SDL_haptic.c
@@ -366,7 +366,7 @@ int SDL_GetMaxHapticEffectsPlaying(SDL_Haptic *haptic)
     return haptic->nplaying;
 }
 
-unsigned int SDL_GetHapticFeatures(SDL_Haptic *haptic)
+Uint32 SDL_GetHapticFeatures(SDL_Haptic *haptic)
 {
     CHECK_HAPTIC_MAGIC(haptic, 0);
 
diff --git a/src/haptic/SDL_syshaptic.h b/src/haptic/SDL_syshaptic.h
index bb12613b9bfe..31c2cf6b0312 100644
--- a/src/haptic/SDL_syshaptic.h
+++ b/src/haptic/SDL_syshaptic.h
@@ -48,7 +48,7 @@ struct SDL_Haptic
     struct haptic_effect *effects; /* Allocated effects */
     int neffects; /* Maximum amount of effects */
     int nplaying; /* Maximum amount of effects to play at the same time */
- unsigned int supported; /* Supported effects */
+ Uint32 supported; /* Supported effects and features */
     int naxes; /* Number of axes on the device. */
 
     struct haptic_hwdata *hwdata; /* Driver dependent */
diff --git a/src/haptic/darwin/SDL_syshaptic.c b/src/haptic/darwin/SDL_syshaptic.c
index 11ed1a73bba9..54ae0502b352 100644
--- a/src/haptic/darwin/SDL_syshaptic.c
+++ b/src/haptic/darwin/SDL_syshaptic.c
@@ -445,8 +445,7 @@ static unsigned int GetSupportedFeatures(SDL_Haptic *haptic)
     /* Test for effects. */
     FF_TEST(FFCAP_ET_CONSTANTFORCE, SDL_HAPTIC_CONSTANT);
     FF_TEST(FFCAP_ET_RAMPFORCE, SDL_HAPTIC_RAMP);
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* FF_TEST(FFCAP_ET_SQUARE, SDL_HAPTIC_SQUARE); */
+ FF_TEST(FFCAP_ET_SQUARE, SDL_HAPTIC_SQUARE);
     FF_TEST(FFCAP_ET_SINE, SDL_HAPTIC_SINE);
     FF_TEST(FFCAP_ET_TRIANGLE, SDL_HAPTIC_TRIANGLE);
     FF_TEST(FFCAP_ET_SAWTOOTHUP, SDL_HAPTIC_SAWTOOTHUP);
@@ -862,8 +861,7 @@ static int SDL_SYS_ToFFEFFECT(SDL_Haptic *haptic, FFEFFECT *dest, SDL_HapticEffe
         break;
 
     case SDL_HAPTIC_SINE:
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* case SDL_HAPTIC_SQUARE: */
+ case SDL_HAPTIC_SQUARE:
     case SDL_HAPTIC_TRIANGLE:
     case SDL_HAPTIC_SAWTOOTHUP:
     case SDL_HAPTIC_SAWTOOTHDOWN:
@@ -1077,9 +1075,8 @@ SDL_SYS_HapticEffectType(Uint16 type)
     case SDL_HAPTIC_RAMP:
         return kFFEffectType_RampForce_ID;
 
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* case SDL_HAPTIC_SQUARE:
- return kFFEffectType_Square_ID; */
+ case SDL_HAPTIC_SQUARE:
+ return kFFEffectType_Square_ID;
 
     case SDL_HAPTIC_SINE:
         return kFFEffectType_Sine_ID;
diff --git a/src/haptic/linux/SDL_syshaptic.c b/src/haptic/linux/SDL_syshaptic.c
index 01b82883ed31..53a1c1f3f892 100644
--- a/src/haptic/linux/SDL_syshaptic.c
+++ b/src/haptic/linux/SDL_syshaptic.c
@@ -100,8 +100,7 @@ static int EV_IsHaptic(int fd)
     /* Convert supported features to SDL_HAPTIC platform-neutral features. */
     EV_TEST(FF_CONSTANT, SDL_HAPTIC_CONSTANT);
     EV_TEST(FF_SINE, SDL_HAPTIC_SINE);
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* EV_TEST(FF_SQUARE, SDL_HAPTIC_SQUARE); */
+ EV_TEST(FF_SQUARE, SDL_HAPTIC_SQUARE);
     EV_TEST(FF_TRIANGLE, SDL_HAPTIC_TRIANGLE);
     EV_TEST(FF_SAW_UP, SDL_HAPTIC_SAWTOOTHUP);
     EV_TEST(FF_SAW_DOWN, SDL_HAPTIC_SAWTOOTHDOWN);
@@ -761,8 +760,7 @@ static int SDL_SYS_ToFFEffect(struct ff_effect *dest, SDL_HapticEffect *src)
         break;
 
     case SDL_HAPTIC_SINE:
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* case SDL_HAPTIC_SQUARE: */
+ case SDL_HAPTIC_SQUARE:
     case SDL_HAPTIC_TRIANGLE:
     case SDL_HAPTIC_SAWTOOTHUP:
     case SDL_HAPTIC_SAWTOOTHDOWN:
@@ -785,9 +783,8 @@ static int SDL_SYS_ToFFEffect(struct ff_effect *dest, SDL_HapticEffect *src)
         /* Periodic */
         if (periodic->type == SDL_HAPTIC_SINE) {
             dest->u.periodic.waveform = FF_SINE;
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* else if (periodic->type == SDL_HAPTIC_SQUARE)
- dest->u.periodic.waveform = FF_SQUARE; */
+ } else if (periodic->type == SDL_HAPTIC_SQUARE) {
+ dest->u.periodic.waveform = FF_SQUARE;
         } else if (periodic->type == SDL_HAPTIC_TRIANGLE) {
             dest->u.periodic.waveform = FF_TRIANGLE;
         } else if (periodic->type == SDL_HAPTIC_SAWTOOTHUP) {
diff --git a/src/haptic/windows/SDL_dinputhaptic.c b/src/haptic/windows/SDL_dinputhaptic.c
index 7c7f9ff64e51..0b7513497f7d 100644
--- a/src/haptic/windows/SDL_dinputhaptic.c
+++ b/src/haptic/windows/SDL_dinputhaptic.c
@@ -260,8 +260,7 @@ static BOOL CALLBACK DI_EffectCallback(LPCDIEFFECTINFO pei, LPVOID pv)
     EFFECT_TEST(GUID_ConstantForce, SDL_HAPTIC_CONSTANT);
     EFFECT_TEST(GUID_CustomForce, SDL_HAPTIC_CUSTOM);
     EFFECT_TEST(GUID_Sine, SDL_HAPTIC_SINE);
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* EFFECT_TEST(GUID_Square, SDL_HAPTIC_SQUARE); */
+ EFFECT_TEST(GUID_Square, SDL_HAPTIC_SQUARE);
     EFFECT_TEST(GUID_Triangle, SDL_HAPTIC_TRIANGLE);
     EFFECT_TEST(GUID_SawtoothUp, SDL_HAPTIC_SAWTOOTHUP);
     EFFECT_TEST(GUID_SawtoothDown, SDL_HAPTIC_SAWTOOTHDOWN);
@@ -678,8 +677,7 @@ static int SDL_SYS_ToDIEFFECT(SDL_Haptic *haptic, DIEFFECT *dest,
         break;
 
     case SDL_HAPTIC_SINE:
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* case SDL_HAPTIC_SQUARE: */
+ case SDL_HAPTIC_SQUARE:
     case SDL_HAPTIC_TRIANGLE:
     case SDL_HAPTIC_SAWTOOTHUP:
     case SDL_HAPTIC_SAWTOOTHDOWN:
@@ -889,9 +887,8 @@ static REFGUID SDL_SYS_HapticEffectType(SDL_HapticEffect *effect)
     case SDL_HAPTIC_RAMP:
         return &GUID_RampForce;
 
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* case SDL_HAPTIC_SQUARE:
- return &GUID_Square; */
+ case SDL_HAPTIC_SQUARE:
+ return &GUID_Square;
 
     case SDL_HAPTIC_SINE:
         return &GUID_Sine;
diff --git a/test/testhaptic.c b/test/testhaptic.c
index c3d703413f0d..e4ab85daf51b 100644
--- a/test/testhaptic.c
+++ b/test/testhaptic.c
@@ -351,9 +351,8 @@ HapticPrintSupported(SDL_Haptic *ptr)
     if (supported & SDL_HAPTIC_SINE) {
         SDL_Log(" sine\n");
     }
- /* !!! FIXME: put this back when we have more bits in 2.1 */
- /* if (supported & SDL_HAPTIC_SQUARE)
- SDL_Log(" square\n"); */
+ if (supported & SDL_HAPTIC_SQUARE)
+ SDL_Log(" square\n");
     if (supported & SDL_HAPTIC_TRIANGLE) {
         SDL_Log(" triangle\n");
     }

```