Skip to content

Conversation

julienrull
Copy link
Contributor

I encountered certain errors related to unavailable C syntaxes in MSVC, as well as issues with cimgui API when attempting to set clipboard functions. I'm unsure if I overlooked something, but I've come up with a potential solution.

COMMAND ( MSVC )
cl /nologo .\main.c rlcimgui.c /Fe.\app.exe /I.\include /link /LIBPATH:.\lib msvcrt.lib raylib.lib cimgui.lib openGL32.lib gdi32.lib winmm.lib kernel32.lib shell32.lib user32.lib /NODEFAULTLIB:msvcrtd /NODEFAULTLIB:libcmt

OUTPUTS

main.c
C:\Users\...\Git\app\app_raylib\hot.h(48): warning C4312: 'type cast': conversion from 'BOOL' to 'void *' of greater size
rlcimgui.c
rlcimgui.c(61): warning C4047: '=': 'ImTextureID' differs in levels of indirection from 'Texture2D *'
rlcimgui.c(301): error C2055: expected formal parameter list, not a type list
rlcimgui.c(306): error C2055: expected formal parameter list, not a type list
rlcimgui.c(475): error C2039: 'SetClipboardTextFn': is not a member of 'ImGuiIO'
C:\Users\...\Git\app\app_raylib\include\cimgui.h(1032): note: see declaration of 'ImGuiIO'
rlcimgui.c(476): error C2039: 'GetClipboardTextFn': is not a member of 'ImGuiIO'
C:\Users\...\Git\app\app_raylib\include\cimgui.h(1032): note: see declaration of 'ImGuiIO'
rlcimgui.c(478): error C2039: 'ClipboardUserData': is not a member of 'ImGuiIO'
C:\Users\...\Git\app\app_raylib\include\cimgui.h(1032): note: see declaration of 'ImGuiIO'
rlcimgui.c(632): warning C4022: 'ImGuiRenderTriangles': pointer mismatch for actual parameter 5

@alfredbaudisch
Copy link
Owner

alfredbaudisch commented Jan 27, 2025

Thanks! I will try to review and merge soon, please ping me back here if I don't, as I am in the middle of moving out :)

Copy link

@dmsun dmsun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, just wanted to add I've tested these changes - and seems to address the issues currently! Little bump to remind you to merge when available to do so!

Cheers

@alfredbaudisch alfredbaudisch merged commit 033c91e into alfredbaudisch:master Jul 27, 2025
@alfredbaudisch
Copy link
Owner

Thanks! (and sorry for the delay, it was a long trip, and I will move out again soon).

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.

3 participants