RPCS3/rpcs3

Pattern Findings

1059 matches across 4 categories. Click a row to expand file-level details.

Over-Commented Block1045 hits · 770 pts

Severity	File	Line	Snippet
LOW	objdump.cpp	1	// objdump injection utility for Linux perf tools.
LOW	Utilities/StrFmt.cpp	1	#include "StrFmt.h"
LOW	Utilities/Thread.cpp	1	#include "stdafx.h"
LOW	Utilities/Thread.cpp	21
LOW	Utilities/Thread.cpp	41	#include <mach/thread_policy.h>
LOW	Utilities/Thread.cpp	61	#endif
LOW	Utilities/Thread.cpp	81	# define KP_FLAGS kp_proc.p_flag
LOW	Utilities/Thread.cpp	1021	#define XMMREG(context, reg) (reinterpret_cast<v128*>(&(context)->uc_mcontext.fpregs->_xmm[reg]))
LOW	Utilities/lockless.h	1	#pragma once
LOW	Utilities/sync.h	1	#pragma once
LOW	Utilities/Config.h	1	#pragma once
LOW	Utilities/rXml.h	1	#pragma once
LOW	Utilities/CRC.h	41	#define crcpp_uint8 - Specifies the type used to store CRCs that have a width of 8 bits or l
LOW	Utilities/CRC.h	61
LOW	Utilities/CRC.h	81	/// @brief Unsigned 8-bit integer definition, used primarily for parameter definitions.
LOW	Utilities/CRC.h	101	/// @brief Unsigned 32-bit integer definition, used primarily for parameter definitions.
LOW	Utilities/CRC.h	121	/// @brief Unsigned size definition, used for specifying data sizes.
LOW	Utilities/File.h	1	#pragma once // No BOM and only basic ASCII in this header, or a neko will die
LOW	Utilities/JITLLVM.cpp	1	#include "util/types.hpp"
LOW	Utilities/JITLLVM.cpp	21	#ifdef LLVM_AVAILABLE
LOW	Utilities/JITLLVM.cpp	41	#include "llvm/ExecutionEngine/ExecutionEngine.h"
LOW	Utilities/JIT.h	1	#pragma once
LOW	Utilities/JIT.h	21	#pragma GCC diagnostic ignored "-Wunused-parameter"
LOW	Utilities/stack_trace.cpp	1	#include "stdafx.h"
LOW	Utilities/File.cpp	1	#include "File.h"
LOW	Utilities/File.cpp	141
LOW	Utilities/bin_patch.cpp	1	#include "bin_patch.h"
LOW	Utilities/JITASM.cpp	1	#include "util/types.hpp"
LOW	rpcs3/display_sleep_control.cpp	1	#include "display_sleep_control.h"
LOW	rpcs3/stdafx.h	1	#pragma once // No BOM and only basic ASCII in this header, or a neko will die
LOW	rpcs3/stdafx.h	21	#include <functional> // IWYU pragma: export
LOW	rpcs3/headless_application.cpp	1	#include "headless_application.h"
LOW	rpcs3/rpcs3.cpp	1	// Qt6 frontend implementation for rpcs3. Known to work on Windows, Linux, Mac
LOW	rpcs3/rpcs3.cpp	21	#include "rpcs3qt/curl_handle.h"
LOW	rpcs3/rpcs3.cpp	41	// Copied from ntdef.h
LOW	rpcs3/rpcs3.cpp	61	#include <dispatch/dispatch.h>
LOW	rpcs3/main_application.cpp	1	#include "stdafx.h"
LOW	rpcs3/main_application.cpp	21	#include "Emu/Io/MouseHandler.h"
LOW	rpcs3/module_verifier.cpp	1	#include "stdafx.h"
LOW	rpcs3/resource.h	1	//{{NO_DEPENDENCIES}}
LOW	rpcs3/rpcs3qt/debugger_add_bp_window.cpp	1	#include "debugger_add_bp_window.h"
LOW	rpcs3/rpcs3qt/debugger_frame.h	1	#pragma once
LOW	rpcs3/rpcs3qt/screenshot_manager_dialog.h	1	#pragma once
LOW	rpcs3/rpcs3qt/pkg_install_dialog.cpp	1	#include "pkg_install_dialog.h"
LOW	rpcs3/rpcs3qt/game_list_frame.cpp	1	#include "game_list_frame.h"
LOW	rpcs3/rpcs3qt/game_list_frame.cpp	21	#include "util/types.hpp"
LOW	rpcs3/rpcs3qt/cheat_manager.h	1	#pragma once
LOW	rpcs3/rpcs3qt/camera_settings_dialog.h	1	#pragma once
LOW	rpcs3/rpcs3qt/screenshot_manager_dialog.cpp	1	#include "stdafx.h"
LOW	rpcs3/rpcs3qt/settings_dialog.cpp	1	#include <QButtonGroup>
LOW	rpcs3/rpcs3qt/settings_dialog.cpp	21	#include "tooltips.h"
LOW	rpcs3/rpcs3qt/gs_frame.h	1	#pragma once
LOW	rpcs3/rpcs3qt/game_list_table.cpp	1	#include "stdafx.h"
LOW	rpcs3/rpcs3qt/pad_motion_settings_dialog.h	1	#pragma once
LOW	rpcs3/rpcs3qt/gui_application.h	1	#pragma once
LOW	rpcs3/rpcs3qt/camera_settings_dialog.cpp	1	#include "stdafx.h"
LOW	rpcs3/rpcs3qt/save_manager_dialog.cpp	1	#include "save_manager_dialog.h"
LOW	rpcs3/rpcs3qt/trophy_manager_dialog.h	1	#pragma once
LOW	rpcs3/rpcs3qt/pad_settings_dialog.h	1	#pragma once
LOW	rpcs3/rpcs3qt/steam_utils.cpp	1	#include "stdafx.h"
985 more matches not shown…

AI Slop Vocabulary9 hits · 26 pts

Severity	File	Line	Snippet
MEDIUM	Utilities/JITLLVM.cpp	315	// Allocating more than one region does not work for relocations, needs more robust solution
MEDIUM	rpcs3/rpcs3qt/vfs_dialog_path_widget.cpp	22	// Code made robust even in case the path was manually removed from the list stored in "CurrentSettings.ini" file
MEDIUM	rpcs3/Emu/VFS.cpp	199	// Just to make the code more robust.
MEDIUM	rpcs3/Emu/VFS.cpp	382	// Just to make the code more robust.
MEDIUM	rpcs3/Emu/Cell/PPUThread.cpp	1781	bool non_leaf = false; // Absolutely not a leaf
MEDIUM	rpcs3/Emu/RSX/VK/VKCompute.cpp	267	// Technically robust buffer access should keep the driver from crashing in OOB situations
MEDIUM	rpcs3/Emu/RSX/GL/GLVertexProgram.cpp	288	// ln = (clip * 2) - 1 to fully utilize the 0-1 range of the depth buffer
MEDIUM	rpcs3/Emu/RSX/GL/GLCompute.cpp	205	// Technically robust buffer access should keep the driver from crashing in OOB situations
MEDIUM	rpcs3/util/atomic.cpp	158	// Essentially a fat semaphore

Redundant / Tautological Comments3 hits · 4 pts

Severity	File	Line	Snippet
LOW	.ci/setup-windows.sh	59	# Check if the file exists and the checksum is correct
LOW	.ci/setup-llvm.sh	34	# Check if the file exists and the checksum is correct
LOW	3rdparty/version_check.sh	117	# Print results

Verbosity Indicators2 hits · 3 pts

Severity	File	Line	Snippet
LOW	rpcs3/Crypto/unpkg.cpp	1278	// Use a seperate map for each reader. We need to check if the target app version exists for each package in sequence.
LOW	rpcs3/Emu/RSX/Overlays/overlay_controls.cpp	614	// The line may be wrapped, so we need to check if the next glyph's position is below the current position.

Score History

Severity Breakdown

Pattern Findings