Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#529 still unfixed after merge #718

Closed
paulcdca opened this issue Mar 15, 2024 · 2 comments
Closed

#529 still unfixed after merge #718

paulcdca opened this issue Mar 15, 2024 · 2 comments

Comments

@paulcdca
Copy link

The merge that was meant to fix this issue only moved the problem.
During the call to UnityConcludeTest at line 300 in test_unity_core.c, UNITY_FLUSH_CALL is called.
UNITY_FLUSH_CALL is defined to UNITY_OUTPUT_FLUSH
UNITY_OUTPUT_FLUSH is defined to flushSpy by the makefile at line 22
flushSpy increments flushSpyCalls from 0 to 1

As the count of flushSpyCalls is being captured after the call to UnityConcludeTest, the assert against 0 fails.

@mvandervoord
Copy link
Member

Can you double-check the cutting edge? I believe it's fixed now?

@paulcdca
Copy link
Author

paulcdca commented Apr 1, 2024

Yep, is fixed :)
Needed to disable -Weverything in the Makefile to get it to build, but thats a new issue that I'm sure you found via CI already

@paulcdca paulcdca closed this as completed Apr 1, 2024
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

No branches or pull requests

2 participants