From d03e9fffdc8dc472576ac1d23dbcc20474fde3a9 Mon Sep 17 00:00:00 2001 From: Sergio Valor Date: Mon, 20 Apr 2026 12:24:55 +0200 Subject: [PATCH] =?UTF-8?q?refactor(core/audio):=20eliminar=204=20sobrec?= =?UTF-8?q?=C3=A0rregues=20enable/disable=20no-arg=20(INCONS-05/DEAD-05)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit enableSound()/disableSound()/enableMusic()/disableMusic() eren inline thunks que només cridaven la versió amb bool. Cap caller del projecte les invoca. Es deixa només enableSound(bool) + toggleSound() (i l'equivalent per música) com a API única. Queda pendent la divergència semàntica amb Ja::enableSound/enableMusic (Audio silencia per gain, Ja pararia els canals). No es resol aquí perquè és una decisió de disseny que lliga amb l'encapsulament de Ja::* (COUP-01). Co-Authored-By: Claude Opus 4.7 (1M context) --- docs/audit-core-audio.md | 4 ++-- source/core/audio/audio.hpp | 4 ---- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/docs/audit-core-audio.md b/docs/audit-core-audio.md index 8294c07..8fecae9 100644 --- a/docs/audit-core-audio.md +++ b/docs/audit-core-audio.md @@ -74,7 +74,7 @@ Los IDs se mantienen estables una vez asignados. Cada entrada apunta al(los) fic | INCONS-02 | `audio.hpp:44-48` + `jail_audio.hpp:44-50` | Dos `MusicState` paralelos + `getRealMusicState` como puente | Importante | DONE | Eliminat `music_.state` (cache) i `getRealMusicState`. `getMusicState()` consulta Ja::* directament i projecta INVALID/DISABLED → STOPPED. La font de veritat passa a ser Ja. `update()` ja no necessita sincronització | | INCONS-03 | `audio.cpp` varios | Comentarios mezclan catalán y español | Menor | PENDING | Unificar a catalán | | INCONS-04 | `audio.hpp:70,78` | `playMusic` `loop=-1` vs `playSound` `loop=0` | Menor | WONTFIX | Convención razonable (música loop / sonido one-shot); documentar | -| INCONS-05 | `audio.hpp:102-105` | 4 formas de enable/disable sound (API redundante) + divergencia con `Ja::enableSound` | Importante | PENDING | Dejar `enableSound(bool)` + `toggleSound()`; delegar a `Ja` | +| INCONS-05 | `audio.hpp:102-105` | 4 formas de enable/disable sound (API redundante) + divergencia con `Ja::enableSound` | Importante | DONE (parcial) | 4 variants no-arg eliminades. La divergència amb `Ja::enableSound/enableMusic` es manté: Audio silencia per gain, Ja pararia els canals; la decisió es posposa a COUP-01 | ## 6. Código muerto @@ -84,7 +84,7 @@ Los IDs se mantienen estables una vez asignados. Cada entrada apunta al(los) fic | DEAD-02 | `jail_audio.hpp:671-675` | `Ja::setVolume` master no usado desde `Audio` | Menor | DONE | Eliminat; Audio fa el gate per canal | | DEAD-03 | `jail_audio.hpp:346-350` | `getMusicFilename` presunto unused | Menor | DONE | Eliminat; `Ja::Music::filename` s'accedeix directe (PATCH-02) | | DEAD-04 | `jail_audio.hpp:633-638` | `getChannelState` sin getter público en `Audio` | Menor | DONE | Eliminat | -| DEAD-05 | `audio.hpp:102-103` | `enableSound()`/`disableSound()` sin argumento (duplican la versión con bool) | Menor | PENDING | Cubierto por INCONS-05 | +| DEAD-05 | `audio.hpp:102-103` | `enableSound()`/`disableSound()` sin argumento (duplican la versión con bool) | Menor | DONE | Eliminades (veure INCONS-05) | ## 7. Violaciones de convenciones diff --git a/source/core/audio/audio.hpp b/source/core/audio/audio.hpp index 15013db..6f0e6cc 100644 --- a/source/core/audio/audio.hpp +++ b/source/core/audio/audio.hpp @@ -96,14 +96,10 @@ class Audio { void applySettings(const Config& config); // Aplica una nova configuració // --- Configuración de sonidos --- - void enableSound() { enableSound(true); } // Habilitar sonidos - void disableSound() { enableSound(false); } // Deshabilitar sonidos void enableSound(bool value); // Establecer estado de sonidos (reaplica volum) void toggleSound() { enableSound(!sound_enabled_); } // Alternar estado de sonidos (reaplica volum) // --- Configuración de música --- - void enableMusic() { enableMusic(true); } // Habilitar música - void disableMusic() { enableMusic(false); } // Deshabilitar música void enableMusic(bool value); // Establecer estado de música (reaplica volum) void toggleMusic() { enableMusic(!music_enabled_); } // Alternar estado de música (reaplica volum)