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

remove Clone node and use class lib implementation #943

Merged
merged 1 commit into from
Dec 20, 2021

Conversation

dgrove-oss
Copy link
Collaborator

@dgrove-oss dgrove-oss commented Dec 17, 2021

Fixes #744.
Fixes #674.

@dgrove-oss dgrove-oss added the needs classlib support ⏳ An issue or PR which requires a class libraries release label Dec 17, 2021
@dgrove-oss
Copy link
Collaborator Author

Needs various fixes to Array, Class, and Object to get into a classlib release before we can merge this. Unfortunately, one of our CI test requires a partially-working clone to pass, so we can't break the existing clone support...

@dmlloyd dmlloyd merged commit 2248563 into qbicc:main Dec 20, 2021
@dgrove-oss dgrove-oss deleted the clone-redux branch December 20, 2021 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs classlib support ⏳ An issue or PR which requires a class libraries release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Interpreter clone() should check if receiver implements Cloneable Implement Object.clone()
2 participants