> The debug agent needs to deallocate memory that is allocated by calls to
> GetAllThreads. There were two places were this was not being done, resulting
> in a memory leak.
Chris Plummer has updated the pull request incrementally with one additional
commit since the last revision:
Remove un
On Fri, 30 Oct 2020 20:30:49 GMT, Chris Plummer wrote:
> The debug agent needs to deallocate memory that is allocated by calls to
> GetAllThreads. There were two places were this was not being done, resulting
> in a memory leak.
This looks good.
Nit: semicolon sign can be removed:
err: ;
On Fri, 30 Oct 2020 20:30:49 GMT, Chris Plummer wrote:
> The debug agent needs to deallocate memory that is allocated by calls to
> GetAllThreads. There were two places were this was not being done, resulting
> in a memory leak.
Marked as reviewed by amenkov (Reviewer).
-
PR: htt
The debug agent needs to deallocate memory that is allocated by calls to
GetAllThreads. There were two places were this was not being done, resulting in
a memory leak.
-
Commit messages:
- Make sure to deallocate jthread array allocated by GetAllThreads.
Changes: https://git.openj