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

Update imgui_stdlib.h to respect IMGUI_DISABLE #8294

Closed
wants to merge 1 commit into from
Closed

Update imgui_stdlib.h to respect IMGUI_DISABLE #8294

wants to merge 1 commit into from

Conversation

juur
Copy link
Contributor

@juur juur commented Jan 6, 2025

Issue: currently if you include imgui_stdlib.h with IMGUI_DISABLE set, it generates lots of errors.
Proposed solution: Instead of having additional conditional code in the user's code, replicate the use of IMGUI_DISABLE into this header, per imgui.h etc.

(For the record I agree to the copyright & contributor license for this and any future contributions to ocornut/imgui)

@ocornut
Copy link
Owner

ocornut commented Jan 6, 2025

Thank you @juur, much appreciated.
I've tweaked this as f169102 as the same block needs to be added to imgui_stdlib.cpp in case it is compiled into the project.

@ocornut ocornut closed this Jan 6, 2025
@juur juur deleted the patch-1 branch January 6, 2025 14:41
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.

2 participants