Skip to content

Conversation

@DaniGP17
Copy link
Contributor

Goal of this PR

Remove the slow performance warning, since it provides no actionable information for users and can appear even on high-end systems, leading to confusion rather than helping diagnose real issues.

How is this PR achieving the goal

Remove the warn block

This PR applies to the following area(s)

FiveM, RedM

Successfully tested on

Game builds: none
Platforms: Windows, Linux

Checklist

  • Code compiles and has been tested successfully.
  • Code explains itself well and/or is documented.
  • My commit message explains what the changes do and what they are for.
  • No extra compilation warnings are added by these changes.

Fixes issues

fixes https://forum.cfx.re/t/suggestion-allow-toggle-remove-slow-system-performance-warning/5356888

@github-actions github-actions bot added RedM Issues/PRs related to RedM invalid Requires changes before it's considered valid and can be (re)triaged labels Nov 15, 2025
@radium-cfx
Copy link
Contributor

When removing this, we can also get rid of the logic around lastHitch and currentTotal and do some further cleanup there.

@DaniGP17 DaniGP17 force-pushed the tweak/remove-slow-perf-warn branch from 62f1f63 to 5e39e83 Compare November 18, 2025 19:02
@DaniGP17
Copy link
Contributor Author

Thanks, I've removed lastHitch and currentTotal, let me know what else do we need to clean 🙂

@radium-cfx radium-cfx added ready-to-merge This PR is enqueued for merging and removed invalid Requires changes before it's considered valid and can be (re)triaged RedM Issues/PRs related to RedM labels Nov 19, 2025
@prikolium-cfx prikolium-cfx merged commit 6155eff into citizenfx:master Nov 25, 2025
2 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR is enqueued for merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants