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

replaced "static unsigned const" to "static const unsigned" to avoid SWIG parsing error. #95

Merged
merged 1 commit into from
Nov 9, 2020

Conversation

0x0f0f0f
Copy link
Contributor

@0x0f0f0f 0x0f0f0f commented Nov 9, 2020

Solves issue #94 . This is hilarious and should be probably reported to SWIG.

…WIG can generate wrappers without a syntax error
@axsaucedo
Copy link
Member

Thanks for the contribution! Looks good to me, I'll run teh tests and make sure there are no issue with the end to end examples (ie android, python bindings, etc) and will merge. Appreciate it.

@axsaucedo axsaucedo added the enhancement New feature or request label Nov 9, 2020
@axsaucedo
Copy link
Member

All tests and examples pass, looks good, thank you @0x0f0f0f !

@axsaucedo axsaucedo merged commit 562c6e8 into KomputeProject:master Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants