Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workaround for ped props limit in FiveM #2421

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

Disquse
Copy link
Contributor

@Disquse Disquse commented Mar 12, 2024

Goal of this PR

The game has a "soft limit" for the number of available ped props per anchor per model. This patch is intended to fix this limit to allow players to stream more custom ped props.

How is this PR achieving the goal

This PR achieves the goal by patching a set of game functions that clamp 32-bit ped prop indices to 8-bit integers. More detailed explanations are available in the code.

This PR applies to the following area(s)

FiveM

Successfully tested on

Game builds: 1604, 2060, 2189, 2372, 2545, 2612, 2699, 2802, 2944, 3095

Platforms: Windows

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

Addressing a platform suggestion left on the forum:
https://forum.cfx.re/t/request-to-increase-head-prop-limit-in-fivem/5201885

@github-actions github-actions bot added the triage Needs a preliminary assessment to determine the urgency and required action label Mar 12, 2024
@Moozdzn
Copy link

Moozdzn commented Mar 12, 2024

Closes #1486

The game has a "soft limit" for the number of available ped props per anchor per model. This patch is intended to fix this limit.
}

{
auto location = hook::get_pattern<char>("0F B6 C8 83 C8 FF 3B D8 7D 04");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the helmet bits in AppearanceDataNode being serialized with 10 bits be documented anywhere?

@gottfriedleibniz gottfriedleibniz added ready-to-merge This PR is enqueued for merging and removed triage Needs a preliminary assessment to determine the urgency and required action labels Mar 15, 2024
@thorium-cfx thorium-cfx merged commit 70dcfd4 into citizenfx:master Mar 19, 2024
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.

5 participants