commit f2d8bb48366d812180a8684e3ff58f26425854fa
author Jonathan Doman <jonathan.doman@intel.com> Wed Jul 26 10:13:34 2023 -0700
committer Zhikui Ren <zhikui.ren@intel.com> Sat Aug 05 00:27:59 2023 +0000
tree e39c681398743785dac2ca065922b27d0d1d434a
parent 410bbc271b8e90520905589ae7c505ec78396ccc

Fix compilation warnings

Various small issues:
* Member initialization order
* Comparison of different signedness
* Unused parameters
* Unused variable

Change-Id: Ie59db239b4216ad089f7cf0f289e6ed3d6ac8e18
Signed-off-by: Jonathan Doman <jonathan.doman@intel.com>

include/system.hpp

diff --git a/include/system.hpp b/include/system.hpp
index 992408e..4753a01 100644
--- a/include/system.hpp
+++ b/include/system.hpp
@@ -45,11 +45,10 @@
         sdbusplus::server::object_t<
             sdbusplus::server::xyz::openbmc_project::common::UUID>(
             bus, objPath.c_str()),
-        bus(bus),
         sdbusplus::server::object_t<sdbusplus::server::xyz::openbmc_project::
                                         inventory::decorator::Revision>(
             bus, objPath.c_str()),
-        path(objPath), storage(smbiosTableStorage)
+        bus(bus), path(objPath), storage(smbiosTableStorage)
     {
         std::string input = "0";
         uuid(input);