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

replace nested functions with blocks on clang #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cute
Copy link

@cute cute commented Jul 14, 2016

No description provided.

@orangeduck
Copy link
Owner

Thanks this looks great 👍 Can you put it behind another processor flag as well as __clang__? Some people may either be using clang already with nested functions or not want this behaviour.

@XVilka
Copy link

XVilka commented Nov 28, 2017

Are there any updates on this PR?

@orangeduck
Copy link
Owner

Are you interested in using clang blocks for your own project? I can merge this in if there is some interest - just need to add an extra pre-processor flag to guard it.

@sjml
Copy link
Contributor

sjml commented Jul 8, 2019

FWIW, clang doesn't support GCC-style nested functions at all, so there wouldn't be anybody using clang with LuaAutoC from what I can tell. Thus there wouldn't be a need for an additional guard. Using blocks might be the only way to get it building on clang.

@orangeduck
Copy link
Owner

You can use clang with LuaAutoC you just need to have separate declaration and registration functions as mentioned in the FAQ under "nested functions": https://github.com/orangeduck/LuaAutoC#faq but yes in general in clang you need to use blocks instead of nested functions.

@sjml
Copy link
Contributor

sjml commented Jul 9, 2019

Ah, yes, my bad. Sorry I spoke overly-definitively!

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.

4 participants