commit c1d4de5e1277356e74dfb9de647ff5e19ef75445
author Patrick Williams <patrick@stwcx.xyz> Wed Oct 06 12:45:57 2021 -0500
committer Patrick Williams <patrick@stwcx.xyz> Wed Oct 06 12:45:57 2021 -0500
tree 5b54a6ca02d6b6f086b22e0e962105d6affece5f
parent a588eafec450a1efcfd0d14b819f703e0e4e8d8d

catch exceptions as const

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: If6aba71f482f75143e6d669763b8853db22318cf

phosphor-power-supply/power_supply.cpp

diff --git a/phosphor-power-supply/power_supply.cpp b/phosphor-power-supply/power_supply.cpp
index ed1f9af..96dfaa0 100644
--- a/phosphor-power-supply/power_supply.cpp
+++ b/phosphor-power-supply/power_supply.cpp
@@ -114,7 +114,7 @@
         file << bindDevice;
         file.close();
     }
-    catch (std::exception& e)
+    catch (const std::exception& e)
     {
         auto err = errno;
 
@@ -156,7 +156,7 @@
             present = false;
         }
     }
-    catch (std::exception& e)
+    catch (const std::exception& e)
     {
         log<level::ERR>(
             fmt::format("presenceGPIO read fail: {}", e.what()).c_str());
@@ -261,7 +261,7 @@
                 vinUVFault = false;
             }
         }
-        catch (ReadFailure& e)
+        catch (const ReadFailure& e)
         {
             readFail++;
             phosphor::logging::commit<ReadFailure>();
@@ -316,7 +316,7 @@
             static_cast<void>(
                 pmbusIntf->read("in1_input", phosphor::pmbus::Type::Hwmon));
         }
-        catch (ReadFailure& e)
+        catch (const ReadFailure& e)
         {
             // Since I do not care what the return value is, I really do not
             // care much if it gets a ReadFailure either. However, this
@@ -426,7 +426,7 @@
             assetProps.emplace(MODEL_PROP, ccin);
             modelName = ccin;
         }
-        catch (ReadFailure& e)
+        catch (const ReadFailure& e)
         {
             // Ignore the read failure, let pmbus code indicate failure,
             // path...
@@ -441,7 +441,7 @@
             pn = pmbusIntf->readString(PART_NUMBER, Type::HwmonDeviceDebug);
             assetProps.emplace(PN_PROP, pn);
         }
-        catch (ReadFailure& e)
+        catch (const ReadFailure& e)
         {
             // Ignore the read failure, let pmbus code indicate failure,
             // path...
@@ -451,7 +451,7 @@
         {
             fn = pmbusIntf->readString(FRU_NUMBER, Type::HwmonDeviceDebug);
         }
-        catch (ReadFailure& e)
+        catch (const ReadFailure& e)
         {
             // Ignore the read failure, let pmbus code indicate failure,
             // path...
@@ -464,7 +464,7 @@
             sn = pmbusIntf->readString(SERIAL_NUMBER, Type::HwmonDeviceDebug);
             assetProps.emplace(SN_PROP, sn);
         }
-        catch (ReadFailure& e)
+        catch (const ReadFailure& e)
         {
             // Ignore the read failure, let pmbus code indicate failure,
             // path...
@@ -476,7 +476,7 @@
                 pmbusIntf->readString(FW_VERSION, Type::HwmonDeviceDebug);
             versionProps.emplace(VERSION_PROP, fwVersion);
         }
-        catch (ReadFailure& e)
+        catch (const ReadFailure& e)
         {
             // Ignore the read failure, let pmbus code indicate failure,
             // path...
@@ -541,7 +541,7 @@
 
             auto reply = bus.call(method);
         }
-        catch (std::exception& e)
+        catch (const std::exception& e)
         {
             log<level::ERR>(
                 std::string(e.what() + std::string(" PATH=") + inventoryPath)