Skip to content

Conversation

robbielyman
Copy link
Collaborator

This patch removes the use of std.meta.Tuple for compile-time metaprogramming. It's not clear to me that the convenience it brings makes the use of std.meta worth it. In particular, parsing the arguments list twice (once for TypeOfWrap and once in wrap) seems like an obvious area for improvement.

@robbielyman
Copy link
Collaborator Author

@SuSonicTH Will you take a look? I suspect this might help your @evalBranchQuota problem.

@SuSonicTH
Copy link

The patch worked for me, now my project compiles without "evaluation exceeded 10000 backwards branches" error.

@robbielyman
Copy link
Collaborator Author

@SuSonicTH Really glad to hear it! I'll try and fix up the CI such that this PR passes and give it a merge :)

This patch removes the use of `std.meta.Tuple` for compile-time
metaprogramming. It's not clear to me that the convenience it brings
makes the use of `std.meta` worth it. In particular, parsing the
arguments list twice (once for `TypeOfWrap` and once in `wrap`) seems
like an obvious area for improvement.
@robbielyman robbielyman merged commit 31d1abd into main Sep 8, 2025
12 checks passed
@robbielyman robbielyman deleted the push-pqvzpxwmlnuz branch September 8, 2025 22:17
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