isaacscript-common 6.6.4 → 6.7.1

package/src/features/extraConsoleCommands/commandsSubroutines.ts

@@ -8,7 +8,7 @@ import {
 import { game } from "../../cachedClasses";
 import { HealthType } from "../../enums/HealthType";
 import { directionToVector } from "../../functions/direction";
-import { spawnGridWithVariant } from "../../functions/gridEntity";
+import { spawnGridEntityWithVariant } from "../../functions/gridEntities";
 import { logEntities, logGridEntities } from "../../functions/log";
 import { addPlayerHealthType } from "../../functions/playerHealth";
 import { getRoomData, getRoomDescriptor } from "../../functions/roomData";
@@ -118,7 +118,7 @@ export function spawnTrapdoorOrCrawlSpace(trapdoor: boolean): void {
     ? GridEntityType.TRAPDOOR
     : GridEntityType.CRAWL_SPACE;
 
-  spawnGridWithVariant(gridEntityType, 0, gridIndex);
+  spawnGridEntityWithVariant(gridEntityType, 0, gridIndex);
 }
 
 export function warpToRoomType(roomType: RoomType): void {

package/src/features/extraConsoleCommands/listCommands.ts

@@ -53,10 +53,10 @@ import { getCardName } from "../../functions/cards";
 import { getCharacterName } from "../../functions/character";
 import { addCharge } from "../../functions/charge";
 import { isValidCollectibleType } from "../../functions/collectibles";
-import { getNPCs } from "../../functions/entitySpecific";
+import { getNPCs } from "../../functions/entitiesSpecific";
 import { getEnumValues } from "../../functions/enums";
 import { addFlag } from "../../functions/flag";
-import { spawnGrid } from "../../functions/gridEntity";
+import { spawnGridEntity } from "../../functions/gridEntities";
 import {
   logEffects,
   logRoom,
@@ -64,15 +64,19 @@ import {
   logSounds,
 } from "../../functions/log";
 import { getMapPartialMatch } from "../../functions/map";
-import { spawnCard, spawnPill, spawnTrinket } from "../../functions/pickups";
+import {
+  spawnCard,
+  spawnPill,
+  spawnTrinket,
+} from "../../functions/pickupsSpecific";
 import { getPillEffectName } from "../../functions/pills";
+import { getPlayers } from "../../functions/playerIndex";
 import {
   addCollectibleCostume,
   getPlayerName,
   removeCollectibleCostume,
   useActiveItemTemp,
-} from "../../functions/player";
-import { getPlayers } from "../../functions/playerIndex";
+} from "../../functions/players";
 import { gridCoordinatesToWorldPosition } from "../../functions/roomGrid";
 import { changeRoom, getRoomGridIndexesForType } from "../../functions/rooms";
 import { onSetSeed, restart, setUnseeded } from "../../functions/run";
@@ -337,7 +341,7 @@ export function cards(): void {
 
       const position = gridCoordinatesToWorldPosition(x, y);
       spawnCard(cardType as Card, position);
-      cardType += 1;
+      cardType++;
     }
   }
 }
@@ -647,14 +651,14 @@ export function gridEntities(): void {
   let gridEntityTypeIndex = -1;
   for (let y = 0; y <= 6; y++) {
     for (let x = 0; x <= 12; x++) {
-      gridEntityTypeIndex += 1;
+      gridEntityTypeIndex++;
       const gridEntityType = gridEntityTypes[gridEntityTypeIndex];
       if (gridEntityType === undefined) {
         return;
       }
 
       const position = gridCoordinatesToWorldPosition(x, y);
-      spawnGrid(gridEntityType, position);
+      spawnGridEntity(gridEntityType, position);
     }
   }
 }
@@ -884,7 +888,7 @@ export function pills(): void {
   let pillColorIndex = -1;
   for (let y = 0; y <= 6; y++) {
     for (let x = 0; x <= 12; x++) {
-      pillColorIndex += 1;
+      pillColorIndex++;
       const pillColor = pillColors[pillColorIndex];
       if (pillColor === undefined) {
         return;

package/src/features/pause.ts

@@ -5,7 +5,7 @@ import {
   ModCallback,
 } from "isaac-typescript-definitions";
 import { logError } from "../functions/log";
-import { useActiveItemTemp } from "../functions/player";
+import { useActiveItemTemp } from "../functions/players";
 import { disableAllInputsExceptFor, enableAllInputs } from "./disableInputs";
 import { saveDataManager } from "./saveDataManager/exports";
 

package/src/features/persistentEntities.ts

@@ -7,7 +7,7 @@ import { game } from "../cachedClasses";
 import { ModUpgraded } from "../classes/ModUpgraded";
 import { ModCallbackCustom } from "../enums/ModCallbackCustom";
 import { errorIfFeaturesNotInitialized } from "../featuresInitialized";
-import { spawn } from "../functions/entity";
+import { spawn } from "../functions/entities";
 import { getRoomListIndex } from "../functions/roomData";
 import { saveDataManager } from "./saveDataManager/exports";
 
@@ -123,7 +123,7 @@ export function spawnPersistentEntity(
 ): [Entity, int] {
   errorIfFeaturesNotInitialized(FEATURE_NAME);
 
-  persistentEntityIndexCounter += 1;
+  persistentEntityIndexCounter++;
 
   const entity = spawnAndTrack(
     entityType,

package/src/features/preventCollectibleRotation.ts

@@ -10,7 +10,7 @@ import {
 import { game } from "../cachedClasses";
 import { errorIfFeaturesNotInitialized } from "../featuresInitialized";
 import { setCollectibleSubType } from "../functions/collectibles";
-import { getEntityID } from "../functions/entity";
+import { getEntityID } from "../functions/entities";
 import { isCollectible } from "../functions/pickupVariants";
 import { saveDataManager } from "./saveDataManager/exports";
 

package/src/features/saveDataManager/exports.ts

@@ -169,8 +169,11 @@ export function saveDataManagerSave(): void {
   forceSaveDataManagerSave();
 }
 
-declare let g: LuaTable<string, SaveData>; // Globals
-declare let gd: LuaTable<string, SaveData>; // Globals defaults
+/** "g" stands for "globals". */
+declare let g: LuaTable<string, SaveData>; // eslint-disable-line @typescript-eslint/no-unused-vars
+
+/** "gd" stands for "globals defaults". */
+declare let gd: LuaTable<string, SaveData>; // eslint-disable-line @typescript-eslint/no-unused-vars
 
 /**
  * - Sets the global variable of "g" equal to all of the save data variables for this mod.
@@ -182,8 +185,8 @@ declare let gd: LuaTable<string, SaveData>; // Globals defaults
 export function saveDataManagerSetGlobal(): void {
   errorIfFeaturesNotInitialized(SAVE_DATA_MANAGER_FEATURE_NAME);
 
-  g = saveDataMap; // eslint-disable-line @typescript-eslint/no-unused-vars
-  gd = saveDataDefaultsMap; // eslint-disable-line @typescript-eslint/no-unused-vars
+  g = saveDataMap;
+  gd = saveDataDefaultsMap;
 }
 
 /**

package/src/features/sirenHelpers.ts

@@ -4,7 +4,7 @@ import {
   ModCallback,
 } from "isaac-typescript-definitions";
 import { errorIfFeaturesNotInitialized } from "../featuresInitialized";
-import { getEntities } from "../functions/entity";
+import { getEntities } from "../functions/entities";
 import { saveDataManager } from "./saveDataManager/exports";
 
 const FEATURE_NAME = "sirenHelpers";

package/src/functions/ambush.ts

@@ -1,7 +1,7 @@
 import { SackSubType } from "isaac-typescript-definitions";
 import { runNextGameFrame } from "../features/runInNFrames";
-import { removeEntities } from "./entity";
-import { getCoins, spawnSackWithSeed } from "./pickups";
+import { removeEntities } from "./entities";
+import { getCoins, spawnSackWithSeed } from "./pickupsSpecific";
 
 /** Hard-coding this makes it easier to clean up the pickups afterwards. */
 const SACK_SEED_THAT_SPAWNS_TWO_COINS = 6 as Seed;

package/src/functions/array.ts

@@ -527,7 +527,7 @@ export function shuffleArrayInPlace<T>(
   const rng = isRNG(seedOrRNG) ? seedOrRNG : newRNG(seedOrRNG);
 
   while (currentIndex > 0) {
-    currentIndex -= 1;
+    currentIndex--;
 
     const randomIndex = getRandomArrayIndex(array, rng);
     swapArrayElements(array, currentIndex, randomIndex);

package/src/functions/bitwise.ts

@@ -57,7 +57,7 @@ export function countSetBits(n: int): int {
 
   while (n > 0) {
     n &= n - 1;
-    count += 1;
+    count++;
   }
 
   return count;
@@ -72,7 +72,7 @@ export function getKBitOfN(k: int, n: int): int {
 export function getNumBitsOfN(n: int): int {
   let numBits = 0;
   while (n > 0) {
-    numBits += 1;
+    numBits++;
     n >>>= 1;
   }
 

package/src/functions/{boss.ts → bosses.ts}

@@ -10,8 +10,8 @@ import {
   STAGE_TO_STAGE_TYPE_TO_BOSS_SET_MAP,
 } from "../sets/bossSets";
 import { SIN_ENTITY_TYPES_SET } from "../sets/sinEntityTypesSet";
-import { getNPCs, spawnNPC } from "./entitySpecific";
-import { getAliveNPCs } from "./npc";
+import { getNPCs, spawnNPC } from "./entitiesSpecific";
+import { getAliveNPCs } from "./npcs";
 import { isRNG } from "./rng";
 import { copySet } from "./set";
 import { repeat } from "./utils";
@@ -31,19 +31,22 @@ const DEFAULT_BOSS_MULTI_SEGMENTS = 4;
  *
  * This function will not include bosses on an internal blacklist, such as Death's scythes or Big
  * Horn holes.
+ *
+ * @param entityType Optional. If specified, will only get the bosses that match the type. Default
+ *                   is -1, which matches every type.
+ * @param variant Optional. If specified, will only get the bosses that match the variant. Default
+ *                is -1, which matches every variant.
+ * @param subType Optional. If specified, will only get the bosses that match the sub-type. Default
+ *                is -1, which matches every sub-type.
+ * @param ignoreFriendly Optional. Default is false.
  */
 export function getAliveBosses(
-  matchingEntityType?: EntityType,
-  matchingVariant?: int,
-  matchingSubType?: int,
+  entityType: EntityType = -1,
+  variant = -1,
+  subType = -1,
   ignoreFriendly = false,
 ): EntityNPC[] {
-  const aliveNPCs = getAliveNPCs(
-    matchingEntityType,
-    matchingVariant,
-    matchingSubType,
-    ignoreFriendly,
-  );
+  const aliveNPCs = getAliveNPCs(entityType, variant, subType, ignoreFriendly);
   return aliveNPCs.filter((aliveNPC) => aliveNPC.IsBoss());
 }
 
@@ -83,19 +86,24 @@ export function getBossSet(
   return copySet(bossSet);
 }
 
-/** Helper function to get all of the bosses in the room. */
+/**
+ * Helper function to get all of the bosses in the room.
+ *
+ * @param entityType Optional. If specified, will only get the bosses that match the type. Default
+ *                   is -1, which matches every type.
+ * @param variant Optional. If specified, will only get the bosses that match the variant. Default
+ *                is -1, which matches every variant.
+ * @param subType Optional. If specified, will only get the bosses that match the sub-type. Default
+ *                is -1, which matches every sub-type.
+ * @param ignoreFriendly Optional. Default is false.
+ */
 export function getBosses(
-  matchingEntityType?: EntityType,
-  matchingVariant?: int,
-  matchingSubType?: int,
+  entityType?: EntityType,
+  variant?: int,
+  subType?: int,
   ignoreFriendly = false,
 ): EntityNPC[] {
-  const npcs = getNPCs(
-    matchingEntityType,
-    matchingVariant,
-    matchingSubType,
-    ignoreFriendly,
-  );
+  const npcs = getNPCs(entityType, variant, subType, ignoreFriendly);
   return npcs.filter((npc) => npc.IsBoss());
 }
 

package/src/functions/collectibleTag.ts

@@ -3,7 +3,7 @@ import { itemConfig } from "../cachedClasses";
 import { getCollectibleArray } from "./collectibleSet";
 import { getEnumValues } from "./enums";
 import { getFlagName } from "./flag";
-import { getPlayerCollectibleCount } from "./player";
+import { getPlayerCollectibleCount } from "./players";
 import { copySet } from "./set";
 
 const TAG_TO_COLLECTIBLE_TYPES_MAP = new Map<

package/src/functions/collectibles.ts

@@ -27,10 +27,10 @@ import {
 } from "../maps/collectibleNameMap";
 import { SINGLE_USE_ACTIVE_COLLECTIBLE_TYPES_SET } from "../sets/singleUseActiveCollectibleTypesSet";
 import { CollectibleIndex } from "../types/CollectibleIndex";
-import { getEntityID } from "./entity";
+import { getEntityID } from "./entities";
 import { isCollectible } from "./pickupVariants";
 import { getRoomListIndex } from "./roomData";
-import { clearSprite, spriteEquals } from "./sprite";
+import { clearSprite, spriteEquals } from "./sprites";
 import { irange } from "./utils";
 
 const COLLECTIBLE_SPRITE_LAYER = 1;

package/src/functions/deepCopy.ts

@@ -51,15 +51,19 @@ const COPYABLE_ISAAC_API_CLASS_TYPES_SET = new Set<string>(
  * - other Isaac API objects such as `EntityPtr` (that have a type of "userdata")
  *
  * @param value The primitive or object to copy.
- * @param serializationType Has 3 possible values. Can leave objects as-is, or can serialize objects
- *                          to Lua tables, or can deserialize Lua tables to objects. Default is
- *                          `SerializationType.NONE`.
- * @param traversalDescription Used to track the current key that we are operating on.
+ * @param serializationType Optional. Has 3 possible values. Can copy objects as-is, or can
+ *                          serialize objects to Lua tables, or can deserialize Lua tables to
+ *                          objects. Default is `SerializationType.NONE`.
+ * @param traversalDescription Optional. Used to track the current key that we are operating on.
+ *                             Default is an empty string.
+ * @param insideMap Optional. Tracks whether or not the deep copy function is in the process of
+ *                  recursively copying a TSTL Map. Default is false.
  */
 export function deepCopy(
   value: unknown,
   serializationType = SerializationType.NONE,
   traversalDescription = "",
+  insideMap = false,
 ): unknown {
   // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
   if (SAVE_DATA_MANAGER_DEBUG) {
@@ -98,7 +102,12 @@ export function deepCopy(
 
     case "table": {
       const luaTable = value as LuaTable<AnyNotNil, unknown>;
-      return deepCopyTable(luaTable, serializationType, traversalDescription);
+      return deepCopyTable(
+        luaTable,
+        serializationType,
+        traversalDescription,
+        insideMap,
+      );
     }
 
     case "userdata": {
@@ -111,6 +120,7 @@ function deepCopyTable(
   luaTable: LuaTable<AnyNotNil, unknown>,
   serializationType: SerializationType,
   traversalDescription: string,
+  insideMap: boolean,
 ) {
   // First, handle the cases of TSTL classes or serialized TSTL classes.
   if (isDefaultMap(luaTable) || luaTable.has(SerializationBrand.DEFAULT_MAP)) {
@@ -118,15 +128,26 @@ function deepCopyTable(
       luaTable,
       serializationType,
       traversalDescription,
+      insideMap,
     );
   }
 
   if (isTSTLMap(luaTable) || luaTable.has(SerializationBrand.MAP)) {
-    return deepCopyMap(luaTable, serializationType, traversalDescription);
+    return deepCopyMap(
+      luaTable,
+      serializationType,
+      traversalDescription,
+      insideMap,
+    );
   }
 
   if (isTSTLSet(luaTable) || luaTable.has(SerializationBrand.SET)) {
-    return deepCopySet(luaTable, serializationType, traversalDescription);
+    return deepCopySet(
+      luaTable,
+      serializationType,
+      traversalDescription,
+      insideMap,
+    );
   }
 
   const className = getTSTLClassName(luaTable);
@@ -144,7 +165,12 @@ function deepCopyTable(
   }
 
   if (isUserDefinedTSTLClass(luaTable)) {
-    return deepCopyTSTLClass(luaTable, serializationType, traversalDescription);
+    return deepCopyTSTLClass(
+      luaTable,
+      serializationType,
+      traversalDescription,
+      insideMap,
+    );
   }
 
   // This is not a TSTL Map/Set/class. If it has a metatable, abort.
@@ -160,7 +186,12 @@ function deepCopyTable(
 
   // Handle the special case of an array.
   if (isArray(luaTable)) {
-    return deepCopyArray(luaTable, serializationType, traversalDescription);
+    return deepCopyArray(
+      luaTable,
+      serializationType,
+      traversalDescription,
+      insideMap,
+    );
   }
 
   // Base case: copy a normal Lua table
@@ -168,6 +199,7 @@ function deepCopyTable(
     luaTable,
     serializationType,
     traversalDescription,
+    insideMap,
   );
 }
 
@@ -175,19 +207,39 @@ function deepCopyDefaultMap(
   defaultMap: DefaultMap<AnyNotNil, unknown> | LuaTable<AnyNotNil, unknown>,
   serializationType: SerializationType,
   traversalDescription: string,
+  insideMap: boolean,
 ) {
-  // First, handle the special case of serializing a DefaultMap instantiated with a factory
-  // function. If this is the case, then we cannot serialize it, so we serialize it as a normal
-  // `Map` instead. We do not throw a runtime error because the merge function does not need to
-  // instantiate the DefaultMap class in most circumstances.
   const constructorArg = isDefaultMap(defaultMap)
     ? defaultMap.getConstructorArg()
-    : undefined;
+    : undefined; // The undefined case is handled explicitly in the "getNewDefaultMap" function.
+
+  // First, handle the special case of serializing a DefaultMap instantiated with a factory
+  // function. If this is the case, then we cannot serialize it (because there is no way to
+  // serialize a function).
   if (
     serializationType === SerializationType.SERIALIZE &&
     !isPrimitive(constructorArg)
   ) {
-    return deepCopyMap(defaultMap, serializationType, traversalDescription);
+    if (insideMap) {
+      // The case of a DefaultMap within another map is complicated. Unlike a DefaultMap attached to
+      // a "normal" object, the `merge` function will have no reference to the factory function that
+      // was used to instantiate it. Thus, there is no way to copy this object. In this case, we
+      // throw a run-time error to immediately alert the end-user that their data structure is
+      // invalid.
+      error(
+        'Failed to deep copy a DefaultMap because it was instantiated with a factory function and was also inside of another map. You cannot use a nested DefaultMap in this way because factory functions are not serializable. (In other words, there is no way to copy the function that you are using for the DefaultMap into the "save#.dat" file.) Instead, refactor your data structure so that the DefaultMap is not nested.',
+      );
+    } else {
+      // In most cases, the DefaultMap will be attached to a normal table element. In this case, if
+      // we serialize it as a normal `Map`, then everything will work out fine, because the `merge`
+      // function only needs to copy the values (and not instantiate the object itself).
+      return deepCopyMap(
+        defaultMap,
+        serializationType,
+        traversalDescription,
+        insideMap,
+      );
+    }
   }
 
   const newDefaultMap = getNewDefaultMap(
@@ -196,11 +248,13 @@ function deepCopyDefaultMap(
     traversalDescription,
     constructorArg,
   );
+  insideMap = true;
 
   const { entries, convertedNumberKeysToStrings } = getCopiedEntries(
     defaultMap,
     serializationType,
     traversalDescription,
+    insideMap,
   );
 
   if (convertedNumberKeysToStrings) {
@@ -223,12 +277,14 @@ function deepCopyDefaultMap(
     }
   }
 
+  insideMap = false;
+
   return newDefaultMap;
 }
 
 /**
- * The new default map with either be a TSTL `DefaultMap` class or a Lua table, depending on whether
- * we are serializing or not.
+ * The new copied default map with either be a TSTL `DefaultMap` class or a Lua table, depending on
+ * whether we are serializing or not.
  */
 function getNewDefaultMap(
   defaultMap: DefaultMap<AnyNotNil, unknown> | LuaTable<AnyNotNil, unknown>,
@@ -278,6 +334,7 @@ function deepCopyMap(
   map: Map<AnyNotNil, unknown> | LuaTable<AnyNotNil, unknown>,
   serializationType: SerializationType,
   traversalDescription: string,
+  insideMap: boolean,
 ) {
   let newMap: Map<AnyNotNil, unknown> | LuaTable<AnyNotNil, unknown>;
   if (serializationType === SerializationType.SERIALIZE) {
@@ -287,11 +344,13 @@ function deepCopyMap(
   } else {
     newMap = new Map();
   }
+  insideMap = true;
 
   const { entries, convertedNumberKeysToStrings } = getCopiedEntries(
     map,
     serializationType,
     traversalDescription,
+    insideMap,
   );
 
   if (convertedNumberKeysToStrings) {
@@ -314,6 +373,8 @@ function deepCopyMap(
     }
   }
 
+  insideMap = false;
+
   return newMap;
 }
 
@@ -321,6 +382,7 @@ function deepCopySet(
   set: Set<AnyNotNil> | LuaTable<AnyNotNil, unknown>,
   serializationType: SerializationType,
   traversalDescription: string,
+  insideMap: boolean,
 ) {
   let newSet: Set<AnyNotNil> | LuaTable<AnyNotNil, string>;
   if (serializationType === SerializationType.SERIALIZE) {
@@ -336,6 +398,7 @@ function deepCopySet(
     set,
     serializationType,
     traversalDescription,
+    insideMap,
   );
 
   if (convertedNumberKeysToStrings) {
@@ -368,6 +431,7 @@ function deepCopyTSTLClass(
   tstlClass: TSTLClass,
   serializationType: SerializationType,
   traversalDescription: string,
+  insideMap: boolean,
 ) {
   let newClass: TSTLClass | LuaTable<AnyNotNil, unknown>;
   if (serializationType === SerializationType.SERIALIZE) {
@@ -383,6 +447,7 @@ function deepCopyTSTLClass(
     tstlClass,
     serializationType,
     traversalDescription,
+    insideMap,
   );
 
   if (convertedNumberKeysToStrings) {
@@ -400,11 +465,17 @@ function deepCopyArray(
   array: unknown[],
   serializationType: SerializationType,
   traversalDescription: string,
+  insideMap: boolean,
 ) {
   const newArray: unknown[] = [];
 
   for (const value of array) {
-    const newValue = deepCopy(value, serializationType, traversalDescription);
+    const newValue = deepCopy(
+      value,
+      serializationType,
+      traversalDescription,
+      insideMap,
+    );
     newArray.push(newValue);
   }
 
@@ -415,12 +486,14 @@ function deepCopyNormalLuaTable(
   luaTable: LuaTable<AnyNotNil, unknown>,
   serializationType: SerializationType,
   traversalDescription: string,
+  insideMap: boolean,
 ) {
   const newTable = new LuaTable<AnyNotNil, unknown>();
   const { entries, convertedNumberKeysToStrings } = getCopiedEntries(
     luaTable,
     serializationType,
     traversalDescription,
+    insideMap,
   );
 
   if (convertedNumberKeysToStrings) {
@@ -442,6 +515,7 @@ function getCopiedEntries(
   object: unknown,
   serializationType: SerializationType,
   traversalDescription: string,
+  insideMap: boolean,
 ): {
   entries: Array<[key: AnyNotNil, value: unknown]>;
   convertedNumberKeysToStrings: boolean;
@@ -479,7 +553,12 @@ function getCopiedEntries(
     }
 
     traversalDescription = getTraversalDescription(key, traversalDescription);
-    const newValue = deepCopy(value, serializationType, traversalDescription);
+    const newValue = deepCopy(
+      value,
+      serializationType,
+      traversalDescription,
+      insideMap,
+    );
 
     const keyToUse = convertNumberKeysToStrings ? tostring(key) : key;
     copiedEntries.push([keyToUse, newValue]);

package/src/functions/deepCopyTests.ts

@@ -131,6 +131,7 @@ function copiedTableDoesNotCoerceTypes() {
   }
 }
 
+/** In this context, a reference is a pointer. */
 function copiedObjectHasNoReferencesForPrimitivesForward() {
   const originalStringValue = "abcdef";
   const originalNumberValue = 123;
@@ -181,6 +182,7 @@ function copiedObjectHasNoReferencesForPrimitivesBackward() {
   }
 }
 
+/** In this context, a reference is a pointer. */
 function copiedObjectHasNoReferencesForArray() {
   const oldObject = {
     abc: [1, 2, 3],
@@ -200,21 +202,21 @@ function copiedObjectHasNoReferencesForArray() {
     error("The copied object does not have an equal array.");
   }
 
-  oldObject.abc[0] += 1;
+  oldObject.abc[0]++;
   if (arrayEquals(oldObject.abc, newObject.abc)) {
     error(
       "The copied object has an equal array after a modification to the old array.",
     );
   }
-  oldObject.abc[0] -= 1;
+  oldObject.abc[0]--;
 
-  newObject.abc[0] += 1;
+  newObject.abc[0]++;
   if (arrayEquals(oldObject.abc, newObject.abc)) {
     error(
       "The copied object has an equal array after a modification to the new array.",
     );
   }
-  newObject.abc[0] -= 1;
+  newObject.abc[0]--;
 }
 
 function copiedObjectHasChildObject() {