diff options
author | Lukas Fleischer <calcurse@cryptocrack.de> | 2013-02-17 09:01:46 +0100 |
---|---|---|
committer | Lukas Fleischer <calcurse@cryptocrack.de> | 2013-02-17 09:19:04 +0100 |
commit | a363cb9b9111aed22d105adb0e7a8e9caab9bbe3 (patch) | |
tree | b0be8fac73932dcde5b40d6f8cee777f501d869c /src/getstring.c | |
parent | 8e16853201f8a608905cacbf1c7e4fb8ac7e568e (diff) | |
download | calcurse-a363cb9b9111aed22d105adb0e7a8e9caab9bbe3.zip |
Fix braces in if-else statements
From the Linux kernel coding guidelines:
Do not unnecessarily use braces where a single statement will do.
[...] This does not apply if one branch of a conditional statement
is a single statement. Use braces in both branches.
Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>
Diffstat (limited to 'src/getstring.c')
-rw-r--r-- | src/getstring.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/getstring.c b/src/getstring.c index 3b7224d..aedd842 100644 --- a/src/getstring.c +++ b/src/getstring.c @@ -193,8 +193,9 @@ enum getstr getstring(WINDOW * win, char *str, int l, int x, int y) if (st.pos > 0) { st.pos--; getstr_del_char(&st); - } else + } else { bell(); + } break; case CTRL('D'): /* delete next character */ if (st.pos < st.len) @@ -212,8 +213,9 @@ enum getstr getstring(WINDOW * win, char *str, int l, int x, int y) st.pos--; getstr_del_char(&st); } - } else + } else { bell(); + } break; case CTRL('K'): /* delete to end-of-line */ st.s[st.ci[st.pos].offset] = 0; |