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

example for Backtrace::frames #211

Merged
merged 2 commits into from Aug 31, 2017

Conversation

Projects
None yet
2 participants
@AndyGauge
Copy link
Contributor

AndyGauge commented Aug 22, 2017

Fixes #171

@AndyGauge AndyGauge force-pushed the AndyGauge:ex-backtrace branch from 4bf571e to aa7b5b6 Aug 22, 2017

src/lib.rs Outdated
//! if let Some(backtrace) = e.backtrace() {
//! let frames = backtrace.frames();
//! for (_, frame) in frames.iter().enumerate() {
//! for (_, symbol) in frame.symbols().iter().enumerate() {

This comment has been minimized.

@Yamakaky

Yamakaky Aug 23, 2017

Collaborator

Why enumerate?

@AndyGauge AndyGauge force-pushed the AndyGauge:ex-backtrace branch from 4a8f644 to e5d1ce2 Aug 23, 2017

@Yamakaky

This comment has been minimized.

Copy link
Collaborator

Yamakaky commented Aug 31, 2017

Seems good, thanks!

@Yamakaky Yamakaky merged commit bac8d49 into rust-lang-nursery:master Aug 31, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.