diff options
| author | Randall Spangler <rspangler@chromium.org> | 2013-12-17 13:43:17 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-12-19 00:12:28 +0000 |
| commit | 6ab8e91658f1efc894b648cc0748af8d804915e4 (patch) | |
| tree | c33da7914793452ecb37084e505230171d4eff9e /common/util.c | |
| parent | e5935f17d1798a1f19c6003e57f140446774484f (diff) | |
| download | chrome-ec-6ab8e91658f1efc894b648cc0748af8d804915e4.tar.gz | |
cleanup: Remove checkpatch warnings
This make minor syntactic changes and renames some camel-cased symbols
to keep checkpatch from complaining. The goal is to reduce the
temptation to use 'repo upload --no-verify'.
This is a big furball of find/replace, but no functional changes.
BUG=chromium:322144
BRANCH=none
TEST=build all boards; pass unit tests
Change-Id: I0269b7dd95836ef9a6e33f88c003ab0f24f842a0
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/180495
Diffstat (limited to 'common/util.c')
| -rw-r--r-- | common/util.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/common/util.c b/common/util.c index 46e11372ed..32439edaaa 100644 --- a/common/util.c +++ b/common/util.c @@ -80,7 +80,8 @@ int atoi(const char *nptr) int neg = 0; char c = '\0'; - while ((c = *nptr++) && isspace(c)) {} + while ((c = *nptr++) && isspace(c)) + ; if (c == '-') { neg = 1; @@ -106,7 +107,8 @@ int strtoi(const char *nptr, char **endptr, int base) if (endptr) *endptr = (char *)nptr; - while((c = *nptr++) && isspace(c)) {} + while ((c = *nptr++) && isspace(c)) + ; if (c == '0' && *nptr == 'x') { base = 16; @@ -336,8 +338,7 @@ int cond_went(cond_t *c, int val) if (val) { ret = *c & COND_RISE_MASK; *c &= ~COND_RISE_MASK; - } - else { + } else { ret = *c & COND_FALL_MASK; *c &= ~COND_FALL_MASK; } |