diff options
author | Patrick Venture <venture@google.com> | 2018-10-20 20:52:27 -0700 |
---|---|---|
committer | Patrick Venture <venture@google.com> | 2018-10-29 09:07:22 -0700 |
commit | 524ba9c1bd349141c79f213c56afc3b25f699593 (patch) | |
tree | 43d7bcddc31cfd8b6e8817ace8f7b530aaec3c07 | |
parent | b9d33736436382db5c1a9788cae17894da36aec4 (diff) | |
download | ipmi-fru-parser-524ba9c1bd349141c79f213c56afc3b25f699593.tar.gz ipmi-fru-parser-524ba9c1bd349141c79f213c56afc3b25f699593.zip |
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>
-rw-r--r-- | fru_area.hpp | 74 |
1 files changed, 37 insertions, 37 deletions
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 @@ using std::uint8_t; 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 @@ class IPMIFruArea // 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 |