From: William King <william.king@quentustech.com>
Date: Sat, 25 May 2013 14:46:47 +0000 (-0700)
Subject: Redundant code. The if/else block makes more sense than the equivalent in two separat... 
X-Git-Tag: v1.4.1~171
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=7be8184b2adbeb949137edacb4e053fa3ccc28f8;p=thirdparty%2Ffreeswitch.git

Redundant code. The if/else block makes more sense than the equivalent in two separate if's.
---

diff --git a/src/switch_apr.c b/src/switch_apr.c
index aeb10185a9..789179820e 100644
--- a/src/switch_apr.c
+++ b/src/switch_apr.c
@@ -25,6 +25,7 @@
  * 
  * Michael Jerris <mike@jerris.com>
  * Eliot Gable <egable@gmail.com>
+ * William King <william.king@quentustech.com>
  *
  * switch_apr.c -- apr wrappers and extensions
  *
@@ -597,10 +598,6 @@ SWITCH_DECLARE(const char *) switch_dir_next_file(switch_dir_t *thedir, char *bu
 			name = thedir->finfo.name;
 		}
 
-		if (!name) {
-			continue;
-		}
-
 		if (name) {
 			switch_copy_string(buf, name, len);
 			fname = buf;