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

Use {.reorder: on.} by default #36

Open
genotrance opened this issue Aug 20, 2018 · 0 comments
Open

Use {.reorder: on.} by default #36

genotrance opened this issue Aug 20, 2018 · 0 comments

Comments

@genotrance
Copy link
Owner

Setting this pragma by default will help all wrapper creators considering C/C++ is capable of forward references.

Need to table this for the near term since it is only working correctly in 0.19.0 devel and nimgen and wrappers need to continue to support older versions of Nim. Perhaps nimgen could do a Nim version test and add the flag if the version of Nim is new enough. Of course, if people go back and forth with choosenim, an installed wrapper might break randomly.

In the near term, the move flag can be used to move things around so that backwards compatibility is maintained. E.g. from nimui

Long term, it is preferable to avoid such complex regexes and rely on {.reorder: on.} instead.

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

No branches or pull requests

1 participant