Skip to content

Support WaitHandleWait events#1958

Merged
brianrob merged 4 commits into
microsoft:mainfrom
verdie-g:wait-handle
Dec 8, 2023
Merged

Support WaitHandleWait events#1958
brianrob merged 4 commits into
microsoft:mainfrom
verdie-g:wait-handle

Conversation

@verdie-g

@verdie-g verdie-g commented Dec 7, 2023

Copy link
Copy Markdown
Contributor

New event introduced in dotnet/runtime#94737

@verdie-g verdie-g changed the title Add WaitHandleWaitStart to AnyStacks Support WaitHandleWait events Dec 8, 2023
@verdie-g
verdie-g marked this pull request as ready for review December 8, 2023 16:25

@brianrob brianrob left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this contribution. Overall, looks good - one change noted below.

Comment thread src/TraceEvent/Parsers/ClrTraceEventParser.cs Outdated
@brianrob

brianrob commented Dec 8, 2023

Copy link
Copy Markdown
Member

@verdie-g, can I ask you to fix the merge conflict, since I just merged your other PR. :)

@verdie-g

verdie-g commented Dec 8, 2023

Copy link
Copy Markdown
Contributor Author

@brianrob done!

@brianrob
brianrob merged commit dbf41bd into microsoft:main Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants