From a363cb9b9111aed22d105adb0e7a8e9caab9bbe3 Mon Sep 17 00:00:00 2001
From: Lukas Fleischer <calcurse@cryptocrack.de>
Date: Sun, 17 Feb 2013 09:01:46 +0100
Subject: 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>
---
 src/notify.c | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

(limited to 'src/notify.c')

diff --git a/src/notify.c b/src/notify.c
index cb56e8c..36aad5a 100644
--- a/src/notify.c
+++ b/src/notify.c
@@ -140,9 +140,9 @@ static void extract_aptsfile(void)
   char *file;
 
   file = strrchr(path_apts, '/');
-  if (!file)
+  if (!file) {
     notify.apts_file = path_apts;
-  else {
+  } else {
     notify.apts_file = file;
     notify.apts_file++;
   }
@@ -455,8 +455,9 @@ void notify_check_added(char *mesg, long start, char state)
       update_notify = 1;
   } else if (start < notify_app.time && start >= current_time) {
     update_notify = 1;
-  } else if (start == notify_app.time && state != notify_app.state)
+  } else if (start == notify_app.time && state != notify_app.state) {
     update_notify = 1;
+  }
 
   if (update_notify) {
     notify_update_app(start, state, mesg);
@@ -482,8 +483,9 @@ void notify_check_repeated(struct recur_apoint *i)
         update_notify = 1;
     } else if (real_app_time < notify_app.time && real_app_time >= current_time) {
       update_notify = 1;
-    } else if (real_app_time == notify_app.time && i->state != notify_app.state)
+    } else if (real_app_time == notify_app.time && i->state != notify_app.state) {
       update_notify = 1;
+    }
   }
   if (update_notify) {
     notify_update_app(real_app_time, i->state, i->mesg);
@@ -548,9 +550,9 @@ print_option(WINDOW * win, unsigned x, unsigned y, char *name,
 
     maxlen = MAXCOL - x_opt - 2;
     custom_apply_attr(win, ATTR_HIGHEST);
-    if (len < maxlen)
+    if (len < maxlen) {
       mvwaddstr(win, y, x_opt, valstr);
-    else {
+    } else {
       char buf[BUFSIZ];
 
       strncpy(buf, valstr, maxlen - 1);
@@ -558,8 +560,9 @@ print_option(WINDOW * win, unsigned x, unsigned y, char *name,
       mvwprintw(win, y, x_opt, "%s...", buf);
     }
     custom_remove_attr(win, ATTR_HIGHEST);
-  } else
+  } else {
     print_bool_option_incolor(win, valbool, y, x_opt);
+  }
   mvwaddstr(win, y + 1, x, desc);
 }
 
-- 
cgit v1.2.3-54-g00ecf