[asterisk-commits] mmichelson: trunk r150255 - /trunk/apps/app_skel.c

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Thu Oct 16 17:36:08 2008
New Revision: 150255

URL: http://svn.digium.com/view/asterisk?view=rev&rev=150255
Log:
Updating app_skel.c to follow coding guidelines
with regards to braces used on if statements.

(closes issue #13696)
Reported by: alecdavis
Patches:
      app_skel.bug13696B.115850.diff.txt uploaded by alecdavis (license 585)


Modified:
    trunk/apps/app_skel.c

Modified: trunk/apps/app_skel.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_skel.c?view=diff&rev=150255&r1=150254&r2=150255
==============================================================================
--- trunk/apps/app_skel.c (original)
+++ trunk/apps/app_skel.c Thu Oct 16 17:36:08 2008
@@ -14,6 +14,9 @@
  * This program is free software, distributed under the terms of
  * the GNU General Public License Version 2. See the LICENSE file
  * at the top of the source tree.
+ *
+ * Please follow coding guidelines 
+ * http://svn.digium.com/view/asterisk/trunk/doc/CODING-GUIDELINES
  */
 
 /*! \file
@@ -89,20 +92,25 @@
 
 	AST_STANDARD_APP_ARGS(args, parse);
 
-	if (args.argc == 2)
+	if (args.argc == 2) {
 		ast_app_parse_options(app_opts, &flags, opts, args.options);
+	}
 
-	if (!ast_strlen_zero(args.dummy)) 
+	if (!ast_strlen_zero(args.dummy)) {
 		ast_log(LOG_NOTICE, "Dummy value is : %s\n", args.dummy);
+	}
 
-	if (ast_test_flag(&flags, OPTION_A))
+	if (ast_test_flag(&flags, OPTION_A)) {
 		ast_log(LOG_NOTICE, "Option A is set\n");
+	}
 
-	if (ast_test_flag(&flags, OPTION_B))
+	if (ast_test_flag(&flags, OPTION_B)) {
 		ast_log(LOG_NOTICE, "Option B is set with : %s\n", opts[OPTION_ARG_B] ? opts[OPTION_ARG_B] : "<unspecified>");
+	}
 
-	if (ast_test_flag(&flags, OPTION_C))
+	if (ast_test_flag(&flags, OPTION_C)) {
 		ast_log(LOG_NOTICE, "Option C is set with : %s\n", opts[OPTION_ARG_C] ? opts[OPTION_ARG_C] : "<unspecified>");
+	}
 
 	return res;
 }