IPMIFruArea: move private area under public
For standard readability a class is typically written with public
preceding private.
Change-Id: Icdce1ee8426d651293d16f9a1ba16176a3f767c3
Signed-off-by: Patrick Venture <venture@google.com>
diff --git a/fru_area.hpp b/fru_area.hpp
index 2a6eed0..8cb9c00 100644
--- a/fru_area.hpp
+++ b/fru_area.hpp
@@ -13,43 +13,6 @@
class IPMIFruArea
{
- private:
- // Unique way of identifying a FRU
- uint8_t fruid = 0;
-
- // Type of the fru matching offsets in common header
- ipmi_fru_area_type type = IPMI_FRU_AREA_INTERNAL_USE;
-
- // Name of the fru area. ( BOARD/CHASSIS/PRODUCT )
- std::string name;
-
- // Length of a specific fru area.
- size_t len = 0;
-
- // Special bit for BMC readable eeprom only.
- bool bmc_fru = false;
-
- // If a FRU is physically present.
- bool isPresent = false;
-
- // Whether a particular area is valid ?
- bool isValid = false;
-
- // Actual area data.
- uint8_t* data = nullptr;
-
- // fru inventory dbus name
- std::string bus_name;
-
- // fru inventory dbus object path
- std::string obj_path;
-
- // fru inventory dbus interface name
- std::string intf_name;
-
- // Default constructor disabled.
- IPMIFruArea();
-
public:
// constructor
IPMIFruArea(const uint8_t fruid, const ipmi_fru_area_type type,
@@ -117,6 +80,43 @@
// Sets the dbus parameters
void update_dbus_paths(const char*, const char*, const char*);
+
+ private:
+ // Unique way of identifying a FRU
+ uint8_t fruid = 0;
+
+ // Type of the fru matching offsets in common header
+ ipmi_fru_area_type type = IPMI_FRU_AREA_INTERNAL_USE;
+
+ // Name of the fru area. ( BOARD/CHASSIS/PRODUCT )
+ std::string name;
+
+ // Length of a specific fru area.
+ size_t len = 0;
+
+ // Special bit for BMC readable eeprom only.
+ bool bmc_fru = false;
+
+ // If a FRU is physically present.
+ bool isPresent = false;
+
+ // Whether a particular area is valid ?
+ bool isValid = false;
+
+ // Actual area data.
+ uint8_t* data = nullptr;
+
+ // fru inventory dbus name
+ std::string bus_name;
+
+ // fru inventory dbus object path
+ std::string obj_path;
+
+ // fru inventory dbus interface name
+ std::string intf_name;
+
+ // Default constructor disabled.
+ IPMIFruArea();
};
#endif