From ee93e988bc84075962a8fee5cdcbb26c866c7c27 Mon Sep 17 00:00:00 2001
From: adelikat <adelikat@tasvideos.org>
Date: Mon, 21 Jul 2014 18:14:27 +0000
Subject: [PATCH] a small and pointless code refactor in the record movie
 dialog

---
 BizHawk.Client.EmuHawk/movie/RecordMovie.cs | 28 ++++++++++-----------
 1 file changed, 13 insertions(+), 15 deletions(-)

diff --git a/BizHawk.Client.EmuHawk/movie/RecordMovie.cs b/BizHawk.Client.EmuHawk/movie/RecordMovie.cs
index dc012dfcec..0f7d35cbc3 100644
--- a/BizHawk.Client.EmuHawk/movie/RecordMovie.cs
+++ b/BizHawk.Client.EmuHawk/movie/RecordMovie.cs
@@ -11,8 +11,7 @@ namespace BizHawk.Client.EmuHawk
 {
 	public partial class RecordMovie : Form
 	{
-		// TODO
-		// Allow relative paths in record textbox
+		// TODO - Allow relative paths in record textbox
 		public RecordMovie()
 		{
 			InitializeComponent();
@@ -20,24 +19,23 @@ namespace BizHawk.Client.EmuHawk
 
 		private string MakePath()
 		{
-			if (RecordBox.Text.Length == 0)
-			{
-				return string.Empty;
-			}
-
 			var path = RecordBox.Text;
-			if (path.LastIndexOf(Path.DirectorySeparatorChar) == -1)
+
+			if (!string.IsNullOrWhiteSpace(path))
 			{
-				if (path[0] != Path.DirectorySeparatorChar)
+				if (path.LastIndexOf(Path.DirectorySeparatorChar) == -1)
 				{
-					path = path.Insert(0, Path.DirectorySeparatorChar.ToString());
-				}
+					if (path[0] != Path.DirectorySeparatorChar)
+					{
+						path = path.Insert(0, Path.DirectorySeparatorChar.ToString());
+					}
 
-				path = PathManager.MakeAbsolutePath(Global.Config.PathEntries.MoviesPathFragment, null) + path;
+					path = PathManager.MakeAbsolutePath(Global.Config.PathEntries.MoviesPathFragment, null) + path;
 
-				if (path[path.Length - 4] != '.') // If no file extension, add movie extension
-				{
-					path += "." + Global.MovieSession.Movie.PreferredExtension;
+					if (path[path.Length - 4] != '.') // If no file extension, add movie extension
+					{
+						path += "." + Global.MovieSession.Movie.PreferredExtension;
+					}
 				}
 			}