From 16b3bbf4599a0977f102dd4c5890c943d796f15c Mon Sep 17 00:00:00 2001
From: "sonarqube-agent[bot]"
 <210722872+sonarqube-agent[bot]@users.noreply.github.com>
Date: Sat, 23 May 2026 05:04:41 +0000
Subject: [PATCH] fix: Address 5 SonarQube issues

Fixed issues:
- AZFqcxYr7Cab0JIMjah2 for csharpsquid:S2325 rule
- AWMB0jMNmtQEh111EJCt for csharpsquid:S1168 rule
- AZC2zPMC3yATU8exn0Z2 for csharpsquid:S2589 rule
- AYqIsIc9RlgCy-_ohqpM for csharpsquid:S2589 rule
- AZFqcxBW7Cab0JIMjahj for csharpsquid:S2325 rule

Generated by SonarQube Agent (task: ef450648-78d7-4bd0-8bc9-1228593e4023)
---
 src/Integration.Vsix/ErrorList/IssuesSnapshot.cs             | 2 +-
 src/Integration.Vsix/SonarLintTagger/CancellableJobRunner.cs | 4 ++--
 src/Integration/WPF/NotifyErrorViewModelBase.cs              | 3 ++-
 src/IssueViz/Models/AnalysisIssueVisualizationConverter.cs   | 4 ++--
 4 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/Integration.Vsix/ErrorList/IssuesSnapshot.cs b/src/Integration.Vsix/ErrorList/IssuesSnapshot.cs
index 8ad2e381e2..a3424573ea 100644
--- a/src/Integration.Vsix/ErrorList/IssuesSnapshot.cs
+++ b/src/Integration.Vsix/ErrorList/IssuesSnapshot.cs
@@ -259,7 +259,7 @@ private string GetErrorCategory(IAnalysisIssue issue)
         /// Returns true/false if the ErrorList should hide the requested issue.
         /// </summary>
         /// <returns>Returns true if the issue doesn't have a valid span.</returns>
-        private bool ShouldHideIssue(IAnalysisIssueVisualization issue)
+        private static bool ShouldHideIssue(IAnalysisIssueVisualization issue)
         {
             if (issue.IsFileLevel())
             {
diff --git a/src/Integration.Vsix/SonarLintTagger/CancellableJobRunner.cs b/src/Integration.Vsix/SonarLintTagger/CancellableJobRunner.cs
index 094b22df75..9a8c32428a 100644
--- a/src/Integration.Vsix/SonarLintTagger/CancellableJobRunner.cs
+++ b/src/Integration.Vsix/SonarLintTagger/CancellableJobRunner.cs
@@ -160,7 +160,7 @@ private async System.Threading.Tasks.Task ExecuteAsync()
                 // We're cancelling here purely for testing purposes: tests can wait on the token WaitHandle
                 // to know that the job runner has finished. Ugly, but it means we can write reliable tests.
                 cancellationSource.Cancel();
-                cancellationSource?.Dispose();
+                cancellationSource.Dispose();
                 cancellationSource = null;
             }
         }
@@ -174,7 +174,7 @@ public void Cancel()
                 {
                     State = JobRunnerProgress.RunnerState.Cancelled;
                     logger.WriteLine(Strings.JobRunner_CancellingJob, jobDescription, startTime.ToLongTimeString());
-                    cancellationSource?.Cancel();
+                    cancellationSource.Cancel();
                     progress?.Report(new JobRunnerProgress(State, completedOperations, totalOperations));
                 }
             }
diff --git a/src/Integration/WPF/NotifyErrorViewModelBase.cs b/src/Integration/WPF/NotifyErrorViewModelBase.cs
index 471aa112d9..a950795402 100644
--- a/src/Integration/WPF/NotifyErrorViewModelBase.cs
+++ b/src/Integration/WPF/NotifyErrorViewModelBase.cs
@@ -20,6 +20,7 @@
 
 using System.Collections;
 using System.ComponentModel;
+using System.Linq;
 using System.Runtime.CompilerServices;
 using SonarLint.VisualStudio.Core.WPF;
 
@@ -40,7 +41,7 @@ IEnumerable INotifyDataErrorInfo.GetErrors(string propertyName)
             {
                 return new[] { error };
             }
-            return null;
+            return Enumerable.Empty<string>();
         }
 
         #endregion
diff --git a/src/IssueViz/Models/AnalysisIssueVisualizationConverter.cs b/src/IssueViz/Models/AnalysisIssueVisualizationConverter.cs
index 707114c38e..b831045aab 100644
--- a/src/IssueViz/Models/AnalysisIssueVisualizationConverter.cs
+++ b/src/IssueViz/Models/AnalysisIssueVisualizationConverter.cs
@@ -80,7 +80,7 @@ private void CalculateSpanForSecondaryLocationsInSameFile(IAnalysisIssueVisualiz
             }
         }
 
-        private IReadOnlyList<IAnalysisIssueFlowVisualization> Convert(IEnumerable<IAnalysisIssueFlow> flows)
+        private static IReadOnlyList<IAnalysisIssueFlowVisualization> Convert(IEnumerable<IAnalysisIssueFlow> flows)
         {
             if (!flows.Any())
             {
@@ -97,7 +97,7 @@ private IReadOnlyList<IAnalysisIssueFlowVisualization> Convert(IEnumerable<IAnal
             return convertedFlows;
         }
 
-        private IReadOnlyList<IAnalysisIssueLocationVisualization> Convert(IEnumerable<IAnalysisIssueLocation> locations)
+        private static IReadOnlyList<IAnalysisIssueLocationVisualization> Convert(IEnumerable<IAnalysisIssueLocation> locations)
         {
             var locationNumber = 1;