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.

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

reflection doesn't work for fixed length strs #2930

Closed
eholk opened this issue Jul 16, 2012 · 3 comments
Closed

reflection doesn't work for fixed length strs #2930

eholk opened this issue Jul 16, 2012 · 3 comments

Comments

@eholk
Copy link
Contributor

eholk commented Jul 16, 2012

The problem seems to be with the signature of visit_estr_fixed in intrinsic.rs.

See https://gist.github.com/3125194 for a backtrace showing what goes wrong.
@ghost ghost assigned graydon Jul 16, 2012
@graydon
Copy link
Contributor

graydon commented Jul 16, 2012

Can I get the testcase that's triggering this?

eholk added a commit that referenced this issue Jul 16, 2012
@eholk
Test case for #2930
053bdca
@eholk
Copy link
Contributor Author

eholk commented Jul 16, 2012

There is now a test case in https://github.com/mozilla/rust/blob/incoming/src/test/run-pass/issue-2930.rs

@graydon
Copy link
Contributor

graydon commented Jul 17, 2012

Fixed by d809336.

@graydon graydon closed this as completed Jul 17, 2012
RalfJung pushed a commit to RalfJung/rust that referenced this issue Jun 29, 2023
@bors
Auto merge of rust-lang#2930 - RalfJung:rustup, r=RalfJung
c853744 
Rustup
celinval pushed a commit to celinval/rust-dev that referenced this issue Jun 4, 2024
@github-actions
Automatic toolchain upgrade to nightly-2023-12-10 (rust-lang#2930)
00671d1 
Update Rust toolchain from nightly-2023-12-09 to nightly-2023-12-10
without any other source changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@graydon @eholk