jaildesigner-jailgames/projecte-2026
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

elimina 4 NOLINT cognitive-complexity obsolets restants a map_editor

Browse Source
This commit is contained in:
Sergio Valor
2026-05-17 22:07:32 +02:00
parent 464abd505a
commit 20ed92965b
1 changed files with 0 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
source/game/editor/map_editor.cpp
-4
View File
@@ -708,7 +708,6 @@ void MapEditor::handleMouseUp() {
} }
// Commit de un drag de entidad (initial, bound1, bound2) para cualquier EntityType. // Commit de un drag de entidad (initial, bound1, bound2) para cualquier EntityType.
// NOLINTNEXTLINE(readability-function-cognitive-complexity) -- switch sobre EntityType con una rama por tipo; refactor a visitor requiere cambio de diseño.
auto MapEditor::commitEntityDrag() -> bool { auto MapEditor::commitEntityDrag() -> bool {
const int IDX = drag_.index; const int IDX = drag_.index;
const int SNAP_X = static_cast<int>(drag_.snap_x); const int SNAP_X = static_cast<int>(drag_.snap_x);
@@ -816,7 +815,6 @@ auto MapEditor::commitEntityDrag() -> bool {
} }
// Mueve visualmente la entidad arrastrada a la posición snapped. // Mueve visualmente la entidad arrastrada a la posición snapped.
// NOLINTNEXTLINE(readability-function-cognitive-complexity) -- switch sobre EntityType con cases paralelos para cada tipo.
void MapEditor::moveEntityVisual() { void MapEditor::moveEntityVisual() {
switch (drag_.target) { switch (drag_.target) {
case DragTarget::ENTITY_INITIAL: case DragTarget::ENTITY_INITIAL:
@@ -958,7 +956,6 @@ void MapEditor::renderSelectionHighlight() {
} }
// Estampa el patrón del brush en la posición indicada (anclaje top-left). // Estampa el patrón del brush en la posición indicada (anclaje top-left).
// NOLINTNEXTLINE(readability-function-cognitive-complexity) -- nested loops + casos TRANSPARENT/ERASE/tile normal y ramas collision vs normal.
void MapEditor::stampBrushAt(int tile_x, int tile_y) { void MapEditor::stampBrushAt(int tile_x, int tile_y) {
if (brush_.isEmpty()) { return; } if (brush_.isEmpty()) { return; }
for (int dy = 0; dy < brush_.height; ++dy) { for (int dy = 0; dy < brush_.height; ++dy) {
@@ -1233,7 +1230,6 @@ auto MapEditor::entityLabel(EntityType type) -> const char* {
} }
// Dibuja marcadores de boundaries y líneas de ruta para enemigos y plataformas. // Dibuja marcadores de boundaries y líneas de ruta para enemigos y plataformas.
// NOLINTNEXTLINE(readability-function-cognitive-complexity) -- switch sobre EntityType con ramas de enemigo patrullante vs plataforma con waypoints.
void MapEditor::renderEntityBoundaries() { void MapEditor::renderEntityBoundaries() {
auto game_surface = Screen::get()->getRendererSurface(); auto game_surface = Screen::get()->getRendererSurface();
if (!game_surface) { return; } if (!game_surface) { return; }