OpenRCT2/OpenRCT2

Pattern Findings

493 matches across 8 categories. Click a row to expand file-level details.

Over-Commented Block480 hits · 480 pts

Severity	File	Line	Snippet
LOW	test/tests/PlayTests.cpp	21	#include <openrct2/actions/park/ParkSetParameterAction.h>
LOW	test/tests/FormattingTests.cpp	1	/*****************************************************************************
LOW	test/tests/Pathfinding.cpp	1	#include "TestData.h"
LOW	test/tests/tests_pch.h	1	#pragma once
LOW	test/tests/S6ImportExportTests.cpp	21	#include <openrct2/core/Crypt.h>
LOW	test/tests/tests_pch.cpp	1	// Empty file for MSVC's PCH
LOW	resources/version.h	1	#define OPENRCT2_FILE_VERSION 0, 0, 0, 0
LOW	resources/resource.h	1	//{{NO_DEPENDENCIES}}
LOW	scripts/run-clang-format.py	141	if args.style:
LOW	src/openrct2/EditorObjectSelectionSession.cpp	21	#include "entity/Guest.h"
LOW	src/openrct2/ReplayManager.cpp	21	#include "actions/cheats/CheatSetAction.h"
LOW	src/openrct2/ReplayManager.cpp	41	#include "object/ObjectManager.h"
LOW	src/openrct2/Version.h	21	#elif defined(__aarch64__) \|\| defined(_M_ARM64)
LOW	src/openrct2/Version.h	41	#define OPENRCT2_ARCHITECTURE "wasm64"
LOW	src/openrct2/Version.h	61	#define OPENRCT2_PLATFORM "FreeBSD"
LOW	src/openrct2/GameState.h	21	#include "ride/Ride.h"
LOW	src/openrct2/Version.cpp	41	#if defined(OPENRCT2_BRANCH) && defined(OPENRCT2_COMMIT_SHA1_SHORT)
LOW	src/openrct2/Context.cpp	21	#include "GameStateSnapshots.h"
LOW	src/openrct2/Context.cpp	41	#include "core/Timer.hpp"
LOW	src/openrct2/Context.cpp	61	#include "park/ParkFile.h"
LOW	src/openrct2/openrct2_pch.h	1	#pragma once
LOW	src/openrct2/openrct2_pch.h	21	#include "Context.h"
LOW	src/openrct2/openrct2_pch.cpp	1	// Empty file for MSVC's PCH
LOW	src/openrct2/Diagnostic.h	41	* The defines will be either 0 or 1 so compiler will complain about undefined
LOW	src/openrct2/Diagnostic.h	61	#define DEBUG_LEVEL_1 0
LOW	src/openrct2/Diagnostic.h	81	#define DIAGNOSTIC_LOG_MACRO(level, format, ...)
LOW	src/openrct2/Editor.cpp	21	#include "actions/park/LandBuyRightsAction.h"
LOW	src/openrct2/Editor.cpp	41	#include "rct1/RCT1.h"
LOW	src/openrct2/GameState.cpp	21	#include "entity/PatrolArea.h"
LOW	src/openrct2/Game.cpp	21	#include "ParkImporter.h"
LOW	src/openrct2/Game.cpp	41	#include "entity/Staff.h"
LOW	src/openrct2/Game.cpp	61	#include "scenario/Scenario.h"
LOW	src/openrct2/rct1/S4Importer.cpp	1	/*****************************************************************************
LOW	src/openrct2/rct1/S4Importer.cpp	21	#include "../core/Guard.hpp"
LOW	src/openrct2/rct1/S4Importer.cpp	41	#include "../object/ObjectManager.h"
LOW	src/openrct2/rct1/S4Importer.cpp	61	#include "../world/MapAnimation.h"
LOW	src/openrct2/rct1/S4Importer.cpp	1381	dst->PathfindGoal.z = 0xFF;
LOW	src/openrct2/rct1/T4Importer.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/Paint.cpp	21	#include "../localisation/Currency.h"
LOW	src/openrct2/paint/Paint.Entity.cpp	21	#include "../profiling/Profiling.h"
LOW	src/openrct2/paint/track/thrill/3dCinema.cpp	1	/*****************************************************************************
LOW	…c/openrct2/paint/track/thrill/SwingingInverterShip.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/thrill/MagicCarpet.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/thrill/SwingingShip.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/thrill/Enterprise.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/thrill/Twist.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/thrill/TopSpin.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/thrill/MotionSimulator.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/transport/Chairlift.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/transport/MiniatureRailway.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/water/RiverRapids.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/gentle/SpaceRings.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/gentle/FerrisWheel.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/gentle/MerryGoRound.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/gentle/MiniGolf.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/gentle/Maze.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/gentle/HauntedHouse.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/gentle/Circus.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/gentle/CrookedHouse.cpp	1	/*****************************************************************************
LOW	src/openrct2/paint/track/shops/Facility.cpp	1	/*****************************************************************************
420 more matches not shown…

Synthetic Comment Markers1 hit · 8 pts

Severity	File	Line	Snippet
HIGH	src/openrct2/peep/PeepAnimations.cpp	197	// Get available costumes, excluding from random placement as requested

Verbosity Indicators3 hits · 4 pts

Severity	File	Line	Snippet
LOW	src/openrct2/actions/terraform/LandSetHeightAction.cpp	114	// We need to check if there is _currently_ a level crossing on the tile.
LOW	…droid/app/src/main/java/org/libsdl/app/SDLSurface.java	230	// We need to check if we're in relative mouse mode and get the axis offset rather than the x/y values
LOW	src/openrct2-ui/windows/TextInput.cpp	432	// The window can be potentially closed within a callback, we need to check if its still alive.

Excessive Try-Catch Wrapping2 hits · 3 pts

Severity	File	Line	Snippet
LOW	scripts/run-clang-format.py	124	except Exception as e:
MEDIUM	scripts/run-clang-format.py	118	def run_clang_format_diff_wrapper(args, file):

Deep Nesting3 hits · 3 pts

Severity	File	Line
LOW	scripts/run-clang-format.py	45
LOW	scripts/run-clang-format.py	63
LOW	scripts/run-clang-format.py	211

AI Slop Vocabulary1 hit · 3 pts

Severity	File	Line	Snippet
MEDIUM	…droid/app/src/main/java/org/libsdl/app/SDLSurface.java	231	// if we are. We'll leverage our existing mouse motion listener

Unused Imports2 hits · 2 pts

Severity	File	Line	Snippet
LOW	scripts/run-clang-format.py	13
LOW	scripts/run-clang-format.py	13

Hyper-Verbose Identifiers1 hit · 1 pts

Severity	File	Line	Snippet
LOW	scripts/run-clang-format.py	118	def run_clang_format_diff_wrapper(args, file):

Score History

Severity Breakdown

Pattern Findings