Fix exceptions referenced in AlexandreRouma#1287

Author: AlexandreRouma

core/src/command_args.cpp

@@ -88,7 +88,7 @@ int CommandArgsParser::parse(int argc, char* argv[]) {
             try {
                 carg.ival = std::stoi(arg);
             }
-            catch (std::exception e) {
+            catch (const std::exception& e) {
                 printf("Invalid argument, failed to parse integer\n");
                 showHelp();
                 return -1;
@@ -98,7 +98,7 @@ int CommandArgsParser::parse(int argc, char* argv[]) {
             try {
                 carg.fval = std::stod(arg);
             }
-            catch (std::exception e) {
+            catch (const std::exception& e) {
                 printf("Invalid argument, failed to parse float\n");
                 showHelp();
                 return -1;

core/src/config.cpp

@@ -36,8 +36,8 @@ void ConfigManager::load(json def, bool lock) {
         file >> conf;
         file.close();
     }
-    catch (std::exception e) {
-        flog::error("Config file '{0}' is corrupted, resetting it", path);
+    catch (const std::exception& e) {
+        flog::error("Config file '{}' is corrupted, resetting it: {}", path, e.what());
         conf = def;
         save(false);
     }

core/src/utils/networking.cpp

@@ -320,7 +320,7 @@ namespace net {
                 }
                 entry.handler(std::move(client), entry.ctx);
             }
-            catch (std::exception e) {
+            catch (const std::exception& e) {
                 listening = false;
                 return;
             }

misc_modules/rigctl_client/src/main.cpp

@@ -80,8 +80,8 @@ class RigctlClientModule : public ModuleManager::Instance {
         try {
             client = net::rigctl::connect(host, port);
         }
-        catch (std::exception e) {
-            flog::error("Could not connect: {0}", e.what());
+        catch (const std::exception& e) {
+            flog::error("Could not connect: {}", e.what());
             return;
         }
 

misc_modules/rigctl_server/src/main.cpp

@@ -200,8 +200,8 @@ class SigctlServerModule : public ModuleManager::Instance {
             listener = net::listen(hostname, port);
             listener->acceptAsync(clientHandler, this);
         }
-        catch (std::exception e) {
-            flog::error("Could not start rigctl server: {0}", e.what());
+        catch (const std::exception& e) {
+            flog::error("Could not start rigctl server: {}", e.what());
         }
     }
 

source_modules/airspy_source/src/main.cpp

@@ -141,10 +141,10 @@ class AirspySourceModule : public ModuleManager::Instance {
                 return;
             }
         }
-        catch (std::exception e) {
+        catch (const std::exception& e) {
             char buf[1024];
             sprintf(buf, "%016" PRIX64, serial);
-            flog::error("Could not open Airspy {0}", buf);
+            flog::error("Could not open Airspy {}", buf);
         }
         selectedSerial = serial;
 

source_modules/airspyhf_source/src/main.cpp

@@ -144,10 +144,10 @@ class AirspyHFSourceModule : public ModuleManager::Instance {
                 return;
             }
         }
-        catch (std::exception e) {
+        catch (const std::exception& e) {
             char buf[1024];
             sprintf(buf, "%016" PRIX64, serial);
-            flog::error("Could not open Airspy HF+ {0}", buf);
+            flog::error("Could not open Airspy HF+ {}", buf);
         }
 
         selectedSerial = serial;

source_modules/audio_source/src/main.cpp

@@ -200,11 +200,11 @@ class AudioSourceModule : public ModuleManager::Instance {
             _this->audio.startStream();
             _this->running = true;
         }
-        catch (std::exception e) {
-            flog::error("Error opening audio device: {0}", e.what());
+        catch (const std::exception& e) {
+            flog::error("Error opening audio device: {}", e.what());
         }
 
-        flog::info("AudioSourceModule '{0}': Start!", _this->name);
+        flog::info("AudioSourceModule '{}': Start!", _this->name);
     }
 
     static void stop(void* ctx) {

source_modules/file_source/src/main.cpp

@@ -139,8 +139,8 @@ class FileSourceModule : public ModuleManager::Instance {
                     //gui::freqSelect.maxFreq = _this->centerFreq + (_this->sampleRate/2);
                     //gui::freqSelect.limitFreq = true;
                 }
-                catch (std::exception& e) {
-                    flog::error("Error: {0}", e.what());
+                catch (const std::exception& e) {
+                    flog::error("Error: {}", e.what());
                 }
                 config.acquire();
                 config.conf["path"] = _this->fileSelect.path;

source_modules/rfspace_source/src/main.cpp

@@ -154,8 +154,8 @@ class RFSpaceSourceModule : public ModuleManager::Instance {
                 _this->client = rfspace::connect(_this->hostname, _this->port, &_this->stream);
                 _this->deviceInit();
             }
-            catch (std::exception e) {
-                flog::error("Could not connect to SDR: {0}", e.what());
+            catch (const std::exception& e) {
+                flog::error("Could not connect to SDR: {}", e.what());
             }
         }
         else if (connected && SmGui::Button("Disconnect##rfspace_source")) {