Skip to content

compilersupport: fix build with GCC < 11#322

Merged
thiagomacieira merged 1 commit intointel:mainfrom
florolf:fix-old-gcc
Apr 13, 2026
Merged

compilersupport: fix build with GCC < 11#322
thiagomacieira merged 1 commit intointel:mainfrom
florolf:fix-old-gcc

Conversation

@florolf
Copy link
Copy Markdown
Contributor

@florolf florolf commented Apr 13, 2026

Commit 45e4641 ("Fix build with GCC < 11: [[fallthrough]] is supported but not allowed in C") introduced a fix for old GCC versions, but commit 91d1c50 ("compilersupport: fix compilation in C23 mode") reverted it, reintroducing the build failure.

Revert the revert.

This builds fine with both an old toolchain and a new one when I force C23 (despite the top-level CMakeFile normally forcing it to C99). I'm not sure if this was an intentional change or just an accident?

Commit 45e4641 ("Fix build with GCC < 11: [[fallthrough]] is supported
but not allowed in C") introduced a fix for old GCC versions, but commit
91d1c50 ("compilersupport: fix compilation in C23 mode") reverted it,
reintroducing the build failure.

Revert the revert.

Signed-off-by: Florian Larysch <fl@n621.de>
@thiagomacieira thiagomacieira enabled auto-merge (rebase) April 13, 2026 22:54
@thiagomacieira
Copy link
Copy Markdown
Member

I'm not sure if this was an intentional change or just an accident?

Pretty sure it was an accident. There was a point in time when my copy on disk, shared with Qt, went out of sync with upstream TinyCBOR (both dev and main branches). That's also the reason I decided to finally bite the bullet and release (0.)7.0.

@thiagomacieira thiagomacieira merged commit 76d7a29 into intel:main Apr 13, 2026
9 checks passed
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