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

Add default arguments #238

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add default arguments #238

wants to merge 2 commits into from

Conversation

Sainan
Copy link
Contributor

@Sainan Sainan commented Aug 28, 2023

I know that only 3 4 natives having this new attribute isn't gonna the most impressive start, but I'm hoping that in the long run, this will help simplify usage of natives:

  • Make it easier to call certain natives, e.g. SET_ENTITY_HEALTH's p2 will pretty much always be 0, no need to specify it explicitly.
  • Make it easier to upgrade to newer natives versions, e.g. if there's a new p2 or p3 it can just be defaulted to 0, so it has exactly the same value as it did before.

Interoperability considerations:

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

Successfully merging this pull request may close these issues.

1 participant