Make mp_show_hintmessages respect bOverride to preserve ReAPI contract

Move the cvar check below the IsAlive gate and skip it when bOverride is
set, so it acts as a server-side sibling of the client _ah (auto-help)
preference rather than an unconditional block.

The hook chain terminal (HintMessageEx OrigFunc) still runs after all
plugin hooks, so RG_CBasePlayer_HintMessageEx keeps firing for vanilla
events. And because bOverride already bypasses m_bShowHints today, ReAPI
callers that need guaranteed delivery (rg_hint_message with override)
keep their existing escape hatch — no new contract is broken.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: belomaxorka

README.md

@@ -74,7 +74,7 @@ This means that plugins that do binary code analysis (Orpheu for example) probab
 | mp_show_radioicon                  | 1       | 0   | 1            | Show radio icon.<br/>`0` disabled<br/>`1` enabled |
 | mp_show_scenarioicon               | 0       | 0   | 1            | Show scenario icon in HUD such as count of alive hostages or ticking bomb.<br/>`0` disabled<br/>`1` enabled |
 | mp_show_bomb_timer                 | 0       | 0   | 1            | Show the time until the planted bomb explodes on the HUD round timer.<br/>`0` disabled<br/>`1` enabled (when the bomb is planted, the round timer shows the C4 countdown) |
-| mp_show_hintmessages               | 1       | 0   | 1            | Send hint messages (Hint_*) to clients.<br/>`0` disabled (hints are blocked server-side)<br/>`1` enabled |
+| mp_show_hintmessages               | 1       | 0   | 1            | Send hint messages (Hint_*) to clients.<br/>`0` disabled (non-forced hints are suppressed server-side; forced hints, e.g. ReAPI `rg_hint_message` with override, still pass)<br/>`1` enabled |
 | mp_old_bomb_defused_sound          | 1       | 0   | 1            | Play "Bomb has been defused" sound instead of "Counter-Terrorists win" when bomb was defused<br/>`0` disabled<br/>`1` enabled |
 | showtriggers                       | 0       | 0   | 1            | Debug cvar shows triggers. |
 | sv_alltalk                         | 0       | 0   | 5            | When players can hear each other ([further explanation](../../wiki/sv_alltalk)).<br/>`0` dead don't hear alive<br/>`1` no restrictions<br/>`2` teammates hear each other<br/>`3` Same as 2, but spectators hear everybody<br/>`4` alive hear alive, dead hear dead and alive.<br/>`5` alive hear alive teammates, dead hear dead and alive.

dist/game.cfg

@@ -207,7 +207,7 @@ mp_show_scenarioicon "0"
 mp_show_bomb_timer "0"
 
 // Send hint messages (Hint_*) to clients.
-// 0 - disabled (hints are blocked server-side)
+// 0 - disabled (non-forced hints suppressed server-side; forced hints still pass)
 // 1 - enabled (default behavior)
 //
 // Default value: "1"

regamedll/dlls/player.cpp

@@ -8108,14 +8108,17 @@ LINK_HOOK_CLASS_CHAIN(bool, CBasePlayer, HintMessageEx, (const char *pMessage, f
 
 bool EXT_FUNC CBasePlayer::__API_HOOK(HintMessageEx)(const char *pMessage, float duration, bool bDisplayIfPlayerDead, bool bOverride)
 {
-#ifdef REGAMEDLL_ADD
-	if (show_hintmessages.value == 0.0f)
-		return false;
-#endif
-
 	if (!bDisplayIfPlayerDead && !IsAlive())
 		return false;
 
+#ifdef REGAMEDLL_ADD
+	// Server-side sibling of the client's _ah (auto-help) preference: suppresses
+	// non-forced hints only. bOverride still bypasses it, exactly as it already
+	// bypasses m_bShowHints, so ReAPI callers keep their existing escape hatch.
+	if (!bOverride && show_hintmessages.value == 0.0f)
+		return true;
+#endif
+
 	if (bOverride || m_bShowHints)
 		return m_hintMessageQueue.AddMessage(pMessage, duration, true, nullptr);