activa -Wextra -Wpedantic i neteja warnings

CMakeLists.txt

@@ -232,7 +232,7 @@ target_link_libraries(${PROJECT_NAME} PRIVATE SDL3::SDL3)
 
 # --- 4. CONFIGURACIÓN PLATAFORMAS Y COMPILADOR  ---
 # Configuración de flags de compilación
-target_compile_options(${PROJECT_NAME} PRIVATE -Wall)
+target_compile_options(${PROJECT_NAME} PRIVATE -Wall -Wextra -Wpedantic)
 target_compile_options(${PROJECT_NAME} PRIVATE $<$<CONFIG:RELEASE>:-Os -ffunction-sections -fdata-sections>)
 
 # Definir _DEBUG en modo Debug
@@ -393,7 +393,7 @@ if(NOT EMSCRIPTEN)
         source/core/resources/resource_pack.cpp
     )
     target_include_directories(pack_resources PRIVATE "${CMAKE_SOURCE_DIR}/source")
-    target_compile_options(pack_resources PRIVATE -Wall)
+    target_compile_options(pack_resources PRIVATE -Wall -Wextra -Wpedantic)
 
     # Regeneració automàtica de resources.pack en cada build si canvia data/.
     file(GLOB_RECURSE DATA_FILES CONFIGURE_DEPENDS "${CMAKE_SOURCE_DIR}/data/*")

source/core/rendering/background.cpp

@@ -83,11 +83,11 @@ void Background::initializeSprites() {
     const float TOP_CLOUDS_Y = base_ - 165;
     const float BOTTOM_CLOUDS_Y = base_ - 101;
 
-    top_clouds_sprite_a_ = std::make_unique<MovingSprite>(top_clouds_texture_, (SDL_FRect){.x = 0, .y = TOP_CLOUDS_Y, .w = rect_.w, .h = static_cast<float>(top_clouds_texture_->getHeight())});
-    top_clouds_sprite_b_ = std::make_unique<MovingSprite>(top_clouds_texture_, (SDL_FRect){.x = rect_.w, .y = TOP_CLOUDS_Y, .w = rect_.w, .h = static_cast<float>(top_clouds_texture_->getHeight())});
+    top_clouds_sprite_a_ = std::make_unique<MovingSprite>(top_clouds_texture_, SDL_FRect{.x = 0, .y = TOP_CLOUDS_Y, .w = rect_.w, .h = static_cast<float>(top_clouds_texture_->getHeight())});
+    top_clouds_sprite_b_ = std::make_unique<MovingSprite>(top_clouds_texture_, SDL_FRect{.x = rect_.w, .y = TOP_CLOUDS_Y, .w = rect_.w, .h = static_cast<float>(top_clouds_texture_->getHeight())});
 
-    bottom_clouds_sprite_a_ = std::make_unique<MovingSprite>(bottom_clouds_texture_, (SDL_FRect){.x = 0, .y = BOTTOM_CLOUDS_Y, .w = rect_.w, .h = static_cast<float>(bottom_clouds_texture_->getHeight())});
-    bottom_clouds_sprite_b_ = std::make_unique<MovingSprite>(bottom_clouds_texture_, (SDL_FRect){.x = rect_.w, .y = BOTTOM_CLOUDS_Y, .w = rect_.w, .h = static_cast<float>(bottom_clouds_texture_->getHeight())});
+    bottom_clouds_sprite_a_ = std::make_unique<MovingSprite>(bottom_clouds_texture_, SDL_FRect{.x = 0, .y = BOTTOM_CLOUDS_Y, .w = rect_.w, .h = static_cast<float>(bottom_clouds_texture_->getHeight())});
+    bottom_clouds_sprite_b_ = std::make_unique<MovingSprite>(bottom_clouds_texture_, SDL_FRect{.x = rect_.w, .y = BOTTOM_CLOUDS_Y, .w = rect_.w, .h = static_cast<float>(bottom_clouds_texture_->getHeight())});
 
     buildings_sprite_ = std::make_unique<Sprite>(buildings_texture_);
     gradient_sprite_ = std::make_unique<Sprite>(gradients_texture_, 0, 0, rect_.w, rect_.h);

source/core/rendering/fade.cpp

@@ -68,7 +68,7 @@ void Fade::render() {
 }
 
 // Actualiza las variables internas
-void Fade::update(float delta_time) {
+void Fade::update(float /*delta_time*/) {
     switch (state_) {
         case State::PRE:
             updatePreState();

source/core/rendering/gif.cpp

@@ -141,7 +141,7 @@ namespace GIF {
         }
 
         // Procesa el Image Descriptor y retorna el vector de datos sin comprimir.
-        auto processImageDescriptor(const uint8_t *&buffer, const std::vector<RGB> &gct, int resolution_bits) -> std::vector<uint8_t> {
+        auto processImageDescriptor(const uint8_t *&buffer, const std::vector<RGB> & /*gct*/, int /*resolution_bits*/) -> std::vector<uint8_t> {
             ImageDescriptor image_descriptor;
             readBytes(buffer, &image_descriptor, sizeof(ImageDescriptor));
 

source/core/rendering/sdl3gpu/sdl3gpu_shader.hpp

@@ -72,7 +72,7 @@ namespace Rendering {
             const std::string& fragment_source) -> bool override;
 
         void render() override;
-        void setTextureSize(float width, float height) override {}
+        void setTextureSize(float /*width*/, float /*height*/) override {}
         void cleanup() final;
         void destroy();
         [[nodiscard]] auto isHardwareAccelerated() const -> bool override { return is_initialized_; }
@@ -117,8 +117,40 @@ namespace Rendering {
         SDL_GPUTransferBuffer* upload_buffer_ = nullptr;
         SDL_GPUSampler* sampler_ = nullptr;
 
-        PostFXUniforms uniforms_{.vignette_strength = 0.6F, .chroma_min = 0.15F, .scanline_strength = 0.7F, .screen_height = 192.0F, .pixel_scale = 1.0F, .chroma_max = 0.15F, .scan_dark_ratio = 0.333F, .scan_dark_floor = 0.42F, .scan_edge_soft = 1.0F};
-        CrtPiUniforms crtpi_uniforms_{.scanline_weight = 6.0F, .scanline_gap_brightness = 0.12F, .bloom_factor = 3.5F, .input_gamma = 2.4F, .output_gamma = 2.2F, .mask_brightness = 0.80F, .curvature_x = 0.05F, .curvature_y = 0.10F, .mask_type = 2, .enable_scanlines = 1, .enable_multisample = 1, .enable_gamma = 1};
+        PostFXUniforms uniforms_{
+            .vignette_strength = 0.6F,
+            .chroma_min = 0.15F,
+            .scanline_strength = 0.7F,
+            .screen_height = 192.0F,
+            .mask_strength = 0.0F,
+            .gamma_strength = 0.0F,
+            .curvature = 0.0F,
+            .bleeding = 0.0F,
+            .pixel_scale = 1.0F,
+            .time = 0.0F,
+            .flicker = 0.0F,
+            .chroma_max = 0.15F,
+            .scan_dark_ratio = 0.333F,
+            .scan_dark_floor = 0.42F,
+            .scan_edge_soft = 1.0F,
+            .pad3 = 0.0F};
+        CrtPiUniforms crtpi_uniforms_{
+            .scanline_weight = 6.0F,
+            .scanline_gap_brightness = 0.12F,
+            .bloom_factor = 3.5F,
+            .input_gamma = 2.4F,
+            .output_gamma = 2.2F,
+            .mask_brightness = 0.80F,
+            .curvature_x = 0.05F,
+            .curvature_y = 0.10F,
+            .mask_type = 2,
+            .enable_scanlines = 1,
+            .enable_multisample = 1,
+            .enable_gamma = 1,
+            .enable_curvature = 0,
+            .enable_sharper = 0,
+            .texture_width = 0.0F,
+            .texture_height = 0.0F};
         ShaderType active_shader_ = ShaderType::POSTFX;
 
         int game_width_ = 0;

source/core/rendering/sprite/sprite.cpp

@@ -8,13 +8,13 @@
 // Constructor
 Sprite::Sprite(std::shared_ptr<Texture> texture, float pos_x, float pos_y, float width, float height)
     : textures_{std::move(texture)},
-      pos_((SDL_FRect){.x = pos_x, .y = pos_y, .w = width, .h = height}),
-      sprite_clip_((SDL_FRect){.x = 0, .y = 0, .w = pos_.w, .h = pos_.h}) {}
+      pos_(SDL_FRect{.x = pos_x, .y = pos_y, .w = width, .h = height}),
+      sprite_clip_(SDL_FRect{.x = 0, .y = 0, .w = pos_.w, .h = pos_.h}) {}
 
 Sprite::Sprite(std::shared_ptr<Texture> texture, SDL_FRect rect)
     : textures_{std::move(texture)},
       pos_(rect),
-      sprite_clip_((SDL_FRect){.x = 0, .y = 0, .w = pos_.w, .h = pos_.h}) {}
+      sprite_clip_(SDL_FRect{.x = 0, .y = 0, .w = pos_.w, .h = pos_.h}) {}
 
 Sprite::Sprite(std::shared_ptr<Texture> texture)
     : textures_{std::move(texture)},

source/core/rendering/text.cpp

@@ -32,7 +32,7 @@ Text::Text(const std::shared_ptr<Texture>& texture, const std::string& text_file
     }
 
     // Crea los objetos
-    sprite_ = std::make_unique<Sprite>(texture, (SDL_FRect){.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
+    sprite_ = std::make_unique<Sprite>(texture, SDL_FRect{.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
 
     // Inicializa variables
     fixed_width_ = false;
@@ -50,7 +50,7 @@ Text::Text(const std::shared_ptr<Texture>& texture, const std::shared_ptr<Text::
     }
 
     // Crea los objetos
-    sprite_ = std::make_unique<Sprite>(texture, (SDL_FRect){.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
+    sprite_ = std::make_unique<Sprite>(texture, SDL_FRect{.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
 
     // Inicializa variables
     fixed_width_ = false;
@@ -71,8 +71,8 @@ Text::Text(const std::shared_ptr<Texture>& texture, const std::shared_ptr<Textur
     }
 
     // Crea los objetos
-    sprite_ = std::make_unique<Sprite>(texture, (SDL_FRect){.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
-    white_sprite_ = std::make_unique<Sprite>(white_texture, (SDL_FRect){.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
+    sprite_ = std::make_unique<Sprite>(texture, SDL_FRect{.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
+    white_sprite_ = std::make_unique<Sprite>(white_texture, SDL_FRect{.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
 
     // Inicializa variables
     fixed_width_ = false;
@@ -90,8 +90,8 @@ Text::Text(const std::shared_ptr<Texture>& texture, const std::shared_ptr<Textur
     }
 
     // Crea los objetos
-    sprite_ = std::make_unique<Sprite>(texture, (SDL_FRect){.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
-    white_sprite_ = std::make_unique<Sprite>(white_texture, (SDL_FRect){.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
+    sprite_ = std::make_unique<Sprite>(texture, SDL_FRect{.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
+    white_sprite_ = std::make_unique<Sprite>(white_texture, SDL_FRect{.x = 0, .y = 0, .w = static_cast<float>(box_width_), .h = static_cast<float>(box_height_)});
 
     // Inicializa variables
     fixed_width_ = false;
@@ -137,7 +137,7 @@ void Text::write2X(int x, int y, const std::string& text, int kerning, int lengt
 }
 
 // Escribe el texto en una textura
-auto Text::writeToTexture(const std::string& text, int zoom, int kerning, int length) -> std::shared_ptr<Texture> {
+auto Text::writeToTexture(const std::string& text, int zoom, int kerning, int /*length*/) -> std::shared_ptr<Texture> {
     auto* renderer = Screen::get()->getRenderer();
     auto texture = std::make_shared<Texture>(renderer);
     auto width = Text::length(text, kerning) * zoom;
@@ -342,7 +342,7 @@ void Text::writeDX(Uint8 flags, int x, int y, const std::string& text, int kerni
 }
 
 // Escribe texto a partir de un TextStyle
-void Text::writeStyle(int x, int y, const std::string& text, const Style& style, int length) {
+void Text::writeStyle(int x, int y, const std::string& text, const Style& style, int /*length*/) {
     writeDX(style.flags, x, y, text, style.kerning, style.text_color, style.shadow_distance, style.shadow_color);
 }
 

source/core/rendering/tiled_bg.cpp

@@ -50,7 +50,7 @@ TiledBG::~TiledBG() {
 // Rellena la textura con el contenido
 void TiledBG::fillTexture() {
     // Crea los objetos para pintar en la textura de fondo
-    auto tile = std::make_unique<Sprite>(Resource::get()->getTexture("title_bg_tile.png"), (SDL_FRect){.x = 0, .y = 0, .w = TILE_WIDTH, .h = TILE_HEIGHT});
+    auto tile = std::make_unique<Sprite>(Resource::get()->getTexture("title_bg_tile.png"), SDL_FRect{.x = 0, .y = 0, .w = TILE_WIDTH, .h = TILE_HEIGHT});
 
     // Prepara para dibujar sobre la textura
     auto* temp = SDL_GetRenderTarget(renderer_);

source/core/system/shutdown.cpp

@@ -50,7 +50,7 @@ namespace SystemShutdown {
         return shutdownSystem(config);
     }
 
-    auto shutdownSystem(const ShutdownConfig& config) -> ShutdownResult {
+    auto shutdownSystem([[maybe_unused]] const ShutdownConfig& config) -> ShutdownResult {
 #ifdef _WIN32
         // Windows: Usar CreateProcess
         STARTUPINFOA si = {0};

source/core/system/system_utils.cpp

@@ -22,7 +22,7 @@ namespace SystemUtils {
 
     // Función auxiliar para crear una carpeta individual
     auto createSingleFolder(const std::string& path, int permissions) -> Result {
-        struct stat st = {.st_dev = 0};
+        struct stat st{};
 
         // Verificar si ya existe
         if (stat(path.c_str(), &st) == 0) {
@@ -131,7 +131,7 @@ namespace SystemUtils {
     }
 
     auto folderExists(const std::string& path) -> bool {
-        struct stat st = {.st_dev = 0};
+        struct stat st{};
         return (stat(path.c_str(), &st) == 0 && S_ISDIR(st.st_mode));
     }
 

source/game/gameplay/balloon_manager.cpp

@@ -114,7 +114,9 @@ void BalloonManager::deployRandomFormation(int stage) {
                     .size = balloon.size,
                     .vel_x = balloon.vel_x,
                     .game_tempo = balloon_speed_,
-                    .creation_counter = creation_time_enabled_ ? balloon.creation_counter : 0.0F};
+                    .creation_counter = creation_time_enabled_ ? balloon.creation_counter : 0.0F,
+                    .animation = {},
+                    .sound = {}};
                 createBalloon(config);
             }
 
@@ -135,7 +137,9 @@ void BalloonManager::deployFormation(int formation_id) {
             .size = balloon.size,
             .vel_x = balloon.vel_x,
             .game_tempo = balloon_speed_,
-            .creation_counter = balloon.creation_counter};
+            .creation_counter = balloon.creation_counter,
+            .animation = {},
+            .sound = {}};
         createBalloon(config);
     }
 }
@@ -151,7 +155,9 @@ void BalloonManager::deployFormation(int formation_id, float y) {
             .size = balloon.size,
             .vel_x = balloon.vel_x,
             .game_tempo = balloon_speed_,
-            .creation_counter = balloon.creation_counter};
+            .creation_counter = balloon.creation_counter,
+            .animation = {},
+            .sound = {}};
         createBalloon(config);
     }
 }
@@ -213,7 +219,9 @@ void BalloonManager::createChildBalloon(const std::shared_ptr<Balloon>& parent_b
             .size = static_cast<Balloon::Size>(static_cast<int>(parent_balloon->getSize()) - 1),
             .vel_x = direction == "LEFT" ? Balloon::VELX_NEGATIVE : Balloon::VELX_POSITIVE,
             .game_tempo = balloon_speed_,
-            .creation_counter = 0};
+            .creation_counter = 0,
+            .animation = {},
+            .sound = {}};
 
         // Crea el globo hijo
         auto child_balloon = createBalloon(config);
@@ -267,6 +275,8 @@ void BalloonManager::createPowerBall() {
             .texture = balloon_textures_.at(4),
             .animation = balloon_animations_.at(4),
             .sound = {
+                .bouncing_file = {},
+                .popping_file = {},
                 .bouncing_enabled = bouncing_sound_enabled_,
                 .poping_enabled = poping_sound_enabled_,
                 .enabled = sound_enabled_}};

source/game/options.hpp

@@ -262,7 +262,7 @@ namespace Options {
         void clearPlayers() { players_.clear(); }                                              // Limpia la lista de jugadores
 
         // Asigna el mando a un jugador
-        void assignTo(const Input::Gamepad& gamepad, Player::Id player_id) {
+        void assignTo(const Input::Gamepad& /*gamepad*/, Player::Id /*player_id*/) {
         }
 
         // Asigna los mandos físicos basándose en la configuración actual de nombres.

source/game/scenes/game.cpp

@@ -800,7 +800,7 @@ void Game::renderPathSprites() {
 }
 
 // Acciones a realizar cuando el jugador colisiona con un globo
-void Game::handlePlayerCollision(std::shared_ptr<Player>& player, const std::shared_ptr<Balloon>& balloon) {
+void Game::handlePlayerCollision(std::shared_ptr<Player>& player, const std::shared_ptr<Balloon>& /*balloon*/) {
     if (!player->isPlaying() || player->isInvulnerable()) {
         return;  // Si no está jugando o tiene inmunidad, no hace nada
     }

source/game/scenes/instructions.cpp

@@ -84,7 +84,7 @@ void Instructions::iniSprites() {
     // Inicializa los sprites
     for (int i = 0; std::cmp_less(i, item_textures_.size()); ++i) {
         auto sprite = std::make_unique<Sprite>(item_textures_[i], 0, 0, Item::WIDTH, Item::HEIGHT);
-        sprite->setPosition((SDL_FPoint){.x = sprite_pos_.x, .y = sprite_pos_.y + ((Item::HEIGHT + item_space_) * i)});
+        sprite->setPosition(SDL_FPoint{.x = sprite_pos_.x, .y = sprite_pos_.y + ((Item::HEIGHT + item_space_) * i)});
         sprites_.push_back(std::move(sprite));
     }
 }

source/game/ui/menu_option.hpp

@@ -39,11 +39,11 @@ class MenuOption {
 
     [[nodiscard]] virtual auto getBehavior() const -> Behavior = 0;
     [[nodiscard]] virtual auto getValueAsString() const -> std::string { return ""; }
-    virtual void adjustValue(bool adjust_up) {}
+    virtual void adjustValue(bool /*adjust_up*/) {}
     [[nodiscard]] virtual auto getTargetGroup() const -> ServiceMenu::SettingsGroup { return ServiceMenu::SettingsGroup::MAIN; }
     virtual void executeAction() {}
 
-    virtual auto getMaxValueWidth(Text* text_renderer) const -> int { return 0; }  // Método virtual para que cada opción calcule el ancho de su valor más largo
+    virtual auto getMaxValueWidth(Text* /*text_renderer*/) const -> int { return 0; }  // Método virtual para que cada opción calcule el ancho de su valor más largo
 
    protected:
     // --- Variables ---

source/game/ui/menu_renderer.cpp

@@ -45,7 +45,7 @@ void MenuRenderer::ShowHideAnimation::stop() {
     elapsed = 0.0F;
 }
 
-MenuRenderer::MenuRenderer(const ServiceMenu* menu_state, std::shared_ptr<Text> element_text, std::shared_ptr<Text> title_text)
+MenuRenderer::MenuRenderer(const ServiceMenu* /*menu_state*/, std::shared_ptr<Text> element_text, std::shared_ptr<Text> title_text)
     : element_text_(std::move(element_text)),
       title_text_(std::move(title_text)) {
     initializeMaxSizes();
@@ -145,7 +145,7 @@ void MenuRenderer::render(const ServiceMenu* menu_state) {
     }
 }
 
-void MenuRenderer::update(const ServiceMenu* menu_state, float delta_time) {
+void MenuRenderer::update(const ServiceMenu* /*menu_state*/, float delta_time) {
     updateAnimations(delta_time);
 
     if (visible_) {

source/game/ui/notifier.cpp

@@ -259,7 +259,7 @@ void Notifier::show(std::vector<std::string> texts, int icon, const std::string&
 
     // Dibuja el icono de la notificación
     if (has_icons_ && icon >= 0 && texts.size() >= 2) {
-        auto sp = std::make_unique<Sprite>(icon_texture_, (SDL_FRect){.x = 0, .y = 0, .w = ICON_SIZE, .h = ICON_SIZE});
+        auto sp = std::make_unique<Sprite>(icon_texture_, SDL_FRect{.x = 0, .y = 0, .w = ICON_SIZE, .h = ICON_SIZE});
         sp->setPosition({.x = PADDING_IN_H, .y = PADDING_IN_V, .w = ICON_SIZE, .h = ICON_SIZE});
         sp->setSpriteClip(SDL_FRect{
             .x = static_cast<float>(ICON_SIZE * (icon % 10)),