Skip to content

Conversation

@MathiasVP
Copy link
Contributor

Generated from rerunning the bulk builder script now that #19969 has been merged

@github-actions github-actions bot added the C++ label Jul 3, 2025
@MathiasVP MathiasVP marked this pull request as ready for review July 3, 2025 23:05
Copilot AI review requested due to automatic review settings July 3, 2025 23:05
@MathiasVP MathiasVP requested a review from a team as a code owner July 3, 2025 23:05
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@@ -16,10 +16,13 @@
| Dubious signature "(..(*)(..))" in summary model. |
| Dubious signature "(..(*)(..),..(*)(..),..(*)(..),..(*)(..))" in summary model. |
| Dubious signature "(..(*)(..),..(*)(..),..(*)(..),void *)" in summary model. |
| Dubious signature "(..(*)(..),const void *,unsigned int)" in summary model. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should actually start fixing this dubious signature thing, because this file is just getting bigger and bigger without being truly useful.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I agree.

@MathiasVP MathiasVP merged commit 9dd3b33 into github:main Jul 4, 2025
16 checks passed
@RIZOF-svg

This comment was marked as abuse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants