Review usage of setUncollectable/unsetUncollectable

mozilla / pluotsorbet

[ARCHIVED] PluotSorbet is a J2ME-compatible virtual machine written in JavaScript.

GNU General Public License v2.0

237 stars 46 forks source link

Review usage of setUncollectable/unsetUncollectable #1840

Open marco-c opened 9 years ago

marco-c commented 9 years ago

There might be cases where it's unneeded, for example https://github.com/mozilla/pluotsorbet/pull/1828/files#r37585373.

If we keep the current behavior of not collecting while a native is in execution, we could remove most calls to setUncollectable/unsetUncollectable from natives.