mobx 6.13.5 → 6.13.7

package/dist/mobx.umd.development.js

@@ -4684,8 +4684,8 @@
           if (!change) {
             return this;
           }
-          // ideally, value = change.value would be done here, so that values can be
-          // changed by interceptor. Same applies for other Set and Map api's.
+          // implemented reassignment same as it's done for ObservableMap
+          value = change.newValue;
         }
         if (!this.has(value)) {
           transaction(function () {
@@ -4757,19 +4757,18 @@
         return this.data_.has(this.dehanceValue_(value));
       };
       _proto.entries = function entries() {
-        var nextIndex = 0;
-        var keys = Array.from(this.keys());
-        var values = Array.from(this.values());
+        var values = this.values();
         return makeIterableForSet({
           next: function next() {
-            var index = nextIndex;
-            nextIndex += 1;
-            return index < values.length ? {
-              value: [keys[index], values[index]],
-              done: false
+            var _values$next = values.next(),
+              value = _values$next.value,
+              done = _values$next.done;
+            return !done ? {
+              value: [value, value],
+              done: done
             } : {
               value: undefined,
-              done: true
+              done: done
             };
           }
         });
@@ -4780,16 +4779,18 @@
       _proto.values = function values() {
         this.atom_.reportObserved();
         var self = this;
-        var nextIndex = 0;
-        var observableValues = Array.from(this.data_.values());
+        var values = this.data_.values();
         return makeIterableForSet({
           next: function next() {
-            return nextIndex < observableValues.length ? {
-              value: self.dehanceValue_(observableValues[nextIndex++]),
-              done: false
+            var _values$next2 = values.next(),
+              value = _values$next2.value,
+              done = _values$next2.done;
+            return !done ? {
+              value: self.dehanceValue_(value),
+              done: done
             } : {
               value: undefined,
-              done: true
+              done: done
             };
           }
         });
@@ -5739,6 +5740,8 @@
       return eq(a, b, depth);
     }
     // Copied from https://github.com/jashkenas/underscore/blob/5c237a7c682fb68fd5378203f0bf22dce1624854/underscore.js#L1186-L1289
+    // Modified: "Deep compare objects" part to iterate over keys in forward order instead of reverse order.
+    //
     // Internal recursive comparison function for `isEqual`.
     function eq(a, b, depth, aStack, bStack) {
       // Identical objects are equal. `0 === -0`, but they aren't identical.
@@ -5851,15 +5854,14 @@
       } else {
         // Deep compare objects.
         var keys = Object.keys(a);
-        var key;
-        length = keys.length;
+        var _length = keys.length;
         // Ensure that both objects contain the same number of properties before comparing deep equality.
-        if (Object.keys(b).length !== length) {
+        if (Object.keys(b).length !== _length) {
           return false;
         }
-        while (length--) {
+        for (var i = 0; i < _length; i++) {
           // Deep compare each member
-          key = keys[length];
+          var key = keys[i];
           if (!(hasProp(b, key) && eq(a[key], b[key], depth - 1, aStack, bStack))) {
             return false;
           }