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.

Sign up for GitHub

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

Jump to bottom

consider using IMGUI_API #26

Open
onqtam opened this issue Aug 17, 2017 · 2 comments
Open

consider using IMGUI_API #26

onqtam opened this issue Aug 17, 2017 · 2 comments
Labels
enhancement

Comments

@onqtam
Copy link

onqtam commented Aug 17, 2017 •
edited

I use ImGui as a dll by setting IMGUI_API to either export or import symbols appropriately - currently the addons are unusable for me because the interface (forward declarations of functions in headers) is not annotated with IMGUI_API accordingly.
@Flix01
Copy link
Owner

Flix01 commented Aug 18, 2017

Truth. I'll see if I manage to set them in the future (even if probably 95% of the users don't use ImGui as DLL).

Flix01 added a commit that referenced this issue Aug 19, 2017
@Flix01
Added IMGUI_API to many more methods (I don't think this is enough to…
b7d8025 
… compile it as DLL on Windows, but this is a start). Please see #26.
@Flix01
Copy link
Owner

Flix01 commented Aug 19, 2017

I've added IMGUI_API to many more methods. However I don't think this is enough to use it as DLL on Windows.

Unluckily, I don't have a test-case that I can check to see what functions are missing...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@onqtam @Flix01