Skip to content
This repository was archived by the owner on Apr 9, 2024. It is now read-only.

feat: Derive Copy for Language#406

Merged
TomAFrench merged 1 commit intomasterfrom
phated/copy-language
Jul 3, 2023
Merged

feat: Derive Copy for Language#406
TomAFrench merged 1 commit intomasterfrom
phated/copy-language

Conversation

@phated
Copy link
Copy Markdown
Contributor

@phated phated commented Jun 30, 2023

Description

Problem*

Resolves

Summary*

This adds Copy to the languages. There's no reason we should need to pass references for Language because it can derive copy.

Additional Context

This PR is applied to the 0.15.1 release so I can us it against noir but it should apply cleanly.

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

BEGIN_COMMIT_OVERRIDE
feat(acvm): Derive Copy for Language (#406)
END_COMMIT_OVERRIDE

@phated
Copy link
Copy Markdown
Contributor Author

phated commented Jun 30, 2023

I just used .clone() in my Noir PR, so I'll update this to latest.

@phated phated force-pushed the phated/copy-language branch from 737aaf9 to b70397f Compare June 30, 2023 22:14
@TomAFrench TomAFrench added this pull request to the merge queue Jul 3, 2023
@TomAFrench TomAFrench removed this pull request from the merge queue due to a manual request Jul 3, 2023
@TomAFrench TomAFrench changed the title feat: Derive Copy for Language feat: Derive Copy for Language Jul 3, 2023
@TomAFrench TomAFrench added this pull request to the merge queue Jul 3, 2023
Merged via the queue into master with commit 69a6c22 Jul 3, 2023
@kevaundray kevaundray mentioned this pull request Jul 3, 2023
@phated phated deleted the phated/copy-language branch July 5, 2023 15:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants