Address Copilot review round 4: whitespace-tolerant parsing, tab test

- ParseReverseListOutput now splits on all whitespace (tabs, spaces) instead of single space
- Added ParseReverseListOutput_TabSeparated test

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: rmarinho

src/Xamarin.Android.Tools.AndroidSdk/Runners/AdbRunner.cs

@@ -343,7 +343,7 @@ internal static IReadOnlyList<AdbPortRule> ParseReverseListOutput (IEnumerable<s
 			if (!trimmed.StartsWith ("(reverse)", StringComparison.Ordinal))
 				continue;
 
-			var parts = trimmed.Substring ("(reverse)".Length).Trim ().Split (new [] { ' ' }, StringSplitOptions.RemoveEmptyEntries);
+			var parts = trimmed.Substring ("(reverse)".Length).Trim ().Split ((char[]?) null, StringSplitOptions.RemoveEmptyEntries);
 			if (parts.Length >= 2) {
 				var remote = AdbPortSpec.TryParse (parts [0]);
 				var local = AdbPortSpec.TryParse (parts [1]);

tests/Xamarin.Android.Tools.AndroidSdk-Tests/AdbRunnerTests.cs

@@ -848,6 +848,21 @@ public void ParseReverseListOutput_WindowsLineEndings ()
 		Assert.AreEqual (8081, rules [1].Remote.Port);
 	}
 
+	[Test]
+	public void ParseReverseListOutput_TabSeparated ()
+	{
+		var output = new [] {
+			"(reverse)\ttcp:5000\ttcp:5000",
+			"(reverse)\ttcp:8081\ttcp:8081",
+		};
+
+		var rules = AdbRunner.ParseReverseListOutput (output);
+
+		Assert.AreEqual (2, rules.Count);
+		Assert.AreEqual (5000, rules [0].Remote.Port);
+		Assert.AreEqual (8081, rules [1].Remote.Port);
+	}
+
 	// --- AdbPortSpec tests ---
 
 	[Test]