Skip to content

Make autoserialize generate fails with a proper error message #3198

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.

Sign up for GitHub

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

Jump to bottom
Closed
catamorphism opened this issue Aug 15, 2012 · 3 comments
Closed

Make autoserialize generate fails with a proper error message #3198

catamorphism opened this issue Aug 15, 2012 · 3 comments
Labels
A-syntaxext Area: Syntax extensions C-cleanup Category: PRs that clean code up or issues documenting cleanup.

Comments

@catamorphism
Copy link
Contributor

Preparing for the removal of match check, I changed autoserialize to generate matches instead of match checks. Currently, the _ case fails with no message. Even though this is an impossible case, it's probably better to fail with a more informative message.
@ghost ghost assigned catamorphism Aug 15, 2012
@bstrie
Copy link
Contributor

bstrie commented Apr 26, 2013

Is autoserialization the same thing as #[auto_encode]? I see exactly one use of #[auto_serialize] in the codebase, which is a test case.

@emberian
Copy link
Member

@nikomatsakis this needs clarification

@nikomatsakis
Copy link
Contributor

The code in question has been removed and refactored many times over. I think it's safe to ust close this bug.

@catamorphism catamorphism removed their assignment Jun 16, 2014
RalfJung pushed a commit to RalfJung/rust that referenced this issue May 5, 2024
@bors
Auto merge of rust-lang#3198 - devnexen:solarish_support, r=RalfJung
fd8f9a6 
illumos support
celinval pushed a commit to celinval/rust-dev that referenced this issue Jun 4, 2024
@remi-delmas-3000
New section about linter configuraton checking in the doc. (rust-lang…
5dc5e0b 
…#3198)

Resolves rust-lang#3197 

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 and MIT licenses.

Co-authored-by: Remi Delmas <delmasrd@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-syntaxext Area: Syntax extensions C-cleanup Category: PRs that clean code up or issues documenting cleanup.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nikomatsakis @catamorphism @emberian @bstrie