From 9644f1dcfe31c89fb5d1003e9932f93ca28bf6d8 Mon Sep 17 00:00:00 2001
From: Russell Bryant <russell@russellbryant.com>
Date: Wed, 25 May 2005 10:57:15 +0000
Subject: [PATCH] various code cleanups (bug #4353)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@5764 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---
 app.c                      |  2 +-
 apps/app_controlplayback.c | 15 +++++----------
 apps/app_dictate.c         |  2 +-
 apps/app_flash.c           |  6 +++---
 channels/chan_zap.c        |  4 ++--
 channels/iax2-parser.c     |  2 +-
 config.c                   |  2 +-
 7 files changed, 14 insertions(+), 19 deletions(-)

diff --git a/app.c b/app.c
index cce0cbeec3..93273b1a73 100755
--- a/app.c
+++ b/app.c
@@ -271,7 +271,7 @@ int ast_dtmf_stream(struct ast_channel *chan,struct ast_channel *peer,char *digi
 	if (!res) {
 		res = ast_waitfor(chan,100);
 		if (res > -1) {
-			for (ptr=digits;*ptr;*ptr++) {
+			for (ptr=digits; *ptr; ptr++) {
 				if (*ptr == 'w') {
 					res = ast_safe_sleep(chan, 500);
 					if (res) 
diff --git a/apps/app_controlplayback.c b/apps/app_controlplayback.c
index bf56dcaa26..02a1cd384e 100755
--- a/apps/app_controlplayback.c
+++ b/apps/app_controlplayback.c
@@ -65,24 +65,19 @@ static int controlplayback_exec(struct ast_channel *chan, void *data)
 	file = tmp;
 
 	if ((skip=strchr(tmp,'|'))) {
-		*skip = '\0';
-		*skip++;
+		*skip++ = '\0';
 		fwd=strchr(skip,'|');
 		if (fwd) {
-			*fwd = '\0';
-			*fwd++;
+			*fwd++ = '\0';
 			rev = strchr(fwd,'|');
 			if (rev) {
-				*rev = '\0';
-				*rev++;
+				*rev++ = '\0';
 				stop = strchr(rev,'|');
 				if (stop) {
-					*stop = '\0';
-					*stop++;
+					*stop++ = '\0';
 					pause = strchr(stop,'|');
 					if (pause) {
-						*pause = '\0';
-						*pause++;
+						*pause++ = '\0';
 					}
 				}
 			}
diff --git a/apps/app_dictate.c b/apps/app_dictate.c
index b88fbf2c3a..97efef19cf 100755
--- a/apps/app_dictate.c
+++ b/apps/app_dictate.c
@@ -16,7 +16,7 @@
 #include <stdlib.h>
 #include <unistd.h>
 #include <string.h>
-
+#include <sys/stat.h>	/* for mkdir */
 #include "asterisk/file.h"
 #include "asterisk/logger.h"
 #include "asterisk/channel.h"
diff --git a/apps/app_flash.c b/apps/app_flash.c
index 9c8e8dbec2..7514ef0bf7 100755
--- a/apps/app_flash.c
+++ b/apps/app_flash.c
@@ -11,6 +11,9 @@
  * the GNU General Public License
  */
  
+#include <string.h>
+#include <errno.h>
+#include <stdlib.h>
 #include "asterisk/lock.h"
 #include "asterisk/file.h"
 #include "asterisk/logger.h"
@@ -26,9 +29,6 @@
 #else
 #include <zaptel.h>
 #endif /* __linux__ */
-#include <string.h>
-#include <errno.h>
-#include <stdlib.h>
 
 static char *tdesc = "Flash zap trunk application";
 
diff --git a/channels/chan_zap.c b/channels/chan_zap.c
index 758429c698..bb3e37a90d 100755
--- a/channels/chan_zap.c
+++ b/channels/chan_zap.c
@@ -267,6 +267,8 @@ static struct ast_channel inuse = { "GR-303InUse" };
 #ifdef PRI_GETSET_TIMERS
 static int pritimers[PRI_MAX_TIMERS];
 #endif
+static int pridebugfd = -1;
+static char pridebugfilename[1024]="";
 #endif
 
 /* Wait up to 16 seconds for first digit (FXO logic) */
@@ -288,8 +290,6 @@ AST_MUTEX_DEFINE_STATIC(iflock);
 static int ifcount = 0;
 
 AST_MUTEX_DEFINE_STATIC(pridebugfdlock);
-static int pridebugfd = -1;
-static char pridebugfilename[1024]="";
 
 /* Whether we answer on a Polarity Switch event */
 static int answeronpolarityswitch = 0;
diff --git a/channels/iax2-parser.c b/channels/iax2-parser.c
index 1851b4a744..cb21253875 100755
--- a/channels/iax2-parser.c
+++ b/channels/iax2-parser.c
@@ -119,7 +119,7 @@ static void dump_datetime(char *output, int maxlen, void *value, int len)
 		tm.tm_mday = (val >> 16) & 0x1f;
 		tm.tm_mon  = ((val >> 21) & 0x0f) - 1;
 		tm.tm_year = ((val >> 25) & 0x7f) + 100;
-		strftime(output, maxlen, "%F  %T", &tm); 
+		strftime(output, maxlen, "%Y-%m-%d  %T", &tm); 
 	} else
 		ast_copy_string(output, "Invalid DATETIME format!", maxlen);
 }
diff --git a/config.c b/config.c
index 6a86b6ea19..b375669910 100755
--- a/config.c
+++ b/config.c
@@ -486,7 +486,7 @@ static int process_text_line(struct ast_config *cfg, struct ast_category **cat,
 				/* #exec </path/to/executable>
 				   We create a tmp file, then we #include it, then we delete it. */
 				if (do_exec) { 
-					snprintf(exec_file, sizeof(exec_file), "/var/tmp/exec.%ld.%ld", time(NULL), pthread_self());
+					snprintf(exec_file, sizeof(exec_file), "/var/tmp/exec.%ld.%ld", time(NULL), (long)pthread_self());
 					snprintf(cmd, sizeof(cmd), "%s > %s 2>&1", cur, exec_file);
 					ast_safe_system(cmd);
 					cur = exec_file;