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

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;