commit 0e3a1c4eb5af9b940c96d262da2af2465b189c66
author Patrick Venture <venture@google.com> Sat Oct 20 21:15:41 2018 -0700
committer Patrick Venture <venture@google.com> Mon Oct 29 09:07:55 2018 -0700
tree 1a92fe0b16b037b74dbe7a8d8df82e77000c8100
parent f22b36a061e4713c75233a68a86b81ed25d1e6a3

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>

writefrudata.cpp

diff --git a/writefrudata.cpp b/writefrudata.cpp
index d6eb8f9..cb0d8eb 100644
--- a/writefrudata.cpp
+++ b/writefrudata.cpp
@@ -463,9 +463,9 @@
             // 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 @@
 #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