Fix PlatformNotSupportedException thrown on Android in ConsoleTitler (#…

…2390) * Remove Immutable packages reference from Forms sample * Catch PlatformNotSupportedException when checking if console output is redirected in ConsoleTitler
dotnet · Aug 2, 2023 · fd5b766 · fd5b766
1 parent 914ee71
commit fd5b766
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/samples/BenchmarkDotNet.Samples.Forms/BenchmarkDotNet.Samples.Forms.csproj b/samples/BenchmarkDotNet.Samples.Forms/BenchmarkDotNet.Samples.Forms.csproj
@@ -10,7 +10,6 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="System.Collections.Immutable" Version="1.7.0" />
     <PackageReference Include="Xamarin.Forms" Version="4.5.0.530" />
     <PackageReference Include="Xamarin.Essentials" Version="1.5.2" />
   </ItemGroup>

diff --git a/src/BenchmarkDotNet/Running/ConsoleTitler.cs b/src/BenchmarkDotNet/Running/ConsoleTitler.cs
@@ -20,10 +20,17 @@ internal class ConsoleTitler : IDisposable
 
         public ConsoleTitler(string initialTitle)
         {
-            // Return without enabling if Console output is redirected.
-            if (Console.IsOutputRedirected)
+            try
+            {
+                // Return without enabling if Console output is redirected.
+                if (Console.IsOutputRedirected)
+                {
+                    return;
+                }
+            }
+            catch (PlatformNotSupportedException)
             {
-                return;
+                // Ignore the exception. Some platforms do not support Console.IsOutputRedirected.
             }
 
             try