Skip to content

debuginfo: Support for std::collections::Hash* in windows debuggers. - #66597

Merged
bors merged 1 commit into
rust-lang:masterfrom
MaulingMonkey:pr-natvis-std-collections-hash
Nov 23, 2019
Merged

bors merged 1 commit into
rust-lang:masterfrom
MaulingMonkey:pr-natvis-std-collections-hash

Conversation

@MaulingMonkey

@MaulingMonkey MaulingMonkey commented Nov 21, 2019 •

Copy link
Copy Markdown
Contributor

Okay, I finally needed to debug code involving a HashMap! Added support for HashSet s as well.

r? @michaelwoerister

Local Testing

Verified these are passing locally:

:: cmd.exe
python x.py test --stage 1 --build x86_64-pc-windows-msvc src/test/debuginfo
python x.py test --stage 1 --build i686-pc-windows-msvc src/test/debuginfo
python x.py test --stage 1 src/tools/tidy

:: MinGW MSYS2
./x.py test --stage 1 --build x86_64-pc-windows-gnu src/test/debuginfo

Related Issues

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 21, 2019
@michaelwoerister

Copy link
Copy Markdown
Member

Thanks, @MaulingMonkey! Approving with the understanding that these natvis files are basically unsupported. cc @rust-lang/compiler & @alexcrichton: someone correct if that is not actually the case.

@bors r+

@bors

bors commented Nov 21, 2019

Copy link
Copy Markdown
Collaborator

📌 Commit 839d58c has been approved by michaelwoerister

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 21, 2019
@bors

bors commented Nov 23, 2019

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 839d58c with merge 9420ff4...

bors added a commit that referenced this pull request Nov 23, 2019
…r=michaelwoerister

debuginfo:  Support for std::collections::Hash* in windows debuggers.

Okay, I finally needed to debug code involving a HashMap!  Added support for HashSet s as well.

r? @michaelwoerister

### Local Testing

Verified these are passing locally:
```cmd
:: cmd.exe
python x.py test --stage 1 --build x86_64-pc-windows-msvc src/test/debuginfo
python x.py test --stage 1 --build i686-pc-windows-msvc src/test/debuginfo
python x.py test --stage 1 src/tools/tidy

:: MinGW MSYS2
./x.py test --stage 1 --build x86_64-pc-windows-gnu src/test/debuginfo
```

### Related Issues

* #36503
* #40460
* rust-gamedev/wg#20
@bors

bors commented Nov 23, 2019

Copy link
Copy Markdown
Collaborator

☀️ Test successful - checks-azure
Approved by: michaelwoerister
Pushing 9420ff4 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Nov 23, 2019
@bors
bors merged commit 839d58c into rust-lang:master Nov 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants