Skip to content

Conversation

@thetic
Copy link
Contributor

@thetic thetic commented Dec 28, 2024

@thetic thetic marked this pull request as draft December 28, 2024 03:04
@thetic
Copy link
Contributor Author

thetic commented Dec 29, 2024

These changes are confusing lcov. I assume this changes the type of cast that is being made.

@basvodde is there a reason that C++-style casts aren't used anywhere? I think the explicit control would be rather useful, but I presume you had a reason to avoid them.

@basvodde
Copy link
Member

Yes, it is the same reason as not using templates and other 'newer' or 'advanced' C++ features. We want CppUTest to be able to work with old compilers because embedded systems often have old compilers or bad C++ support.

@thetic thetic marked this pull request as ready for review January 20, 2025 05:30
@thetic
Copy link
Contributor Author

thetic commented Jan 20, 2025

Coverage build has rotted, but this repairs the tidy job.

@thetic thetic closed this Jan 20, 2025
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