github/meteor
1
1
Fork 0
mirror of https://github.com/meteor/meteor.git synced 2026-05-02 03:01:46 -04:00
Code Issues Packages Projects Releases Wiki Activity

perf(ejson): fast-path primitive comparisons in EJSON.equals

Browse Source
This commit is contained in:
Michael Vogt
2026-03-08 21:55:48 -05:00
parent fc30a303f4
commit 18164ec0ae
1 changed files with 12 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
packages/ejson/ejson.js
View File

@@ -447,18 +447,21 @@ EJSON.equals = (a, b, options) => {
return true;
}
// This differs from the IEEE spec for NaN equality, b/c we don't want
// anything ever with a NaN to be poisoned from becoming equal to anything.
if (Number.isNaN(a) && Number.isNaN(b)) {
return true;
}
// if either one is falsy, they'd have to be === to be equal
if (!a || !b) {
// If types differ, they can't be equal.
// This also handles mixed null/primitive cases since typeof null is 'object'.
if (typeof a !== typeof b) {
return false;
}
if (!(isObject(a) && isObject(b))) {
// Same-type primitives that aren't === can only be equal if both are NaN.
// This skips the NaN check entirely for strings, booleans, etc.
if (typeof a !== 'object') {
return Number.isNaN(a) && Number.isNaN(b);
}
// Both are typeof 'object' — but either could be null.
// (If both were null, a === b would have caught it above.)
if (a === null || b === null) {
return false;
}