Skip to content

Fix C23 compile errors#67

Open
JJL772 wants to merge 1 commit intoepics-modules:masterfrom
JJL772:pr-fix-c23-build-errors
Open

Fix C23 compile errors#67
JJL772 wants to merge 1 commit intoepics-modules:masterfrom
JJL772:pr-fix-c23-build-errors

Conversation

@JJL772
Copy link

@JJL772 JJL772 commented Oct 16, 2025

Insert function pointer casts where necessary

Insert function pointer casts where necessary
@simon-ess
Copy link
Contributor

I'm not sure that I love this solution; I think it would be better to move towards typed dset instead of casting the functions.

I think some similar discussion is happening over epics-modules/asyn#222...

@JJL772
Copy link
Author

JJL772 commented Oct 22, 2025

It seems like only certain typed dsets are available for EPICS versions <7.0.4. Is it acceptable to use these here?
We could use the HAS_XXXdset macros to check, however we aren't really gaining anything if we still need the ugly DEVSUPFUN cast

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