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
CommitInfo
: Fix up threading issues
#9205
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
`CommitInfo`: Fix up threading issues (cherry picked from commit d3c38e6)
Cherry-picked to |
if (disposing) | ||
{ | ||
_asyncLoadCancellation.CancelCurrent(); | ||
ThreadHelper.JoinPendingOperations(); // those run with FileAndForget |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be reworked somehow?
This awaits all FileAndForget(), not just those created by this form.
When trying to find a solution for #9440 and add cancellation token that are disposed in REvDiff, RevGrid Dispose(), they are not executed until this has finished. So the tokens are not cancelled.
Fixes #9204
Proposed changes
CommitInfo.UpdateRevisionInfo
That's why it should go into 3.5.1, too.
CommitInfo
AppSettings.ShowAvailableDiffTools = false
Screenshots
N/A
Test methodology
Test environment(s)
✒️ I contribute this code under The Developer Certificate of Origin.