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

Improve internal visibility support for C++ classes #91

Open
ghost opened this issue Jan 20, 2020 · 1 comment
Open

Improve internal visibility support for C++ classes #91

ghost opened this issue Jan 20, 2020 · 1 comment
Labels
c++ enhancement New feature or request

Comments

@ghost
Copy link

ghost commented Jan 20, 2020

Internal visibility is currently supported inside LimeIDL but has no effect on C++ generated classes. While the C++ language itself doesn't have support for this sort of visibility, I would propose a solution that separates the public vs the internal generated classes, maybe at the file system + namespace level to make it easier for the library creators to package only the public part of their offering.

@DanielKamkha DanielKamkha added c++ enhancement New feature or request labels Jan 20, 2020
@DanielKamkha
Copy link
Contributor

"Separating at the file system level" probably means placing the generated files into different directories. For example, placing "internal" types into an "internal" sub-directory.

"Separating at namespace level" sounds like something that should be done by the IDL file author, not by Gluecodium C++ generator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant