From 0d24d412485b072677bfbf75a5c5db062e8c42c1 Mon Sep 17 00:00:00 2001
From: Frank Kopp <frank@familie-kopp.de>
Date: Sat, 20 Apr 2024 22:10:42 +0200
Subject: [PATCH] reorder of check if on to better compare when running

---
 .../wasm/lvar-provider/src/LvarConverter.cpp  | 40 +++++++++----------
 1 file changed, 19 insertions(+), 21 deletions(-)

diff --git a/fbw-lvar-provider/src/wasm/lvar-provider/src/LvarConverter.cpp b/fbw-lvar-provider/src/wasm/lvar-provider/src/LvarConverter.cpp
index 925eb642f14..dd565b77174 100644
--- a/fbw-lvar-provider/src/wasm/lvar-provider/src/LvarConverter.cpp
+++ b/fbw-lvar-provider/src/wasm/lvar-provider/src/LvarConverter.cpp
@@ -36,36 +36,34 @@ void LvarConverter::update() {
     readVarFile();
   }
 
-  // only run var updates if the bridge is activated
-  if (!get_named_variable_value(isLvarBridgeOnID)) {
-    return;
-  }
-
 #ifdef PROFILING
   profiler.start();
 #endif
 
-  for (std::pair<int, int>& ids : arinc429Vars) {
-    // DEBUG
-    PCSTRINGZ firstName  = get_name_of_named_variable(ids.first);
-    PCSTRINGZ secondName = get_name_of_named_variable(ids.second);
+  if (get_named_variable_value(isLvarBridgeOnID)) {
+    // process vars
+    for (std::pair<int, int>& ids : arinc429Vars) {
+      // DEBUG
+      PCSTRINGZ firstName  = get_name_of_named_variable(ids.first);
+      PCSTRINGZ secondName = get_name_of_named_variable(ids.second);
 
-    auto value = get_named_variable_value(ids.first);
+      auto value = get_named_variable_value(ids.first);
 
-    Arinc429NumericWord arinc429NumericWord{value};
-    arinc429NumericWord.setSsm(Arinc429SignStatus::FunctionalTest);
-    float rawValue = arinc429NumericWord.valueOr(0.0f);
+      Arinc429NumericWord arinc429NumericWord{value};
+      arinc429NumericWord.setSsm(Arinc429SignStatus::FunctionalTest);
+      float rawValue = arinc429NumericWord.valueOr(0.0f);
 
-    // DEBUG
-    if (tickCounter % 100 == 0 && rawValue != 0.0f && get_named_variable_value(isLvarBridgeVerbose)) {
-      std::cout << "LVar: " << firstName << " = " << value << " Raw Value: " << secondName << " = " << rawValue << std::endl;
+      // DEBUG
+      if (tickCounter % 100 == 0 && rawValue != 0.0f && get_named_variable_value(isLvarBridgeVerbose)) {
+        std::cout << "LVar: " << firstName << " = " << value << " Raw Value: " << secondName << " = " << rawValue << std::endl;
+      }
+      set_named_variable_value(ids.second, rawValue ? rawValue : -1.0f);
     }
-    set_named_variable_value(ids.second, rawValue);
-  }
 
-  // DEBUG
-  if (tickCounter % 100 == 0) {
-    std::cout << "Processed " << arinc429Vars.size() << " vars" << std::endl;
+    // DEBUG
+    if (tickCounter % 100 == 0) {
+      std::cout << "Processed " << arinc429Vars.size() << " vars per tick" << std::endl;
+    }
   }
 
 #ifdef PROFILING