thanks for the review. please land for me if lgty =-)
https://codereview.chromium.org/14779011/diff/13001/src/object-observe.js
File src/object-observe.js (right):
https://codereview.chromium.org/14779011/diff/13001/src/object-observe.js#newcode86
src/object-observe.js:86: function createObserver(callback, accept) {
On 2013/05/13 16:39:17, rossberg wrote:
Nit: can we capitalise functions consistently?
Done.
https://codereview.chromium.org/14779011/diff/13001/src/object-observe.js#newcode124
src/object-observe.js:124: for (; i < from.length; i++, j++) {
On 2013/05/13 16:39:17, rossberg wrote:
I think you can simplify the logic of this loop by not doing j++ here.
Done.
https://codereview.chromium.org/14779011/diff/13001/src/object-observe.js#newcode136
src/object-observe.js:136: function moveObserversWhichAre(conditionFn,
from, to, objectInfo) {
On 2013/05/13 16:39:17, rossberg wrote:
Aren't all observers? :)
Can we perhaps call this RepartitionObservers or something like that?
Done.
https://codereview.chromium.org/14779011/diff/13001/src/object-observe.js#newcode136
src/object-observe.js:136: function moveObserversWhichAre(conditionFn,
from, to, objectInfo) {
On 2013/05/13 16:39:17, rossberg wrote:
Aren't all observers? :)
Can we perhaps call this RepartitionObservers or something like that?
Done.
https://codereview.chromium.org/14779011/diff/13001/src/object-observe.js#newcode152
src/object-observe.js:152: if
(IS_UNDEFINED(objectInfo.performing[type]))
On 2013/05/13 16:39:17, rossberg wrote:
How about writing this as:
objectInfo.performing[type] = (objectInfo.performing[type] || 0) + 1
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.
