commit 6753d8c35bb462e978e6d22d5bc306291bb8e544
author Patrick Venture <venture@google.com> Fri Aug 09 16:10:35 2019 -0700
committer Patrick Venture <venture@google.com> Mon Aug 12 19:16:10 2019 -0700
tree 00d96d3746ab79ad553da374e0c7580bf6198ea9
parent e0e6f5f85392eb4152e70ef04d733467210fd191

style: frudevice: drop return after exit

Signed-off-by: Patrick Venture <venture@google.com>
Change-Id: I53fc08b7e0a81bfdb4ee1fa37a503c6666a69ae5

src/FruDevice.cpp

diff --git a/src/FruDevice.cpp b/src/FruDevice.cpp
index aac0d89..d4fbc0b 100644
--- a/src/FruDevice.cpp
+++ b/src/FruDevice.cpp
@@ -526,7 +526,6 @@
         std::cerr << "Illegal blacklist file detected, cannot validate JSON, "
                      "exiting\n";
         std::exit(EXIT_FAILURE);
-        return;
     }
 
     // It's expected to have at least one field, "buses" that is an array of the
@@ -536,7 +535,6 @@
     {
         std::cerr << "Illegal blacklist, expected to read dictionary\n";
         std::exit(EXIT_FAILURE);
-        return;
     }
 
     // If buses field is missing, that's fine.
@@ -549,7 +547,6 @@
             // Buses field present but invalid, therefore this is an error.
             std::cerr << "Invalid contents for blacklist buses field\n";
             std::exit(EXIT_FAILURE);
-            return;
         }
 
         // Catch exception here for type mis-match.
@@ -565,7 +562,6 @@
             // Type mis-match is a critical error.
             std::cerr << "Invalid bus type: " << e.what() << "\n";
             std::exit(EXIT_FAILURE);
-            return;
         }
     }