Use 'not null' pattern

jnm2 · Aug 12, 2023 · 6f0b108 · 6f0b108
1 parent 81a056f
commit 6f0b108
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/src/YouTubeDownloadTool/AvailableToolDownload.cs b/src/YouTubeDownloadTool/AvailableToolDownload.cs
@@ -35,7 +35,7 @@ public async Task<Stream> DownloadAsync(CancellationToken cancellationToken)
 
         var stream = await response.Content.ReadAsStreamAsync(cancellationToken);
 
-        return streamTransform is { }
+        return streamTransform is not null
             ? streamTransform.Invoke(stream)
             : stream;
     }

diff --git a/src/YouTubeDownloadTool/DownloadResolvers.cs b/src/YouTubeDownloadTool/DownloadResolvers.cs
@@ -76,7 +76,7 @@ public static Func<CancellationToken, Task<AvailableToolDownload>> GitHubRelease
 
             var version = document.RootElement.GetProperty("tag_name").GetString()!;
 
-            if (assetName is { })
+            if (assetName is not null)
             {
                 foreach (var asset in document.RootElement.GetProperty("assets").EnumerateArray())
                 {

diff --git a/src/YouTubeDownloadTool/ToolResolver.cs b/src/YouTubeDownloadTool/ToolResolver.cs
@@ -108,7 +108,7 @@ private async Task<LeaseSource> ResolveLatestToolAsync()
     {
         using var download = await getLatestDownloadAsync.Invoke(CancellationToken.None).ConfigureAwait(false);
 
-        if (currentSource is { } && string.Equals(download.Version, currentSource.Version, StringComparison.OrdinalIgnoreCase))
+        if (currentSource is not null && string.Equals(download.Version, currentSource.Version, StringComparison.OrdinalIgnoreCase))
             return currentSource;
 
         var fileLock = await Utils.GetOrDownloadFileAsync(

diff --git a/src/YouTubeDownloadTool/Utils/TaskDeduplicator.cs b/src/YouTubeDownloadTool/Utils/TaskDeduplicator.cs
@@ -31,7 +31,7 @@ public Task<T> StartOrJoin()
             lastSeenTask = Interlocked.CompareExchange(ref currentTask, source.Task, taskToReplace);
             if (lastSeenTask == taskToReplace) break;
 
-            if (lastSeenTask is { })
+            if (lastSeenTask is not null)
             {
                 // Return even if the new task is complete because it’s so close to having both just started and
                 // just finished that it’s just as good. There’s no way to avoid returning a task that was not