diff options
author | Patrick Venture <venture@google.com> | 2018-10-20 21:15:41 -0700 |
---|---|---|
committer | Patrick Venture <venture@google.com> | 2018-10-29 09:07:55 -0700 |
commit | 0e3a1c4eb5af9b940c96d262da2af2465b189c66 (patch) | |
tree | 1a92fe0b16b037b74dbe7a8d8df82e77000c8100 | |
parent | f22b36a061e4713c75233a68a86b81ed25d1e6a3 (diff) | |
download | ipmi-fru-parser-0e3a1c4eb5af9b940c96d262da2af2465b189c66.tar.gz ipmi-fru-parser-0e3a1c4eb5af9b940c96d262da2af2465b189c66.zip |
writefrudata: cleanup extra parentheses
Cleanup extra parentheses from calling accessors against object pointer,
which is actually a reference to a unique_ptr.
Change-Id: Icf47bf376947d1afb0753c18be66417e2bac4507
Signed-off-by: Patrick Venture <venture@google.com>
-rw-r--r-- | writefrudata.cpp | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/writefrudata.cpp b/writefrudata.cpp index d6eb8f9..cb0d8eb 100644 --- a/writefrudata.cpp +++ b/writefrudata.cpp @@ -463,9 +463,9 @@ int ipmi_populate_fru_areas(uint8_t* fru_data, const size_t data_len, // of the fields populated. Update the data portion now. for (auto& iter : fru_area_vec) { - if ((iter)->getType() == get_fru_area_type(fru_entry)) + if (iter->getType() == get_fru_area_type(fru_entry)) { - (iter)->setData(area_data, area_len); + iter->setData(area_data, area_len); } } } // If we have fru data present @@ -597,13 +597,13 @@ int validateFRUArea(const uint8_t fruid, const char* fru_file_name, #ifdef __IPMI_DEBUG__ for (auto& iter : fru_area_vec) { - std::printf("FRU ID : [%d]\n", (iter)->getFruID()); - std::printf("AREA NAME : [%s]\n", (iter)->getName()); - std::printf("TYPE : [%d]\n", (iter)->getType()); - std::printf("LEN : [%d]\n", (iter)->getLength()); - std::printf("BUS NAME : [%s]\n", (iter)->getBusName()); - std::printf("OBJ PATH : [%s]\n", (iter)->getObjectPath()); - std::printf("INTF NAME :[%s]\n", (iter)->getIntfName()); + std::printf("FRU ID : [%d]\n", iter->getFruID()); + std::printf("AREA NAME : [%s]\n", iter->getName()); + std::printf("TYPE : [%d]\n", iter->getType()); + std::printf("LEN : [%d]\n", iter->getLength()); + std::printf("BUS NAME : [%s]\n", iter->getBusName()); + std::printf("OBJ PATH : [%s]\n", iter->getObjectPath()); + std::printf("INTF NAME :[%s]\n", iter->getIntfName()); } #endif |