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 auto bindings for public members #184

Merged
merged 2 commits into from
Jul 30, 2015
Merged

Add auto bindings for public members #184

merged 2 commits into from
Jul 30, 2015

Conversation

pandamicro
Copy link
Contributor

Original idea and implementation from #139 of @samuele3hu

Based on his implementation, I added field to ini configuration to permit adding fields that need to be bound. By default no public field is bound to ensure compatibility with the current cocos2d code base, because there are already some public fields bound manually. In the future, they should be totally replaced by auto bindings, and field should be removed from the configuration.

@dualface I have added the original implementation of @samuele3hu for Lua bindings.
You can improve it and in the future, auto binding should be able to bind lua public members.

Thank you @samuele3hu !

@pandamicro
Copy link
Contributor Author

Task for cocos2d/cocos2d-x#9958

pandamicro added a commit that referenced this pull request Jul 30, 2015
Add auto bindings for public members
@pandamicro pandamicro merged commit 067ef5d into cocos2d:v3 Jul 30, 2015
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