Skip to content

debuggers: update MSVC .natvis definitions #172

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

Merged
merged 2 commits into from
Jun 14, 2023
Merged

Conversation

pezcode
Copy link
Contributor

@pezcode pezcode commented Jun 13, 2023

  • Adds definitions for MoveReference and AnyReference
  • Fixes definition for BitArrayView after changes in cbc700a

pezcode added 2 commits June 13, 2023 20:21
cbc700a changed the type of _data to
void* so we need to cast it back first to the underlying type
@codecov
Copy link

codecov bot commented Jun 14, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (b688468) 98.15% compared to head (43d4c0c) 98.15%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #172   +/-   ##
=======================================
  Coverage   98.15%   98.15%           
=======================================
  Files         141      141           
  Lines       11846    11846           
=======================================
  Hits        11627    11627           
  Misses        219      219           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mosra mosra added this to the 2023.0a milestone Jun 14, 2023
@mosra mosra merged commit 43d4c0c into mosra:master Jun 14, 2023
@mosra
Copy link
Owner

mosra commented Jun 14, 2023

Thank you! Looking forward to the Magnum natvis definitions, in case you have time to submit those as well :)

@pezcode
Copy link
Contributor Author

pezcode commented Jun 14, 2023

Thank you! Looking forward to the Magnum natvis definitions, in case you have time to submit those as well :)

Sorry about the delay on that 🙇 Opened an MR with a few basic types: mosra/magnum#620

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

2 participants