-
Notifications
You must be signed in to change notification settings - Fork 62
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
Consider to split AppData::m_appData
into platform and renderer depended
#63
Comments
ChemistAion
added a commit
to ChemistAion/im3d
that referenced
this issue
Jun 24, 2023
…r depended (m_rendererData introduced) Premises: https://github.com/ocornut/imgui/blob/6b2e03c5b1bc4f99dbd86d10e5fd12af9e3fe1c2/imgui.h#L2005-L2009 This pull-req addresses: john-chapman#63
ChemistAion
added a commit
to ChemistAion/im3d
that referenced
this issue
Jun 24, 2023
…r depended (m_rendererData <-- new one) Premises: https://github.com/ocornut/imgui/blob/6b2e03c5b1bc4f99dbd86d10e5fd12af9e3fe1c2/imgui.h#L2005-L2009 This pull-req addresses: john-chapman#63
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Such as: https://github.com/ocornut/imgui/blob/6b2e03c5b1bc4f99dbd86d10e5fd12af9e3fe1c2/imgui.h#L2005-L2009
Premise for that:
im3d_impl_win32
internals goes to (as-is):AppData::m_appData
im3d_impl_dx12
goes to:AppData::m_rendererData
<-- TODONow, we have a nice
AppData
for different renderers.... 😇The text was updated successfully, but these errors were encountered: