summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPatrick Venture <venture@google.com>2018-10-20 20:52:27 -0700
committerPatrick Venture <venture@google.com>2018-10-29 09:07:22 -0700
commit524ba9c1bd349141c79f213c56afc3b25f699593 (patch)
tree43d7bcddc31cfd8b6e8817ace8f7b530aaec3c07
parentb9d33736436382db5c1a9788cae17894da36aec4 (diff)
downloadipmi-fru-parser-524ba9c1bd349141c79f213c56afc3b25f699593.zip
ipmi-fru-parser-524ba9c1bd349141c79f213c56afc3b25f699593.tar.gz
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.hpp74
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
OpenPOWER on IntegriCloud