diff options
author | Patrick Venture <venture@google.com> | 2018-12-04 09:52:54 -0800 |
---|---|---|
committer | Alexander Amelkin <mocbuhtig@amelkin.msk.ru> | 2019-02-13 21:28:14 +0300 |
commit | 8991f5a9a0c6a4dc13732c940bf92f55e890c9ea (patch) | |
tree | 4e49e7eba26bb475dfa846a60bb90ab8f4622b98 | |
parent | 49384e47beeb8b1c08c2846129369e05bc832c74 (diff) | |
download | ipmitool-8991f5a9a0c6a4dc13732c940bf92f55e890c9ea.tar.gz |
fru: change ipmi_fru_query_new_value to return bool
Change ipmi_fru_query_new_value to return the bool type
instead of an int that's being used as a boolean value.
Signed-off-by: Patrick Venture <venture@google.com>
-rw-r--r-- | lib/ipmi_fru.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/ipmi_fru.c b/lib/ipmi_fru.c index 5e3fde3..9f59283 100644 --- a/lib/ipmi_fru.c +++ b/lib/ipmi_fru.c @@ -1487,17 +1487,17 @@ fru_area_print_multirec(struct ipmi_intf * intf, struct fru_info * fru, * returns : FALSE if data not changed */ static -int +bool ipmi_fru_query_new_value(uint8_t *data,int offset, size_t len) { - int status=FALSE; + bool status = false; int ret; char answer; printf("Would you like to change this value <y/n> ? "); ret = scanf("%c", &answer); if (ret != 1) { - return FALSE; + return false; } if( answer == 'y' || answer == 'Y' ){ @@ -1514,7 +1514,7 @@ ipmi_fru_query_new_value(uint8_t *data,int offset, size_t len) ret = scanf("%x", holder+i); if (ret != 1) { free(holder); - return FALSE; + return false; } } for( i=0;i<len;i++ ){ @@ -1523,7 +1523,7 @@ ipmi_fru_query_new_value(uint8_t *data,int offset, size_t len) /* &data[offset++] */ free(holder); holder = NULL; - status = TRUE; + status = true; } else{ printf("Entered %c\n",answer); |