Skip to content

C: Add compile_flags.txt#772

Merged
marcoroth merged 1 commit into
marcoroth:mainfrom
timkaechele:compile-flags
Nov 8, 2025
Merged

C: Add compile_flags.txt#772
marcoroth merged 1 commit into
marcoroth:mainfrom
timkaechele:compile-flags

Conversation

@timkaechele

Copy link
Copy Markdown
Contributor

This PR adds a compile flags configuration [ref] for clangd.

This way C files using prism are actually correctly resolved.

@timkaechele timkaechele force-pushed the compile-flags branch 5 times, most recently from 19bad7f to 280af1d Compare November 5, 2025 08:39
@marcoroth

marcoroth commented Nov 7, 2025

Copy link
Copy Markdown
Owner

Does it make sense to look at bear so we can automatically generate the files/flags needed?

@marcoroth marcoroth added feature New feature or request c labels Nov 7, 2025
@timkaechele

Copy link
Copy Markdown
Contributor Author

I am not sure what that would solve. The compile flags file is super basic, we could generate it using the makefile. Only dynamic part seems to be the check include path.

@marcoroth

marcoroth commented Nov 7, 2025

Copy link
Copy Markdown
Owner

Mostly so that we don't have to maintain these flags in multiple places. I'm just worried that we add this file now and then it gets out of date.

@marcoroth marcoroth merged commit 33382e8 into marcoroth:main Nov 8, 2025
17 checks passed
@marcoroth marcoroth changed the title C: Add compile flags C: Add compile_flags.txt Nov 8, 2025
@timkaechele timkaechele deleted the compile-flags branch November 8, 2025 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants