Skip to content

Split debuginfo into its own module #20780

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
nrc opened this issue Jan 9, 2015 · 1 comment · Fixed by #24762
Closed

Split debuginfo into its own module #20780

nrc opened this issue Jan 9, 2015 · 1 comment · Fixed by #24762
Assignees
@michaelwoerister
Labels
A-debuginfo Area: Debugging information in compiled programs (DWARF, PDB, etc.)

Comments

@nrc
Copy link
Member

nrc commented Jan 9, 2015

trans/debuginfo.rs is huge. It would be good to split it out into a module and split it up into a few files.

cc @michaelwoerister
@nrc nrc added the A-debuginfo Area: Debugging information in compiled programs (DWARF, PDB, etc.) label Jan 9, 2015
@michaelwoerister
Copy link
Member

I agree. The module has grown quite considerably over the last year and it contains some loosely coupled areas of functionality that can easily be put into modules of their own.

@michaelwoerister michaelwoerister self-assigned this Jan 9, 2015
@nrc nrc mentioned this issue Apr 24, 2015
Merged
alexcrichton added a commit to alexcrichton/rust that referenced this issue Apr 29, 2015
@alexcrichton
rollup merge of rust-lang#24762: nrc/mod-debug-2
3434469 
Closes rust-lang#20780

r? @michaelwoerister

I'm sure this could be done better with deeper knowledge of debuginfo, but this seems like a good start.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michaelwoerister michaelwoerister

Labels
A-debuginfo Area: Debugging information in compiled programs (DWARF, PDB, etc.)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make a module out of debuginfo nrc/rust
2 participants
@nrc @michaelwoerister