Skip to content

Conversation

@NoPressF
Copy link
Contributor

No description provided.

@AmyrAhmady
Copy link
Member

You must create a pull request in https://github.com/openmultiplayer/open.mp-sdk for your fork of SDK repository, because right now this PR is pointing to a commit hash that doesn't exist in our own SDK repository, it's only in yours.

Also you shouldn't add functions randomly in the middle of the virtual table (in ITimer in this example), new ones must be added to the end, after last virtual function. So this way we're not breaking ABI compatibility even more.

Copy link
Member

@AmyrAhmady AmyrAhmady left a comment

Choose a reason for hiding this comment

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

explained in comments

@NoPressF
Copy link
Contributor Author

@NoPressF
Copy link
Contributor Author

@AmyrAhmady what about that?

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.

2 participants