Remove Exts.WithDefault

Smittytron · Aug 7, 2023 · 388222c · 388222c
1 parent 169c608
commit 388222c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/OpenRA.Game/Exts.cs b/OpenRA.Game/Exts.cs
@@ -32,12 +32,6 @@ public static string FormatCurrent(this string format, params object[] args)
 			return string.Format(CultureInfo.CurrentCulture, format, args);
 		}
 
-		public static T WithDefault<T>(T def, Func<T> f)
-		{
-			try { return f(); }
-			catch { return def; }
-		}
-
 		public static Lazy<T> Lazy<T>(Func<T> p) { return new Lazy<T>(p); }
 
 		public static IEnumerable<string> GetNamespaces(this Assembly a)

diff --git a/OpenRA.Mods.Common/Widgets/Logic/DirectConnectLogic.cs b/OpenRA.Mods.Common/Widgets/Logic/DirectConnectLogic.cs
@@ -46,7 +46,8 @@ public DirectConnectLogic(Widget widget, Action onExit, Action openLobby, Connec
 
 			joinButton.OnClick = () =>
 			{
-				var port = Exts.WithDefault(1234, () => int.Parse(portField.Text, NumberFormatInfo.CurrentInfo));
+				if (!int.TryParse(portField.Text, NumberStyles.Integer, NumberFormatInfo.CurrentInfo, out var port))
+					port = 1234;
 
 				Game.Settings.Player.LastServer = $"{ipField.Text}:{port}";
 				Game.Settings.Save();