Thomas Heijligen has uploaded this change for review.

View Change

programmer_table: remove NULL termination

The {0} Object at the end of programmer_table which correspond
to PROGRAMMER_INVALID has no use in currend code

Change-Id: Ib63c2d2941f23a0788e26e5a5feb25d8669acb42
Signed-off-by: Thomas Heijligen <thomas.heijligen@secunet.de>
---
M flashrom.c
1 file changed, 1 insertion(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/15/55115/1
diff --git a/flashrom.c b/flashrom.c
index 0af5057..65f7e73 100644
--- a/flashrom.c
+++ b/flashrom.c
@@ -532,8 +532,6 @@
.delay = internal_delay,
},
#endif
-
- {0}, /* This entry corresponds to PROGRAMMER_INVALID. */
};

#define SHUTDOWN_MAXFN 32
@@ -2191,7 +2189,7 @@
/* Safety check. Instead of aborting after the first error, check
* if more errors exist.
*/
- if (ARRAY_SIZE(programmer_table) - 1 != PROGRAMMER_INVALID) {
+ if (ARRAY_SIZE(programmer_table) != PROGRAMMER_INVALID) {
msg_gerr("Programmer table miscompilation!\n");
ret = 1;
}

To view, visit change 55115. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ib63c2d2941f23a0788e26e5a5feb25d8669acb42
Gerrit-Change-Number: 55115
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Heijligen <src@posteo.de>
Gerrit-MessageType: newchange