-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Update TaskFollowPointRoute.md #943
Conversation
Added flags and updated the name of the intunknown
RIP - I guess thats a no... 😢 |
The last argument should be
|
CI failed because you didn't write followpointroutemode in the same way as in the parameters. Anyway, the correct name of this parameter is |
Made requested changes...hope this follows the pattern needed
Done - Added changes. please approve for moneys |
TASK/TaskFollowPointRoute.md
Outdated
@@ -5,8 +5,15 @@ ns: TASK | |||
|
|||
```c | |||
// 0x595583281858626E 0xB837C816 | |||
void TASK_FOLLOW_POINT_ROUTE(Ped ped, float speed, int unknown); | |||
void TASK_FOLLOW_POINT_ROUTE(Ped ped, float speed, int mode); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
routeMode
would be a better name
(Hopefully) fixed enum so that it's formatted properly! Used the above example...as an example
Committing to change to the last parameter name pending any additional enum changes needed
Hopefully this is the appropriate change. Still willing to try again if not. Thanks!
Clean up old remnants and port over usage example.
Hopefully this follows the recent advice given, nonetheless, roastme! :D
Added flags and updated the name of the intunknown
Before you submit this PR, please make sure: