Mirrors/lodash
1
0
Fork 0
mirror of https://github.com/whoisclebs/lodash.git synced 2026-02-02 08:07:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use forIn in _.isEqual instead of forOwn.

Browse Source 
Former-commit-id: dd057e421be029d67cd293b733ee1cfee2b7715f
This commit is contained in:
John-David Dalton
2012-11-29 20:57:13 -08:00
parent 3a5ed6e800
commit 529c5b8abf
1 changed files with 14 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
lodash.js
View File

@@ -1356,19 +1356,24 @@
}
return result;
}
// deep compare objects
forOwn(a, function(value, key) {
// count the number of properties.
size++;
// deep compare each property value.
return (result = hasOwnProperty.call(b, key) && isEqual(value, b[key], stackA, stackB));
// deep compare objects using `forIn`, instead of `forOwn`, to avoid `Object.keys`
// which, in this case, is more costly
forIn(a, function(value, key, a) {
if (hasOwnProperty.call(a, key)) {
// count the number of properties.
size++;
// deep compare each property value.
return (result = hasOwnProperty.call(b, key) && isEqual(value, b[key], stackA, stackB));
}
});
if (result) {
// ensure both objects have the same number of properties
forOwn(b, function() {
// `size` will be `-1` if `b` has more properties than `a`
return (result = --size > -1);
forIn(b, function(value, key, b) {
if (hasOwnProperty.call(b, key)) {
// `size` will be `-1` if `b` has more properties than `a`
return (result = --size > -1);
}
});
}
return result;