commit 6b094f8b9d0908504358f5bb4b17c87a4511e3bc
author Patrick Venture <venture@google.com> Wed Oct 17 13:16:27 2018 -0700
committer Patrick Venture <venture@google.com> Thu Oct 18 18:49:07 2018 +0000
tree 8361f1e68e6b0b9d3b7fa85c62b114be3cc2251b
parent 52f1f1892e9097aaebb4c1b3ca7fc76cf5126bb6

frup.cpp: drop unused variable

[frup.cpp:628] -> [frup.cpp:694]: (style) The expression 'sdr < 0'
is always false.

This is true because the value is set to 0 and never re-assigned.

Change-Id: I668f5cc5a13a0c87afa3af978c341fb60f51b70d
Signed-off-by: Patrick Venture <venture@google.com>

frup.cpp

diff --git a/frup.cpp b/frup.cpp
index b3b2942..299c7cc 100644
--- a/frup.cpp
+++ b/frup.cpp
@@ -625,7 +625,6 @@
                     IPMI_FRU_TYPE_LENGTH_TYPE_CODE_SHIFT;
     int vpd_val_len =
         type_length & IPMI_FRU_TYPE_LENGTH_NUMBER_OF_DATA_BYTES_MASK;
-    int sdr = 0;
 
     /* Needed to convert each uint8_t byte to a ascii */
     char bin_byte[3] = {0};
@@ -690,15 +689,6 @@
         free(bin_in_ascii);
         bin_in_ascii = NULL;
     }
-
-    if (sdr < 0)
-    {
-#if IPMI_FRU_PARSER_DEBUG
-        printf(
-            "_append_to_dict : sd_bus_message_append Failed [ %d ] for [%s]\n",
-            sdr, vpd_key_names[vpd_key_id]);
-#endif
-    }
 }
 
 int parse_fru_area(const uint8_t area, const void* msgbuf, const size_t len,