code sanitizations

src/display.cpp

@@ -6,6 +6,13 @@
 
 HAS_DISPLAY u8x8(OLED_RST, OLED_SCL, OLED_SDA);
 
+// helper string for converting LoRa spread factor values
+#if defined(CFG_eu868)
+const char lora_datarate[] = {"1211100908077BFSNA"};
+#elif defined(CFG_us915)
+const char lora_datarate[] = {"100908078CNA121110090807"};
+#endif
+
 // helper function, prints a hex key on display
 void DisplayKey(const uint8_t *key, uint8_t len, bool lsb) {
   const uint8_t *p;

src/globals.h

@@ -17,17 +17,10 @@ extern char display_line6[], display_line7[];
 extern int countermode, screensaver, adrmode, lorasf, txpower, rlim;
 extern uint8_t channel, DisplayState;
 extern uint16_t macs_total, macs_wifi, macs_ble; // MAC counters
-extern uint64_t uptimecounter;
 extern std::set<uint16_t> macs;
 extern hw_timer_t *channelSwitch, *sendCycle;
 extern portMUX_TYPE timerMux;
 
-#if defined(CFG_eu868)
-const char lora_datarate[] = {"1211100908077BFSNA"};
-#elif defined(CFG_us915)
-const char lora_datarate[] = {"100908078CNA121110090807"};
-#endif
-
 #ifdef HAS_GPS
 extern gpsStatus_t gps_status; // struct for storing gps data
 extern TinyGPSPlus gps;        // Make TinyGPS++ instance globally availabe

src/macsniff.h

@@ -4,6 +4,7 @@
 // ESP32 Functions
 #include <esp_wifi.h>
 
+// Hash function for scrambling MAC addresses
 #include "hash.h"
 
 #define MAC_SNIFF_WIFI 0

src/main.cpp

@@ -32,7 +32,6 @@ char display_line6[16], display_line7[16]; // display buffers
 uint8_t DisplayState = 0, channel = 0;     // globals for state machine
 uint16_t macs_total = 0, macs_wifi = 0,
          macs_ble = 0;      // MAC counters globals for display
-uint64_t uptimecounter = 0; // timer global for uptime counter
 hw_timer_t *channelSwitch = NULL, *displaytimer = NULL,
            *sendCycle = NULL; // configure hardware timer for cyclic tasks
 
@@ -438,8 +437,6 @@ void loop() {
 
     // state machine for uptime, display, LED, button, lowmemory, senddata
 
-    uptimecounter = uptime() / 1000; // counts uptime in seconds (64bit)
-
 #if (HAS_LED != NOT_A_PIN) || defined(HAS_RGB_LED)
     led_loop();
 #endif

src/main.h

@@ -74,3 +74,4 @@ extern TinyGPSPlus gps;        // Make TinyGPS++ instance globally availabe
 void reset_counters(void);
 void blink_LED(uint16_t set_color, uint16_t set_blinkduration);
 void led_loop(void);
+uint64_t uptime();

src/rcommand.cpp

@@ -284,7 +284,7 @@ void get_status(uint8_t val) {
   uint16_t voltage = 0;
 #endif
   payload.reset();
-  payload.addStatus(voltage, uptimecounter, temperatureRead());
+  payload.addStatus(voltage, uptime() / 1000, temperatureRead());
   senddata(STATUSPORT);
 };