Skip to content

rust_verify: improve error reporting on autoderive Clone#2263

Closed
bsdinis wants to merge 1 commit into
mainfrom
bsdinis/owt
Closed

rust_verify: improve error reporting on autoderive Clone#2263
bsdinis wants to merge 1 commit into
mainfrom
bsdinis/owt

Conversation

@bsdinis
Copy link
Copy Markdown
Collaborator

@bsdinis bsdinis commented Mar 23, 2026

The previous error message was quite cryptic by itself.

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@bsdinis bsdinis requested a review from tjhance March 23, 2026 22:02
@bsdinis bsdinis force-pushed the bsdinis/owt branch 2 times, most recently from 90a32fb to 1c18ae2 Compare April 30, 2026 19:46
@tjhance
Copy link
Copy Markdown
Collaborator

tjhance commented Apr 30, 2026

Were you seeing this error message? When I wrote this, I expected the warn_unsupported error to be the one people saw, with the warn_unexpected just kinda there as a backup.

@bsdinis
Copy link
Copy Markdown
Collaborator Author

bsdinis commented Apr 30, 2026

I saw these errors at some point. It's been a while, so I haven't kept track on how to trigger them exactly

@bsdinis bsdinis force-pushed the bsdinis/owt branch 6 times, most recently from 524a41d to 2bf76e4 Compare May 6, 2026 21:36
The previous error message was quite cryptic by itself.
@tjhance
Copy link
Copy Markdown
Collaborator

tjhance commented May 15, 2026

fyi this will be obsolete by #2453 which adds support for the missing cases.

@bsdinis
Copy link
Copy Markdown
Collaborator Author

bsdinis commented May 15, 2026

sounds good!

@bsdinis bsdinis closed this May 15, 2026
@bsdinis bsdinis deleted the bsdinis/owt branch May 15, 2026 21:26
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