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

Add test for actor with mut arg #1956

Merged
merged 1 commit into from
Oct 21, 2024
Merged

Add test for actor with mut arg #1956

merged 1 commit into from
Oct 21, 2024

Conversation

plajjan
Copy link
Contributor

@plajjan plajjan commented Oct 21, 2024

The compiler currently complains about this, stating that mut leaks the actor seal, but I don't think that is right. I don't see how it is leaking...

kll@Kristians-MacBook-Air builtins_auto % acton actor_mutarg.act Building file actor_mutarg.act
Compiling actor_mutarg.act for release

ERROR: Error when compiling actor_mutarg module: Type error

|
2 |actor Foo(cb: mut() -> None):
| ^^^
Leaking actor seal: mut


kll@Kristians-MacBook-Air builtins_auto %

The compiler currently complains about this, stating that mut leaks the
actor seal, but I don't think that is right. I don't see how it is leaking...

kll@Kristians-MacBook-Air builtins_auto % acton actor_mutarg.act
Building file actor_mutarg.act
  Compiling actor_mutarg.act for release

ERROR: Error when compiling actor_mutarg module: Type error

  |
2 |actor Foo(cb: mut() -> None):
  |              ^^^
Leaking actor seal: mut

- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -

kll@Kristians-MacBook-Air builtins_auto %
@plajjan plajjan merged commit 748de2d into main Oct 21, 2024
24 of 27 checks passed
@plajjan plajjan deleted the actor-leaking-seal-mut branch October 21, 2024 07:41
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.

1 participant