clear tryCatchTarget to release reference of function for gc

[chenxm]

Description:

Related issue (if exists):

[rxjs-bot]

	Warnings
⚠️	commit message does not follows conventional change log (1)

(1) : RxJS uses conventional change log to generate changelog automatically. It seems some of commit messages are not following those, please check contributing guideline and update commit messages.

Generated by 🚫 dangerJS

[cartant]

I would prefer to see undefined assigned to it in a finally block. That would be simpler, IMO.

Like this:

function tryCatcher(this: any): any {
  try {
    return tryCatchTarget.apply(this, arguments);
  } catch (e) {
    errorObject.e = e;
    return errorObject;
  } finally {
    tryCatchTarget = undefined;
  }
}

[coveralls]

Pull Request Test Coverage Report for Build 7852

• 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.2%) to 96.806%

---

Totals
Change from base Build 7837:	-0.2%
Covered Lines:	5244
Relevant Lines:	5417

---

💛 - Coveralls

[cartant]

See comment above.

[benlesh]

I agree. @cartant, I made the changes, and also added cleaning of the errorObject.e reference on subsequent tryCatcher calls. Please review.

[cartant]

With the changes made, result is no longer needed.

[cartant]

LGTM