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

Fixes #148

Merged
merged 9 commits into from
Jun 17, 2024
Merged

Fixes #148

merged 9 commits into from
Jun 17, 2024

Conversation

chriso
Copy link
Contributor

@chriso chriso commented Jun 16, 2024

This PR contains various fixes:

  • underscore imports are now preserved
  • interfaces with embeddeds are now supported
  • ast.Ellipsis is now handled correctly when used in a closure struct
  • ast.IndexListExpr is now supported
  • generic type params are now translated correctly when they're present in an anonymous function return value
  • reflect.Type can now be serialized

Copy link
Contributor

@achille-roussel achille-roussel left a comment

Choose a reason for hiding this comment

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

Do you have a few tests to add that exercise the fixes?

@chriso chriso merged commit 892d764 into main Jun 17, 2024
2 checks passed
@chriso chriso deleted the generics-fixes branch June 17, 2024 04:58
@chriso chriso self-assigned this Jun 24, 2024
chriso added a commit that referenced this pull request Jun 24, 2024
Following on from #148,
this PR contains various fixes:
* we now support generic type parameters that are structs
* we now conservatively register type info for all function decls/lits
in the module, and mark functions/methods in the module that contain
function literals as `noinline`, in case they create a closure that
needs to be serialized
* tweaks the logging so it generates more useful output
@chriso chriso mentioned this pull request Jun 28, 2024
chriso added a commit that referenced this pull request Jun 28, 2024
Following on from #148 and #149, this PR contains various minor fixes:
* slices can now be used with generics
* struct tags are now supported
* lint warnings have been fixed
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.

2 participants