Skip to content

replace nested functions with blocks on clang#24

Open
cute wants to merge 1 commit into
orangeduck:masterfrom
cute:master
Open

replace nested functions with blocks on clang#24
cute wants to merge 1 commit into
orangeduck:masterfrom
cute:master

Conversation

@cute

@cute cute commented Jul 14, 2016

Copy link
Copy Markdown

No description provided.

@orangeduck

Copy link
Copy Markdown
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

XVilka commented Nov 28, 2017

Copy link
Copy Markdown

Are there any updates on this PR?

@orangeduck

Copy link
Copy Markdown
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

sjml commented Jul 8, 2019

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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

sjml commented Jul 9, 2019

Copy link
Copy Markdown
Contributor

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