Skip to content

Conversation

@jojo59516
Copy link
Contributor

I commented related codes and the bug here.

@jojo59516
Copy link
Contributor Author

Oops, PR's title does not support link of commit.
Related commit is 43442d0. It is a fixture for issue #4.

@pschraut
Copy link
Owner

Thank you for the PR, excellent fix! I'm a bit short on time right now. I will review and merge your changes at the weekend.

@jojo59516
Copy link
Contributor Author

Sure.
I'm developing a reference tree view based on this UnityHeapExplorer, for my own usage. So there would be more PRs and issues. Hope you don't mind :)
Thank you for this excellent tool again!

@pschraut
Copy link
Owner

Sure, no problem. Keep it coming!

@pschraut pschraut merged commit 7ba9d01 into pschraut:master Apr 17, 2021
@pschraut
Copy link
Owner

Thanks again for your contribution! I believe the search is also faster than the version I implemented earlier. So overall a really nice improvement!

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