diff options
| author | Anton Staaf <robotboy@chromium.org> | 2011-07-25 11:46:42 -0700 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2011-08-29 10:59:11 -0700 |
| commit | 0ee15331c1303bff9a830ef8cce3d81666df01eb (patch) | |
| tree | 146af906a6d8c4b2de37c63629af8d5a2838fafb /common | |
| parent | 060e61562de8a27401c72998c4f52eb52274f29d (diff) | |
assert: Rely on the compilers dead code removal
By always emitting the assert code we ensure that constructs passed
to assert will be checked by the compiler. This also ensures that
functions that are only used in asserts aren't flagged as unused
when DEBUG is not defined.
BUG=chromium-os:17999
TEST=Build firmware and boot to ChromiumOS on Aebl, Kaen and Seaboard
Change-Id: Id5f5742622d7041c2cfb7f867742701a087f3e59
Reviewed-on: http://gerrit.chromium.org/gerrit/4666
Reviewed-by: Anton Staaf <robotboy@chromium.org>
Tested-by: Anton Staaf <robotboy@chromium.org>
Diffstat (limited to 'common')
| -rw-r--r-- | common/cmd_vboot_twostop.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/cmd_vboot_twostop.c b/common/cmd_vboot_twostop.c index 593b6e5723d..e8d680556ae 100644 --- a/common/cmd_vboot_twostop.c +++ b/common/cmd_vboot_twostop.c @@ -267,7 +267,7 @@ uint32_t twostop_make_selection(struct fdt_twostop_fmap *fmap, memset(&fparams, '\0', sizeof(fparams)); vlength = fmap->readwrite_a.vblock.length; - assert(vlength == fmap->readwrite_b.vblock->vblock.length); + assert(vlength == fmap->readwrite_b.vblock.length); fparams.verification_size_A = fparams.verification_size_B = vlength; |