atem-connection - npm Package Compare versions

Comparing version 1.3.1 to 1.3.2

CHANGELOG.md

@@ -5,2 +5,10 @@ # Changelog
 
+### [1.3.2](https://github.com/nrkno/tv-automation-atem-connection/compare/1.3.1...1.3.2) (2020-06-25)
+
+
+### Bug Fixes
+
+* listVisibleInputs with multiple levels of nesting ([6c1c848](https://github.com/nrkno/tv-automation-atem-connection/commit/6c1c8480a261ec7029906b31af8df95f91ff6e29))
+* listVisibleInputs with multiple levels of nesting ([6c1c848](https://github.com/nrkno/tv-automation-atem-connection/commit/6c1c8480a261ec7029906b31af8df95f91ff6e29))
+
 ### [1.3.1](https://github.com/nrkno/tv-automation-atem-connection/compare/1.3.0...1.3.1) (2020-06-17)
@@ -7,0 +15,0 @@

dist/lib/tally.js

@@ -14,3 +14,3 @@ "use strict";
     // Start with the basics: the surface level of what is in the target ME.
-    _calcActiveMeInputs(mode, state, me).forEach(i => inputs.add(i));
+    _calcActiveMeInputs(inputs, mode, state, me);
     // Loop over the active input IDs we've found so far,
@@ -28,6 +28,6 @@ // and check if any of them are SuperSources or other nested MEs.
         lastSize = inputs.size;
-        Array.from(inputs).slice(lastProcessed).forEach(inputId => {
+        for (const inputId of Array.from(inputs).slice(lastProcessed)) {
             if (!state.inputs[inputId]) {
                 // Data isn't hydrated yet, we'll get 'em next time.
-                return;
+                continue;
             }
@@ -39,7 +39,7 @@ const portType = state.inputs[inputId].internalPortType;
                     const ssrc = state.video.getSuperSource(ssrcId);
-                    Object.values(ssrc.boxes).forEach(box => {
+                    for (const box of Object.values(ssrc.boxes)) {
                         if (box.enabled) {
                             inputs.add(box.source);
                         }
-                    });
+                    }
                     inputs.add(ssrc.properties.artFillSource);
@@ -53,3 +53,3 @@ if (ssrc.properties.artOption === Enums.SuperSourceArtOption.Foreground) {
                     const nestedMeMode = (inputId - 10000) % 10 === 0 ? 'program' : 'preview';
-                    _calcActiveMeInputs(nestedMeMode, state, nestedMeId).forEach(i => inputs.add(i));
+                    _calcActiveMeInputs(inputs, nestedMeMode, state, nestedMeId);
                     break;
@@ -59,4 +59,4 @@ default:
             }
-        });
-        lastProcessed = inputs.size - 1;
+        }
+        lastProcessed = lastSize - 1;
     } while (inputs.size !== lastSize);
@@ -74,4 +74,3 @@ // undefined sometimes sneaks its way in here.
  */
-function _calcActiveMeInputs(mode, state, meId) {
-    const inputs = new Set();
+function _calcActiveMeInputs(inputs, mode, state, meId) {
     const meRef = state.video.getMe(meId);
@@ -165,4 +164,3 @@ if (mode === 'preview') {
     }
-    return Array.from(inputs);
 }
 //# sourceMappingURL=tally.js.map

package.json

 {
   "name": "atem-connection",
-  "version": "1.3.1",
+  "version": "1.3.2",
   "description": "Typescript Node.js library for connecting with an ATEM switcher.",
@@ -5,0 +5,0 @@ "main": "dist/index.js",

No alert changes

Improved metrics

Total package byte prevSize

513701

increased by0.02%

Worsened metrics

Lines of code

7557

decreased by-0.03%

No dependency changes