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

[Canary 7255] GetVehicleHandlingFloat is broken #2307

Closed
DeForce opened this issue Dec 19, 2023 · 1 comment · Fixed by #2308
Closed

[Canary 7255] GetVehicleHandlingFloat is broken #2307

DeForce opened this issue Dec 19, 2023 · 1 comment · Fixed by #2308
Assignees
Labels

Comments

@DeForce
Copy link
Contributor

DeForce commented Dec 19, 2023

What happened?

RegisterCommand("handling-test", function()
    local vehicle = GetVehiclePedIsIn(PlayerPedId(), false)
    local maxSpeed = GetVehicleHandlingFloat(vehicle, 'CHandlingData', 'fInitialDriveMaxFlatVel')
    print(maxSpeed)
end)

Simple command executes on latest Stable, but fails on latest canary

[     83203] [b2802_GTAProce]             MainThrd/ __Lua_InvokeNative: execution failed: Error executing native 0x00000000642fc12f at address handling-loader-five.dll+27DFF.
[     83203] [b2802_GTAProce]             MainThrd/ ^1SCRIPT ERROR: Execution of native 00000000642fc12f in script host failed: Error executing native 0x00000000642fc12f at address handling-loader-five.dll+27DFF.^7
[     83203] [b2802_GTAProce]             MainThrd/ ^3> GetVehicleHandlingFloat^7 (^5GetVehicleHandlingFloat.lua^7:9)
[     83203] [b2802_GTAProce]             MainThrd/ ^3> ref^7 (^5@testing/client.lua^7:3)
[     83203] [b2802_GTAProce]             MainThrd/ ^3> ExecuteCommand^7 (^5ExecuteCommand.lua^7:4)
[     83203] [b2802_GTAProce]             MainThrd/ ^3> callback^7 (^5@chat/client/cl_chat.lua^7:114)

Expected result

Should print something and execute without error

Reproduction steps

Run command from above and on Stable no errors will be thrown.
On Canary there will be an error

Importancy

Unknown

Area(s)

FiveM

Specific version(s)

Canary 7255

Additional information

No response

@DeForce DeForce added bug triage Needs a preliminary assessment to determine the urgency and required action labels Dec 19, 2023
@Disquse Disquse self-assigned this Dec 19, 2023
@Disquse Disquse removed the triage Needs a preliminary assessment to determine the urgency and required action label Dec 19, 2023
@Disquse
Copy link
Contributor

Disquse commented Dec 19, 2023

Made a typo in CVehicle::GetHandlingData(). Thanks for the report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants