-
Notifications
You must be signed in to change notification settings - Fork 71
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
Add MSVC debugger visualization file #20
Conversation
Hello, Thank you for your contribution. I'm currently in holiday without my laptop for the next few weeks, so I can't really test it. I'll check to merge it on my return so that I can check and test it a bit more beforehand as I'm not familiar with the natvis files. |
Hi Thibaut.
OK, the delay is not a problem.
Have nice holidays! :)
…On 08/15/2018 03:54 PM, Thibaut wrote:
Hello,
Thank you for your contribution.
I'm currently in holiday without my laptop for the next few weeks, so I can't really test it. I'll check to merge it on my return so that I can check and test it a bit more beforehand as I'm not familiar with the natvis files.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#20 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AZcCWuqIqlfK-wPunOr16QY9AlfnzcqIks5uRBnvgaJpZM4V9Hgz>.
|
Hello, I made some minor changes to your branch. If it seems OK to you, I will merge the branch to the master. |
Hi.
Looks good.
…On 09/15/2018 02:36 PM, Thibaut wrote:
Hello,
I made some minor changes to your branch. If it seems OK to you, I will merge the branch to the master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#20 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AZcCWiC2_2zQwuBlg8Vw3uFrUrk1hjkIks5ubOYvgaJpZM4V9Hgz>.
|
I also hesitate to remove the |
That's the behavior of the |
Ok, makes sense. I merged the changes. I'll check to merge the 'natvis' files in the other Thank you. |
This pull request contains a file which can be used by the Visual Studio debugger for "pretty-printing"
tsl::ordered_*
containers.