https://codereview.chromium.org/14779011/diff/6001/src/object-observe.js
File src/object-observe.js (right):
https://codereview.chromium.org/14779011/diff/6001/src/object-observe.js#newcode246
src/object-observe.js:246:
ensureObserverRemoved(objectInfo.inactiveObservers, callback);
This pair of calls happens twice. Seems like it should just be a single
function that takes an objectInfo and removed the observer from
whichever list it's in (that'd also be a very minor optimization, as you
wouldn't need to walk through the inactiveObservers if it was found in
changeObservers).
https://codereview.chromium.org/14779011/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
