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

ICE 'index out of bounds' in extra::ebml #8882

Closed
SimonSapin opened this issue Aug 30, 2013 · 2 comments
Closed

ICE 'index out of bounds' in extra::ebml #8882

SimonSapin opened this issue Aug 30, 2013 · 2 comments

Comments

@SimonSapin
Copy link
Contributor

Upgrading Servo to today’s Rust gives (after a few fixes) an ICE:

% RUST_LOG=rustc=1 make -j8
compile: servo
task <unnamed> failed at 'index out of bounds: the len is 52757 but the index is 52757', /home/simon/projects/servo/src/compiler/rust/src/libextra/ebml.rs:119
error: internal compiler error: unexpected failure
note: the compiler hit an unexpected failure path. this is a bug
note: try running with RUST_LOG=rustc=1 to get further details and report the results to github.com/mozilla/rust/issues
task <unnamed> failed at 'explicit failure', /home/simon/projects/servo/src/compiler/rust/src/librustc/rustc.rs:376
make: *** [servo] Erreur 101

Is there more information that I can provide to help with this?
@pcwalton
Copy link
Contributor

I can look into this today. Do you have instructions for reproducing?

@SimonSapin
Copy link
Contributor Author

Not really. I was in the middle of upgrading Rust in Servo, and make clean (rather than just make clean-fast) fixed this issue for me. Closing for now.

@klutzy klutzy mentioned this issue Dec 7, 2013
Closed
flip1995 pushed a commit to flip1995/rust that referenced this issue Jun 4, 2022
@bors
Auto merge of rust-lang#8882 - kyoto7250:get_first, r=llogiq
3cc50a4 
feat(lint): impl lint about use first() instead of get(0)

close rust-lang#8851

This PR adds new lint about considering replacing .get(0) with .first().

Thank you in advance.

changelog: adds new lint [`get_first`] to consider replacing .get(0) with .first()
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
@pcwalton @SimonSapin