Add error_code, error_category support to visualizer#3204
Add error_code, error_category support to visualizer#3204StephanTLavavej merged 1 commit intomicrosoft:mainfrom
Conversation
|
@microsoft-github-policy-service agree |
StephanTLavavej
left a comment
There was a problem hiding this comment.
This looks reasonable to me (although I have not tested it in the debugger). Thanks!
|
taking a look tomorrow (12-01) |
strega-nil-ms
left a comment
There was a problem hiding this comment.
So, this is how error_code looks:
{ value=0, category=[???] }
and if you click the little circle-y thing, you get
{ value=0, category=[system] }
For error_condition, instead you get:
{_Myval=2 _Mycat=0x00007ff60b389008 {OptionalDebuggerIssues.exe!std::_Generic_error_category _Static} [generic] }
which seems like the name() call doesn't work; what's going on here? am I missing something?
|
I misunderstood! I misread |
|
I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed. |
|
Thanks for implementing these visualizers, and congratulations on your first microsoft/STL commit! 🎉 😻 🚀 We've merged this to the GitHub and MSVC-internal repos, but we still need to "triple-mirror" our accumulated visualizer changes to the VS repo so they will actually take effect in the IDE. I believe that @CaseyCarter will be looking into doing that soon. |


This PR aims to implement the requested fixes outlined in https://github.com/microsoft/STL/pull/3044/files
In general, I prefer the approach of requiring function evaluation as it helps keep the visualizer code layout-independent.
Let me know if there's anything to be changed!