build: enable and fix -Wshadow

Shadow names can be prone to bugs.  Enable the warning to avoid them in
the future.  Tested by building and running the unit tests.

Change-Id: Ic3227675f4ff40d266ae2a60c66c894f16e5888e
Signed-off-by: Brad Bishop <bradbish@qti.qualcomm.com>
diff --git a/src/main.cpp b/src/main.cpp
index e84abfd..b79d722 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -67,18 +67,20 @@
     InProgressIntrospect& operator=(const InProgressIntrospect&) = delete;
     InProgressIntrospect& operator=(InProgressIntrospect&&) = delete;
     InProgressIntrospect(
-        sdbusplus::asio::connection* systemBus, boost::asio::io_context& io,
-        const std::string& processName, AssociationMaps& am
+        sdbusplus::asio::connection* systemBusConnection,
+        boost::asio::io_context& ioContext,
+        const std::string& introspectProcessName, AssociationMaps& am
 #ifdef MAPPER_ENABLE_DEBUG
         ,
         std::shared_ptr<std::chrono::time_point<std::chrono::steady_clock>>
-            globalStartTime
+            globalIntrospectStartTime
 #endif
         ) :
-        systemBus(systemBus), io(io), processName(processName), assocMaps(am)
+        systemBus(systemBusConnection), io(ioContext),
+        processName(introspectProcessName), assocMaps(am)
 #ifdef MAPPER_ENABLE_DEBUG
         ,
-        globalStartTime(std::move(globalStartTime)),
+        globalStartTime(std::move(globalIntrospectStartTime)),
         processStartTime(std::chrono::steady_clock::now())
 #endif
     {}