@dcl/ecs 7.27.1-34478989812.commit-0dce4d2 → 7.27.1-34480514651.commit-2778e4b

package/dist/engine/input.js

@@ -57,22 +57,22 @@ export function createInputSystem(engine) {
             const arrayCommands = Array.from(commands);
             for (let i = arrayCommands.length - 1; i >= 0; i--) {
                 const command = arrayCommands[i];
+                // Commands are kept sorted by timestamp, so this one and every one
+                // below it belongs to a frame that was already accounted for. Stopping
+                // on the button state instead would end this entity's whole loop over
+                // a button another entity had already reported this frame.
+                if (command.timestamp <= globalState.previousFrameMaxTimestamp) {
+                    break;
+                }
                 if (command.timestamp > maxTimestamp) {
                     maxTimestamp = command.timestamp;
                 }
-                if (command.timestamp > globalState.previousFrameMaxTimestamp) {
-                    globalState.thisFrameCommands.push(command);
-                }
+                globalState.thisFrameCommands.push(command);
                 if (command.state === 0 /* PointerEventType.PET_UP */ || command.state === 1 /* PointerEventType.PET_DOWN */) {
                     const prevCommand = globalState.buttonState.get(command.button);
                     if (!prevCommand || command.timestamp > prevCommand.timestamp) {
                         globalState.buttonState.set(command.button, command);
                     }
-                    else {
-                        // since we are iterating a descending array, we can early finish the
-                        // loop
-                        break;
-                    }
                 }
             }
         }

package/dist-cjs/engine/input.js

@@ -83,22 +83,22 @@ function createInputSystem(engine) {
             const arrayCommands = Array.from(commands);
             for (let i = arrayCommands.length - 1; i >= 0; i--) {
                 const command = arrayCommands[i];
+                // Commands are kept sorted by timestamp, so this one and every one
+                // below it belongs to a frame that was already accounted for. Stopping
+                // on the button state instead would end this entity's whole loop over
+                // a button another entity had already reported this frame.
+                if (command.timestamp <= globalState.previousFrameMaxTimestamp) {
+                    break;
+                }
                 if (command.timestamp > maxTimestamp) {
                     maxTimestamp = command.timestamp;
                 }
-                if (command.timestamp > globalState.previousFrameMaxTimestamp) {
-                    globalState.thisFrameCommands.push(command);
-                }
+                globalState.thisFrameCommands.push(command);
                 if (command.state === 0 /* PointerEventType.PET_UP */ || command.state === 1 /* PointerEventType.PET_DOWN */) {
                     const prevCommand = globalState.buttonState.get(command.button);
                     if (!prevCommand || command.timestamp > prevCommand.timestamp) {
                         globalState.buttonState.set(command.button, command);
                     }
-                    else {
-                        // since we are iterating a descending array, we can early finish the
-                        // loop
-                        break;
-                    }
                 }
             }
         }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@dcl/ecs",
   "description": "Decentraland ECS",
-  "version": "7.27.1-34478989812.commit-0dce4d2",
+  "version": "7.27.1-34480514651.commit-2778e4b",
   "author": "DCL",
   "bugs": "https://github.com/decentraland/ecs/issues",
   "files": [
@@ -34,5 +34,5 @@
   "dependencies": {},
   "types": "./dist/index.d.ts",
   "typings": "./dist/index.d.ts",
-  "commit": "0dce4d2d791418f47f0e3026829d2559e909afbd"
+  "commit": "2778e4bb891f1626a98380a8c906e5e793b74024"
 }