-
Notifications
You must be signed in to change notification settings - Fork 547
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
Node editor on GCC #1
Comments
1, 2, 4 and 5 are on my todo list. Last two implicitly. I will work on that on weekend. As for 3, IMGUI_API ImTextureID ImGui_LoadTexture(const char* path);
IMGUI_API ImTextureID ImGui_CreateTexture(const void* data, int width, int height);
IMGUI_API void ImGui_DestroyTexture(ImTextureID texture);
IMGUI_API int ImGui_GetTextureWidth(ImTextureID texture);
IMGUI_API int ImGui_GetTextureHeight(ImTextureID texture); They can be replaced by stubs. I used them to load icons and background texture for node header. They're used only in example, Node Editor do not touch them. Add these to // Stubs
inline ImTextureID ImGui_LoadTexture(const char* path) { return nullptr; }
inline ImTextureID ImGui_CreateTexture(const void* data, int width, int height) { return nullptr; }
inline void ImGui_DestroyTexture(ImTextureID texture) {}
inline int ImGui_GetTextureWidth(ImTextureID texture) { return 0; }
inline int ImGui_GetTextureHeight(ImTextureID texture) { return 0; } I will work towards portability not only in editor but in example application too. Ask If I managed to miss something. : ) |
TY for your quick reply! |
Here is a patch of all the changes I made to have it run under linux. Most of it is "noise" due to point 1, but there are a couple interesting lines in there as well. I'm posting the whole diff anyway, you should be able to check it out very quickly anyway. |
I did some cleanup to project structure and replaced Visual Studio solution with CMake files. This should make project easier to hack. |
@francesco-cattoglio master branch now builds on Windows, macOS and Linux. |
This is super! I will check it as soon as possible. TY very much for your efforts, I'll keep you posted on any news! |
I finally had some time to start digging in the code, and as soon as I tried to compile it under Linux I noticed a couple things:
Point 1, 2 and 4 are easily fixed, but I would like to have your opinion about point 3. I got to the point where the simple test (application_simple.cpp) works, but got stuck after that. I was wondering if you had a rough idea of how much work it would be to change the main application code to run on glfw/SDL2, before I start banging my head at the wall 😄
The text was updated successfully, but these errors were encountered: