[DYN-9678] Fix memory leak and prevent process hanging.#16570
[DYN-9678] Fix memory leak and prevent process hanging.#16570reddyashish wants to merge 5 commits intoDynamoDS:masterfrom
Conversation
| { | ||
| try | ||
| { | ||
| Application.Current.Shutdown(); |
There was a problem hiding this comment.
Would this be host application in certain cases instead of sandbox?
There was a problem hiding this comment.
yeah i need to test this in Revit. Also, Made it a draft to run some tests.
There was a problem hiding this comment.
Were you able to test this in Revit?
There was a problem hiding this comment.
Forgot to bring this up at standup, was seeing another issue on Revit with this change, i am looking into it, think i can fix that. Will update soon.
There was a problem hiding this comment.
Is this now validated with Revit?
There was a problem hiding this comment.
yes validated with revit. Can merge and would be nice to have Neal test it as well.
reddyashish
changed the title
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-9678
|
Closing this in favor of #16606, same changes. |
3 participants
Purpose
https://jira.autodesk.com/browse/DYN-9678
Fixing the memory leak and prevent process hanging. Started noticing this after 3.5 release. Will need more testing in host environments.
Declarations
Check these if you believe they are true
Release Notes
Fix memory leak and prevent process hanging.
Reviewers
@DynamoDS/eidos