Skip to content

Adds a crash handler to psyqo.

5fa392a
Select commit
Loading
Failed to load commit list.
Merged

Adds a crash handler to psyqo. #1914

Adds a crash handler to psyqo.
5fa392a
Select commit
Loading
Failed to load commit list.
CodeScene Delta Analysis / CodeScene Cloud Delta Analysis (main) failed Apr 4, 2025 in 31s

CodeScene PR Check

Quality Gate Failed

Gates Failed
New code is healthy (1 new file with code health below 9.00)
Enforce critical code health rules (1 file with Bumpy Road Ahead, Deep, Nested Complexity)
Enforce advisory code health rules (1 file with Complex Method)

Gates Passed
1 Quality Gates Passed

See analysis details in CodeScene

Reason for failure
New code is healthy Violations Code Health Impact
crash-handler.cpp 3 rules 10.00 → 8.50 Suppress
Enforce critical code health rules Violations Code Health Impact
crash-handler.cpp 2 critical rules 10.00 → 8.50 Suppress
Enforce advisory code health rules Violations Code Health Impact
crash-handler.cpp 1 advisory rule 10.00 → 8.50 Suppress

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

Details

🚩 Declining Code Health (highest to lowest):

  • Deep, Nested Complexity crash-handler.cpp: psyqo::Kernel::Internal::crashHandler
  • Complex Method crash-handler.cpp: psyqo::Kernel::Internal::crashHandler
  • Complex Method crash-handler.cpp: printString
  • Bumpy Road Ahead crash-handler.cpp: psyqo::Kernel::Internal::crashHandler

Annotations

Check warning on line 248 in src/mips/psyqo/src/crash-handler.cpp

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)

❌ New issue: Complex Method

psyqo::Kernel::Internal::crashHandler has a cyclomatic complexity of 10, threshold = 9. This function has many conditional statements (e.g. if, for, while), leading to lower code health. Avoid adding more conditionals and code to it without refactoring.

Check warning on line 101 in src/mips/psyqo/src/crash-handler.cpp

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)

❌ New issue: Complex Method

printString has a cyclomatic complexity of 10, threshold = 9. This function has many conditional statements (e.g. if, for, while), leading to lower code health. Avoid adding more conditionals and code to it without refactoring.

Check warning on line 248 in src/mips/psyqo/src/crash-handler.cpp

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)

❌ New issue: Bumpy Road Ahead

psyqo::Kernel::Internal::crashHandler has 3 blocks with nested conditional logic. Any nesting of 2 or deeper is considered. Threshold is one single, nested block per function. The Bumpy Road code smell is a function that contains multiple chunks of nested conditional logic. The deeper the nesting and the more bumps, the lower the code health.

Check warning on line 248 in src/mips/psyqo/src/crash-handler.cpp

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (main)

❌ New issue: Deep, Nested Complexity

psyqo::Kernel::Internal::crashHandler has a nested complexity depth of 4, threshold = 4. This function contains deeply nested logic such as if statements and/or loops. The deeper the nesting, the lower the code health.