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 #20846; fixes #21427; warn on overloaded =copy with refc [backport] #21606

Open
wants to merge 3 commits into
base: devel
Choose a base branch
from

Conversation

ringabout
Copy link
Member

@ringabout ringabout commented Apr 2, 2023

fixes #20846
fixes #21427 (enabling --warningAsError:CopyHookForRefc for projects)

@ringabout ringabout changed the title fixes #20846; warn on overloaded =copy with refc fixes #20846; warn on overloaded =copy with refc [backport] Apr 2, 2023
compiler/semstmts.nim Outdated Show resolved Hide resolved
@ringabout ringabout changed the title fixes #20846; warn on overloaded =copy with refc [backport] fixes #20846; fixes #21427; warn on overloaded =copy with refc [backport] Apr 3, 2023
Copy link
Contributor

This pull request is stale because it has been open for 1 year with no activity. Contribute more commits on the pull request and rebase it on the latest devel, or it will be closed in 30 days. Thank you for your contributions.

@github-actions github-actions bot added the stale Staled PR/issues; remove the label after fixing them label Apr 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Staled PR/issues; remove the label after fixing them
Projects
None yet
Development

Successfully merging this pull request may close these issues.

warn on overloaded = with refc Error: '=copy' is not available in sequtils.delete move-only types in refc
2 participants