PEL: Const accessors for Private/UserHeader
Change the combined non-const accessor/setter functions for the
PrivateHeader and UserHeader fields to the standard const accessors, and
have separate setters for the fields that need to be modified.
In addition, make the 'get PrivateHeader' and 'get UserHeader' functions
on the PEL class return a const reference.
This allows const enforcement on the PEL class, for things like:
void func(const PEL& pel)
{
auto id = pel.privateHeader().id();
}
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I840170e72b41e9e2465f716617500269244de6d9
diff --git a/extensions/openpower-pels/user_header.hpp b/extensions/openpower-pels/user_header.hpp
index c1b6636..459952b 100644
--- a/extensions/openpower-pels/user_header.hpp
+++ b/extensions/openpower-pels/user_header.hpp
@@ -64,9 +64,9 @@
/**
* @brief Returns the subsystem field.
*
- * @return uint8_t& - the subsystem
+ * @return uint8_t - the subsystem
*/
- uint8_t& subsystem()
+ uint8_t subsystem() const
{
return _eventSubsystem;
}
@@ -74,9 +74,9 @@
/**
* @brief Returns the event scope field.
*
- * @return uint8_t& - the event scope
+ * @return uint8_t - the event scope
*/
- uint8_t& scope()
+ uint8_t scope() const
{
return _eventScope;
}
@@ -84,9 +84,9 @@
/**
* @brief Returns the severity field.
*
- * @return uint8_t& - the severity
+ * @return uint8_t - the severity
*/
- uint8_t& severity()
+ uint8_t severity() const
{
return _eventSeverity;
}
@@ -94,9 +94,9 @@
/**
* @brief Returns the event type field.
*
- * @return uint8_t& - the event type
+ * @return uint8_t - the event type
*/
- uint8_t& eventType()
+ uint8_t eventType() const
{
return _eventType;
}
@@ -104,9 +104,9 @@
/**
* @brief Returns the problem domain field.
*
- * @return uint8_t& - the problem domain
+ * @return uint8_t - the problem domain
*/
- uint8_t& problemDomain()
+ uint8_t problemDomain() const
{
return _problemDomain;
}
@@ -114,9 +114,9 @@
/**
* @brief Returns the problem vector field.
*
- * @return uint8_t& - the problem vector
+ * @return uint8_t - the problem vector
*/
- uint8_t& problemVector()
+ uint8_t problemVector() const
{
return _problemVector;
}
@@ -124,9 +124,9 @@
/**
* @brief Returns the action flags field.
*
- * @return uint16_t& - the action flags
+ * @return uint16_t - the action flags
*/
- uint16_t& actionFlags()
+ uint16_t actionFlags() const
{
return _actionFlags;
}