Minor restructuring/clarification to get_backup_name.

(Tweaked by Wayne to follow his preferred style.)
RsyncProject · Apr 24, 2010 · 0b8a9bd · 0b8a9bd
1 parent 7a3ce97
commit 0b8a9bd
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/backup.c b/backup.c
@@ -160,13 +160,13 @@ char *get_backup_name(const char *fname)
 		/* copy fname into backup_dir_buf while validating the dirs. */
 		if (copy_valid_path(fname))
 			return backup_dir_buf;
+		/* copy_valid_path() has printed an error message. */
 		return NULL;
-	} else {
-		if (stringjoin(backup_dir_buf, MAXPATHLEN,
-			       fname, backup_suffix, NULL) < MAXPATHLEN)
-			return backup_dir_buf;
 	}
 
+	if (stringjoin(backup_dir_buf, MAXPATHLEN, fname, backup_suffix, NULL) < MAXPATHLEN)
+		return backup_dir_buf;
+
 	rprintf(FERROR, "backup filename too long\n");
 	return NULL;
 }