[coreboot-gerrit] New patch to review for coreboot: cbfstool: Remove extra comma after {0, NULL} list element

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Wed Jul 15 21:53:00 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10932

-gerrit

commit dbd02be98f068da9f324fd2cf522afede4556969
Author: Sol Boucher <solb at chromium.org>
Date:   Thu May 7 21:00:05 2015 -0700

    cbfstool: Remove extra comma after {0, NULL} list element
    
    Trailing commas are useful for lists that can be extended. These lists are
    0-terminated, and there should be no elements following that.
    
    Change-Id: Iea8c6d5579d6363e77e1f5af666948160c4a9bf9
    Signed-off-by: Sol Boucher <solb at chromium.org>
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Change-Id: I1a117a9473e895feaf455bb30d0f945f57de51eb
    Original-Signed-off-by: Sol Boucher <solb at chromium.org>
---
 util/cbfstool/cbfs_image.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c
index f5fddcb..828d367 100644
--- a/util/cbfstool/cbfs_image.c
+++ b/util/cbfstool/cbfs_image.c
@@ -77,13 +77,13 @@ static const struct typedesc_t types_cbfs_entry[] = {
 	{CBFS_COMPONENT_MRC_CACHE, "mrc_cache"},
 	{CBFS_COMPONENT_DELETED, "deleted"},
 	{CBFS_COMPONENT_NULL, "null"},
-	{0, NULL},
+	{0, NULL}
 };
 
 static const struct typedesc_t types_cbfs_compression[] = {
 	{CBFS_COMPRESS_NONE, "none"},
 	{CBFS_COMPRESS_LZMA, "LZMA"},
-	{0, NULL},
+	{0, NULL}
 };
 
 static const char *lookup_name_by_type(const struct typedesc_t *desc, uint32_t type,



More information about the coreboot-gerrit mailing list