Skip to content
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

[TIMOB-11671] Clear out unrequired variables at end of run #5670

Merged
merged 1 commit into from May 12, 2014
Merged

[TIMOB-11671] Clear out unrequired variables at end of run #5670

merged 1 commit into from May 12, 2014

Conversation

vishalduggal
Copy link
Contributor

This PR does a few things.

  1. removes the static client objects. Now each instance of TiHttpClient has its own client.
  2. nulls out request,response,handler,client and clientThread objects before firing dispose handle event.

I could not reproduce the memory leak as reported in the ticket, though I did see the memory grow which is probably due to XML processing. This is more of a clean up PR to ensure there are no clients in memory when instances of TiHttpClient are GCed.

JIRA TICKET

@ayeung
Copy link
Contributor

ayeung commented May 12, 2014

Code reviewed. Verified that this did not break the test case in TIMOB-11671. I was never able to reproduce the original test case, so I could not verify that this fix resolved the issue. Request Accepted.

ayeung pushed a commit that referenced this pull request May 12, 2014
[TIMOB-11671] Clear out unrequired variables at end of run
@ayeung ayeung merged commit 10cf938 into tidev:master May 12, 2014
farfromrefug pushed a commit to Akylas/titanium_mobile that referenced this pull request Aug 15, 2014
[TIMOB-11671] Clear out unrequired variables at end of run
@vishalduggal vishalduggal deleted the timob-11671 branch October 27, 2014 19:10
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.

None yet

2 participants