Skip to content

Fix lambda syntax error in test case macros - #217

Merged
cschreib merged 1 commit into
snitch-org:mainfrom
RedSkittleFox:patch-1
Jan 31, 2026
Merged

Fix lambda syntax error in test case macros#217
cschreib merged 1 commit into
snitch-org:mainfrom
RedSkittleFox:patch-1

Conversation

@RedSkittleFox

Copy link
Copy Markdown
Contributor

Fixed invalid lambda syntax from []() <typename T> to proper C++ syntax []<typename T>().

Fixed invalid lambda syntax from `[]() <typename T>` to proper C++ syntax `[]<typename T>()`.
@RedSkittleFox RedSkittleFox changed the title Refactor lambda syntax in test case macros Fix lambda syntax error in test case macros Jan 28, 2026
@codecov

codecov Bot commented Jan 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.97%. Comparing base (67cc5c7) to head (cc38283).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #217   +/-   ##
=======================================
  Coverage   93.97%   93.97%           
=======================================
  Files          30       30           
  Lines        1775     1775           
=======================================
  Hits         1668     1668           
  Misses        107      107           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 67cc5c7...cc38283. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cschreib

Copy link
Copy Markdown
Member

How did that ever work?! Thank you for spotting and fixing this. Can I ask how you happened to notice this was not right? Was it causing an issue with a particular compiler?

@cschreib
cschreib merged commit e865d5d into snitch-org:main Jan 31, 2026
47 checks passed
@RedSkittleFox

Copy link
Copy Markdown
Contributor Author

How did that ever work?! Thank you for spotting and fixing this. Can I ask how you happened to notice this was not right? Was it causing an issue with a particular compiler?

I don't think it has ever worked. My guess is that this macro has never been properly tested. I stumbled across this whilst using GCC 15.2.

@cschreib

cschreib commented Feb 1, 2026

Copy link
Copy Markdown
Member

I don't think it has ever worked. My guess is that this macro has never been properly tested.

Ahah yes, you're right.

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