Use switch expressions to reduce nested ternaries.

Author: bgrainger

src/MySqlConnector/Protocol/Serialization/SocketByteHandler.cs

@@ -70,9 +70,12 @@ private ValueTask<int> DoReadBytesSync(Memory<byte> buffer)
 		private async ValueTask<int> DoReadBytesAsync(Memory<byte> buffer)
 		{
 			var startTime = RemainingTimeout == Constants.InfiniteTimeout ? 0 : Environment.TickCount;
-			var timerId = RemainingTimeout == Constants.InfiniteTimeout ? 0 :
-				RemainingTimeout <= 0 ? throw MySqlException.CreateForTimeout() :
-				TimerQueue.Instance.Add(RemainingTimeout, m_closeSocket);
+			var timerId = RemainingTimeout switch
+			{
+				Constants.InfiniteTimeout => 0u,
+				<= 0 => throw MySqlException.CreateForTimeout(),
+				_ => TimerQueue.Instance.Add(RemainingTimeout, m_closeSocket),
+			};
 #if NET45 || NET461 || NET471 || NETSTANDARD1_3 || NETSTANDARD2_0
 			m_socketAwaitable.EventArgs.SetBuffer(buffer);
 #endif

src/MySqlConnector/Utilities/Utility.cs

@@ -489,9 +489,13 @@ public static void SwapBytes(byte[] bytes, int offset1, int offset2)
 
 		public static void GetOSDetails(out string? os, out string osDescription, out string architecture)
 		{
-			os = Environment.OSVersion.Platform == PlatformID.Win32NT ? "Windows" :
-				Environment.OSVersion.Platform == PlatformID.Unix ? "Linux" :
-				Environment.OSVersion.Platform == PlatformID.MacOSX ? "macOS" : null;
+			os = Environment.OSVersion.Platform switch
+			{
+				PlatformID.Win32NT => "Windows",
+				PlatformID.Unix => "Linux",
+				PlatformID.MacOSX => "macOS",
+				_ => null,
+			};
 			osDescription = Environment.OSVersion.VersionString;
 			architecture = IntPtr.Size == 8 ? "X64" : "X86";
 		}