From 358a350e11d26e882996a704821d83b846aab0d8 Mon Sep 17 00:00:00 2001 From: Edward O'Callaghan Date: Sat, 19 Dec 2020 11:21:49 +1100 Subject: layout.h,c: Use 'false' over '0' for bool type The field member 'included' is of type boolean and so keep to using 'true, false' values over numerics like '0'. Get rid of a unnecessary yet trivial tab at the end of layout.h while we are here. BUG=none BRANCH=none TEST=builds Change-Id: Ib594de2834175482ae5e36d9dd354ef2555c53d5 Signed-off-by: Edward O'Callaghan Reviewed-on: https://review.coreboot.org/c/flashrom/+/48743 Tested-by: build bot (Jenkins) Reviewed-by: Sam McNally Reviewed-by: Angel Pons --- layout.c | 4 ++-- layout.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/layout.c b/layout.c index d80b01fa..272912c6 100644 --- a/layout.c +++ b/layout.c @@ -81,7 +81,7 @@ int read_romlayout(const char *name) } layout->entries[layout->num_entries].start = strtol(tstr1, (char **)NULL, 16); layout->entries[layout->num_entries].end = strtol(tstr2, (char **)NULL, 16); - layout->entries[layout->num_entries].included = 0; + layout->entries[layout->num_entries].included = false; layout->entries[layout->num_entries].name = strdup(tempname); if (!layout->entries[layout->num_entries].name) { msg_gerr("Error adding layout entry: %s\n", strerror(errno)); @@ -205,7 +205,7 @@ void layout_cleanup(struct layout_include_args **args) for (i = 0; i < layout->num_entries; i++) { free(layout->entries[i].name); - layout->entries[i].included = 0; + layout->entries[i].included = false; } layout->num_entries = 0; } diff --git a/layout.h b/layout.h index 8e4eb134..cceedd1f 100644 --- a/layout.h +++ b/layout.h @@ -67,4 +67,4 @@ int process_include_args(struct flashrom_layout *l, const struct layout_include_ const struct romentry *layout_next_included_region(const struct flashrom_layout *, chipoff_t); const struct romentry *layout_next_included(const struct flashrom_layout *, const struct romentry *); -#endif /* !__LAYOUT_H__ */ +#endif /* !__LAYOUT_H__ */ -- cgit v1.2.3