Lint: bugs reales + enums uint8_t + use-equals-default
Resuelve la categoría de findings de tidy que son bugs reales o cambios
de tipo concretos, no transforms automáticos:
Bugs reales (bugprone-* y clang-diagnostic-*):
- bugprone-empty-catch en postfx_config.cpp: el catch silencioso del
parser RGB era intencional (fallback a defaults si el array no parsea
como int). Marcado con @INTENTIONAL (keyword ya configurado en
.clang-tidy via bugprone-empty-catch.IgnoreCatchWithKeywords) y
comentario ampliado explicando la decisión.
- clang-diagnostic-unused-private-field en Starfield: el campo
'densitat_' se asignaba en el constructor pero nunca se leía. El
parámetro 'densitat' se reparte directamente en las CapaConfig al
construir, así que el field era código muerto. Eliminado.
- bugprone-branch-clone en vector_text.cpp: el switch de
get_shape_filename tenía dos grupos consecutivos (dígitos 0-9 y
mayúsculas A-Z) con cuerpo idéntico. Fusionados en un único case con
comentario explicando que comparten path porque la shape se llama
igual que el caracter.
- bugprone-switch-missing-default-case en Input::handleEvent: el switch
manejaba solo SDL_EVENT_GAMEPAD_ADDED/REMOVED y caía por fall-through
a un return {} fuera del switch. Añadido default: explícito con
comentario sobre qué hace Input vs el resto del sistema.
- bugprone-implicit-widening-of-multiplication-result en
GameScene::bullets_ y Collision::Context::bullets: 'MAX_BALES * 2'
es int*int y se widening implícitamente a std::size_t para el
template arg de std::array. Cast explícito a size_t en ambos sitios.
Otros mecánicos:
- performance-enum-size: 10 enums sin tipo subyacente pasaron a
': std::uint8_t' (PrimitiveType, InputAction, Mode, SceneType,
Option, AnimationState, TitleState, ModeSpawn, EstatStage,
ShipState). #include <cstdint> añadido donde faltaba.
- modernize-use-equals-default en SpawnController: el ctor por
defecto tenía cuerpo vacío ({}). Pasado a '= default;'.
Cero supresiones. La única "marca" es @INTENTIONAL en el empty-catch,
que es el mecanismo configurado en el .clang-tidy del proyecto para
distinguir intencionales de accidentales.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
1 parent
c45e524109
commit
424d0d2b89
16 files changed
+33
-24
No files matched your search
@@ -3,6 +3,7 @@
|
||||
|
||||
#pragma once
|
||||
|
||||
#include <cstdint>
|
||||
#include <string>
|
||||
#include <vector>
|
||||
|
||||
@@ -11,7 +12,7 @@
|
||||
namespace Graphics {
|
||||
|
||||
// Tipo de primitiva dins de una shape
|
||||
enum class PrimitiveType {
|
||||
enum class PrimitiveType : std::uint8_t {
|
||||
POLYLINE, // Secuencia de points connectats
|
||||
LINE // Línia individual (2 points)
|
||||
};
|
||||
|
||||
@@ -21,8 +21,7 @@ Starfield::Starfield(Rendering::Renderer* renderer,
|
||||
: shape_estrella_(ShapeLoader::load("star.shp")),
|
||||
renderer_(renderer),
|
||||
punt_fuga_(punt_fuga),
|
||||
area_(area),
|
||||
densitat_(densitat) {
|
||||
area_(area) {
|
||||
if (!shape_estrella_ || !shape_estrella_->isValid()) {
|
||||
std::cerr << "ERROR: No s'ha pogut load star.shp" << '\n';
|
||||
return;
|
||||
|
||||
@@ -77,7 +77,6 @@ class Starfield {
|
||||
Vec2 punt_fuga_; // Vec2 de origin de las estrelles
|
||||
SDL_FRect area_; // Àrea activa
|
||||
float radi_max_; // Distancia màxima del centro al límit de pantalla
|
||||
int densitat_; // Nombre total de estrelles
|
||||
float multiplicador_brightness_{1.0F}; // Multiplicador de brightness (1.0 = default)
|
||||
};
|
||||
|
||||
|
||||
@@ -80,7 +80,9 @@ void VectorText::load_charset() {
|
||||
}
|
||||
|
||||
auto VectorText::get_shape_filename(char c) const -> std::string {
|
||||
// Mapeo carácter → nombre de archivo (con prefix "font/")
|
||||
// Mapeo carácter → nombre de archivo (con prefix "font/").
|
||||
// Dígitos 0-9 y mayúsculas A-Z comparten el mismo path: la shape se llama
|
||||
// como el caracter mismo, así que se agrupan en un único case.
|
||||
switch (c) {
|
||||
case '0':
|
||||
case '1':
|
||||
@@ -92,9 +94,6 @@ auto VectorText::get_shape_filename(char c) const -> std::string {
|
||||
case '7':
|
||||
case '8':
|
||||
case '9':
|
||||
return std::string("font/char_") + c + ".shp";
|
||||
|
||||
// Lletres majúscules A-Z
|
||||
case 'A':
|
||||
case 'B':
|
||||
case 'C':
|
||||
|
||||
Reference in new issue
Block a user