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);
On 2013/05/09 16:37:07, adamk wrote:
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).
Done.
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.
