-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 ClearArea* of Cops, Peds and Projectiles. #819
Conversation
Don't arguments with their type changed need a cs_type to maintain compatibility with CS? |
Ah right, my bad. |
af95f7d
to
f318f53
Compare
MISC/ClearAreaOfCops.md
Outdated
``` | ||
flags appears to always be 0 | ||
Clears an area of cops at the given coordinates and radius. | ||
`createNetEvent` specifies whether a `CClearAreaEvent` event of should be created for online use. |
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.
Is this line necessary considering its already defined in the parameters? (same goes for the other natives)
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.
I just corrected it, thanks!
f318f53
to
84f5a6c
Compare
This PR modifies an incorrectly named parameter in the aforementioned natives. The parameter was previously named `flags`, but this parameter actually defines whether a net event of `CClearAreaEvent` should be created or not upon native execution. I'm not entirely sure if changing parameter types is ok to do so (as in won't break anything, never done such thing), if so I can just set the type back to integer, let me know if so, thanks. Correct cs_type decls for ClearArea* natives Correct cs_type declarations for ClearArea* natives to preserve compatibility with C# scripts. Preserve function signature Preserve function signature for ClearAreaOfProjectiles. Last parameter was changed from int to BOOL. Remove redundant description about createNetEvent
84f5a6c
to
713b892
Compare
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.
Lgtm
This PR modifies an incorrectly named parameter in the aforementioned natives.
The parameter was previously named
flags
, but this parameter actually defines whether a net event ofCClearAreaEvent
should be created or not upon native execution.I'm not entirely sure if changing parameter types is ok to do so (as in won't break anything, never done such thing), if so I can just set the type back to integer, let me know if so, thanks.