Fix CA1822 - iteration 1

dotnet · Jan 18, 2021 · 3c4382c · 3c4382c
1 parent 96b0f6d
commit 3c4382c
Show file tree

Hide file tree

Showing 48 changed files with 78 additions and 78 deletions.
diff --git a/src/VisualStudio/Core/Def/Implementation/AnalyzerDependency/AnalyzerFileWatcherService.cs b/src/VisualStudio/Core/Def/Implementation/AnalyzerDependency/AnalyzerFileWatcherService.cs
@@ -70,7 +70,7 @@ private void RaiseAnalyzerChangedWarning(ProjectId projectId, string analyzerPat
             }
         }
 
-        private DateTime? GetLastUpdateTimeUtc(string fullPath)
+        private static DateTime? GetLastUpdateTimeUtc(string fullPath)
         {
             try
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/CallHierarchy/CallHierarchyDetail.cs b/src/VisualStudio/Core/Def/Implementation/CallHierarchy/CallHierarchyDetail.cs
@@ -35,7 +35,7 @@ public CallHierarchyDetail(Location location, Workspace workspace)
             _text = ComputeText(location);
         }
 
-        private string ComputeText(Location location)
+        private static string ComputeText(Location location)
         {
             var lineSpan = location.GetLineSpan();
             var start = location.SourceTree.GetText().Lines[lineSpan.StartLinePosition.Line].Start;

diff --git a/src/VisualStudio/Core/Def/Implementation/CallHierarchy/CallHierarchyProvider.cs b/src/VisualStudio/Core/Def/Implementation/CallHierarchy/CallHierarchyProvider.cs
@@ -66,7 +66,7 @@ public async Task<ICallHierarchyMemberItem> CreateItemAsync(ISymbol symbol,
             return null;
         }
 
-        private ISymbol GetTargetSymbol(ISymbol symbol)
+        private static ISymbol GetTargetSymbol(ISymbol symbol)
         {
             if (symbol is IMethodSymbol methodSymbol)
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/CallHierarchy/Finders/AbstractCallFinder.cs b/src/VisualStudio/Core/Def/Implementation/CallHierarchy/Finders/AbstractCallFinder.cs
@@ -110,7 +110,7 @@ private async Task SearchAsync(Workspace workspace, CallHierarchySearchScope sco
             await SearchWorkerAsync(symbol, project, callback, documents, cancellationToken).ConfigureAwait(false);
         }
 
-        private IImmutableSet<Document> IncludeDocuments(CallHierarchySearchScope scope, Project project)
+        private static IImmutableSet<Document> IncludeDocuments(CallHierarchySearchScope scope, Project project)
         {
             if (scope == CallHierarchySearchScope.CurrentDocument || scope == CallHierarchySearchScope.CurrentProject)
             {

diff --git a/.../Def/Implementation/ChangeSignature/ChangeSignatureDialogViewModel.ParameterViewModels.cs b/.../Def/Implementation/ChangeSignature/ChangeSignatureDialogViewModel.ParameterViewModels.cs
@@ -40,7 +40,7 @@ public ParameterViewModel(ChangeSignatureDialogViewModel changeSignatureDialogVi
                 ChangeSignatureDialogViewModel = changeSignatureDialogViewModel;
             }
 
-            private string ValueOrNone(string value)
+            private static string ValueOrNone(string value)
             {
                 return !string.IsNullOrEmpty(value)
                     ? value

diff --git a/src/VisualStudio/Core/Def/Implementation/ColorSchemes/ColorSchemeApplier.RegistryItem.cs b/src/VisualStudio/Core/Def/Implementation/ColorSchemes/ColorSchemeApplier.RegistryItem.cs
@@ -9,7 +9,7 @@ internal partial class ColorSchemeApplier
         private class RegistryItem
         {
             public string SectionName { get; }
-            public string ValueName => "Data";
+            public static string ValueName => "Data";
             public byte[] ValueData { get; }
 
             public RegistryItem(string sectionName, byte[] valueData)

diff --git a/src/VisualStudio/Core/Def/Implementation/ColorSchemes/ColorSchemeApplier.Settings.cs b/src/VisualStudio/Core/Def/Implementation/ColorSchemes/ColorSchemeApplier.Settings.cs
@@ -47,7 +47,7 @@ private ColorScheme GetColorScheme(SchemeName schemeName)
                 return ColorSchemeReader.ReadColorScheme(colorSchemeStream);
             }
 
-            private Stream GetColorSchemeXmlStream(SchemeName schemeName)
+            private static Stream GetColorSchemeXmlStream(SchemeName schemeName)
             {
                 var assembly = Assembly.GetExecutingAssembly();
                 return assembly.GetManifestResourceStream($"Microsoft.VisualStudio.LanguageServices.ColorSchemes.{schemeName}.xml");
@@ -60,7 +60,7 @@ public void ApplyColorScheme(SchemeName schemeName, ImmutableArray<RegistryItem>
                 foreach (var item in registryItems)
                 {
                     using var itemKey = registryRoot.CreateSubKey(item.SectionName);
-                    itemKey.SetValue(item.ValueName, item.ValueData);
+                    itemKey.SetValue(RegistryItem.ValueName, item.ValueData);
                     // Flush RegistryKeys out of paranoia
                     itemKey.Flush();
                 }

diff --git a/src/VisualStudio/Core/Def/Implementation/CommonControls/MemberSelectionViewModel.cs b/src/VisualStudio/Core/Def/Implementation/CommonControls/MemberSelectionViewModel.cs
@@ -142,7 +142,7 @@ public void UpdateMembersBasedOnDestinationKind(TypeKind destinationType)
             }
         }
 
-        private void SelectMembers(ImmutableArray<PullMemberUpSymbolViewModel> members, bool isChecked = true)
+        private static void SelectMembers(ImmutableArray<PullMemberUpSymbolViewModel> members, bool isChecked = true)
         {
             foreach (var member in members.Where(viewModel => viewModel.IsCheckable))
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/ContainedLanguageRefactorNotifyService.cs b/src/VisualStudio/Core/Def/Implementation/ContainedLanguageRefactorNotifyService.cs
@@ -68,7 +68,7 @@ public bool TryOnAfterGlobalSymbolRenamed(Workspace workspace, IEnumerable<Docum
             return true;
         }
 
-        private ContainedLanguageRenameType GetRenameType(ISymbol symbol)
+        private static ContainedLanguageRenameType GetRenameType(ISymbol symbol)
         {
             if (symbol is INamespaceSymbol)
             {

diff --git a/...tudio/Core/Def/Implementation/DebuggerIntelliSense/AbstractDebuggerIntelliSenseContext.cs b/...tudio/Core/Def/Implementation/DebuggerIntelliSense/AbstractDebuggerIntelliSenseContext.cs
@@ -254,7 +254,7 @@ private void SetupImmediateWindowProjectionBuffer()
             }
         }
 
-        private int GetQuestionIndex(string text)
+        private static int GetQuestionIndex(string text)
         {
             for (var i = 0; i < text.Length; i++)
             {
@@ -269,7 +269,7 @@ private int GetQuestionIndex(string text)
             return -1;
         }
 
-        private bool IsImmediateWindow(IVsUIShell shellService, IVsTextView textView)
+        private static bool IsImmediateWindow(IVsUIShell shellService, IVsTextView textView)
         {
             Marshal.ThrowExceptionForHR(shellService.GetToolWindowEnum(out var windowEnum));
             Marshal.ThrowExceptionForHR(textView.GetBuffer(out _));

diff --git a/...lStudio/Core/Def/Implementation/DesignerAttribute/VisualStudioDesignerAttributeService.cs b/...lStudio/Core/Def/Implementation/DesignerAttribute/VisualStudioDesignerAttributeService.cs
@@ -163,7 +163,7 @@ private async Task NotifyProjectSystemAsync(
             await Task.WhenAll(tasks).ConfigureAwait(false);
         }
 
-        private void AddFilteredInfos(ImmutableArray<DesignerAttributeData> data, ArrayBuilder<DesignerAttributeData> filteredData)
+        private static void AddFilteredInfos(ImmutableArray<DesignerAttributeData> data, ArrayBuilder<DesignerAttributeData> filteredData)
         {
             using var _ = PooledHashSet<DocumentId>.GetInstance(out var seenDocumentIds);
 
@@ -279,7 +279,7 @@ private async Task NotifyCpsProjectSystemAsync(
             await Task.WhenAll(tasks).ConfigureAwait(false);
         }
 
-        private async Task NotifyCpsProjectSystemAsync(
+        private static async Task NotifyCpsProjectSystemAsync(
             IProjectItemDesignerTypeUpdateService updateService,
             DesignerAttributeData data,
             CancellationToken cancellationToken)

diff --git a/...VisualStudio/Core/Def/Implementation/Diagnostics/VisualStudioDiagnosticAnalyzerService.cs b/...VisualStudio/Core/Def/Implementation/Diagnostics/VisualStudioDiagnosticAnalyzerService.cs
@@ -142,7 +142,7 @@ public IReadOnlyDictionary<string, IEnumerable<DiagnosticDescriptor>> GetAllDiag
             }
         }
 
-        private IReadOnlyDictionary<string, IEnumerable<DiagnosticDescriptor>> Transform(
+        private static IReadOnlyDictionary<string, IEnumerable<DiagnosticDescriptor>> Transform(
             ImmutableDictionary<string, ImmutableArray<DiagnosticDescriptor>> map)
         {
             // unfortunately, we had to do this since ruleset editor and us are set to use this signature

diff --git a/src/VisualStudio/Core/Def/Implementation/Diagnostics/VisualStudioDiagnosticModeService.cs b/src/VisualStudio/Core/Def/Implementation/Diagnostics/VisualStudioDiagnosticModeService.cs
@@ -69,7 +69,7 @@ private DiagnosticMode ComputeDiagnosticMode(Option2<DiagnosticMode> option)
                 return _workspace.Options.GetOption(option);
             }
 
-            private bool IsInCodeSpacesServer()
+            private static bool IsInCodeSpacesServer()
             {
                 // hack until there is an officially supported free-threaded synchronous platform API to ask this question.
                 return Environment.GetEnvironmentVariable("VisualStudioServerMode") == "1";

diff --git a/src/VisualStudio/Core/Def/Implementation/Diagnostics/VisualStudioVenusSpanMappingService.cs b/src/VisualStudio/Core/Def/Implementation/Diagnostics/VisualStudioVenusSpanMappingService.cs
@@ -120,7 +120,7 @@ private bool TryAdjustSpanIfNeededForVenus(
             return startChanged || endChanged;
         }
 
-        private LinePositionSpan GetLinePositionSpan(LinePosition position1, LinePosition position2)
+        private static LinePositionSpan GetLinePositionSpan(LinePosition position1, LinePosition position2)
         {
             if (position1 <= position2)
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/FindReferences/Entries/DocumentSpanEntry.cs b/src/VisualStudio/Core/Def/Implementation/FindReferences/Entries/DocumentSpanEntry.cs
@@ -168,7 +168,7 @@ private DisposableToolTip CreateDisposableToolTip(Document document, TextSpan so
                 return controlService.CreateDisposableToolTip(document, textBuffer, contentSpan, EnvironmentColors.ToolWindowBackgroundBrushKey);
             }
 
-            private void SetStaticClassifications(ITextBuffer textBuffer, ImmutableArray<ClassifiedSpan> classifiedSpans)
+            private static void SetStaticClassifications(ITextBuffer textBuffer, ImmutableArray<ClassifiedSpan> classifiedSpans)
             {
                 var key = PredefinedPreviewTaggerKeys.StaticClassificationSpansKey;
                 textBuffer.Properties.RemoveProperty(key);

diff --git a/src/VisualStudio/Core/Def/Implementation/GenerateType/GenerateTypeDialogViewModel.cs b/src/VisualStudio/Core/Def/Implementation/GenerateType/GenerateTypeDialogViewModel.cs
@@ -692,7 +692,7 @@ internal void UpdateFileNameExtension()
             this.FileName = currentFileName;
         }
 
-        private string UpdateExtension(string currentFileName, string desiredFileExtension, string undesiredFileExtension)
+        private static string UpdateExtension(string currentFileName, string desiredFileExtension, string undesiredFileExtension)
         {
             if (currentFileName.EndsWith(desiredFileExtension, StringComparison.OrdinalIgnoreCase))
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/InlineRename/InlineRenameUndoManager.cs b/src/VisualStudio/Core/Def/Implementation/InlineRename/InlineRenameUndoManager.cs
@@ -218,7 +218,7 @@ private IOleUndoManager GetUndoManager(ITextBuffer subjectBuffer)
                 return null;
             }
 
-            private IEnumerable<IOleUndoUnit> GetUndoUnits(IOleUndoManager undoManager)
+            private static IEnumerable<IOleUndoUnit> GetUndoUnits(IOleUndoManager undoManager)
             {
                 IEnumOleUndoUnits undoUnitEnumerator;
                 try

diff --git a/src/VisualStudio/Core/Def/Implementation/LanguageService/AbstractLanguageService`2.cs b/src/VisualStudio/Core/Def/Implementation/LanguageService/AbstractLanguageService`2.cs
@@ -281,7 +281,7 @@ private bool ContainsRegionTag(ITextSnapshot textSnapshot)
             return false;
         }
 
-        private bool StartsWithRegionTag(ITextSnapshotLine line)
+        private static bool StartsWithRegionTag(ITextSnapshotLine line)
         {
             var start = line.GetFirstNonWhitespacePosition();
             if (start != null)

diff --git a/src/VisualStudio/Core/Def/Implementation/Library/ObjectBrowser/AbstractDescriptionBuilder.cs b/src/VisualStudio/Core/Def/Implementation/Library/ObjectBrowser/AbstractDescriptionBuilder.cs
@@ -443,7 +443,7 @@ private void BuildXmlDocumentation(ISymbol symbol, Compilation compilation)
             }
         }
 
-        private bool ShowReturnsDocumentation(ISymbol symbol)
+        private static bool ShowReturnsDocumentation(ISymbol symbol)
         {
             return (symbol.Kind == SymbolKind.NamedType && ((INamedTypeSymbol)symbol).TypeKind == TypeKind.Delegate)
                 || symbol.Kind == SymbolKind.Method

diff --git a/src/VisualStudio/Core/Def/Implementation/Library/ObjectBrowser/AbstractListItemFactory.cs b/src/VisualStudio/Core/Def/Implementation/Library/ObjectBrowser/AbstractListItemFactory.cs
@@ -164,7 +164,7 @@ private static bool IncludeMemberSymbol(ISymbol symbol, IAssemblySymbol assembly
             return false;
         }
 
-        private ImmutableArray<ObjectListItem> CreateListItemsFromSymbols<TSymbol>(
+        private static ImmutableArray<ObjectListItem> CreateListItemsFromSymbols<TSymbol>(
             ImmutableArray<TSymbol> symbols,
             Compilation compilation,
             ProjectId projectId,
@@ -313,7 +313,7 @@ private ImmutableArray<ObjectListItem> GetMemberListItems(
             return builder.ToImmutable();
         }
 
-        private ImmutableArray<ISymbol> GetMemberSymbols(INamedTypeSymbol namedTypeSymbol, Compilation compilation)
+        private static ImmutableArray<ISymbol> GetMemberSymbols(INamedTypeSymbol namedTypeSymbol, Compilation compilation)
         {
             var members = namedTypeSymbol.GetMembers();
             var symbolBuilder = ImmutableArray.CreateBuilder<ISymbol>(members.Length);
@@ -373,7 +373,7 @@ private ImmutableArray<ISymbol> GetInheritedMemberSymbols(INamedTypeSymbol named
             return symbolBuilder.ToImmutable();
         }
 
-        private void AddOverriddenMembers(INamedTypeSymbol namedTypeSymbol, ref HashSet<ISymbol> overriddenMembers)
+        private static void AddOverriddenMembers(INamedTypeSymbol namedTypeSymbol, ref HashSet<ISymbol> overriddenMembers)
         {
             foreach (var member in namedTypeSymbol.GetMembers())
             {
@@ -539,7 +539,7 @@ public ImmutableHashSet<Tuple<ProjectId, IAssemblySymbol>> GetAssemblySet(Projec
             return set.ToImmutable();
         }
 
-        private bool ContainsAccessibleTypeMember(INamespaceOrTypeSymbol namespaceOrTypeSymbol, IAssemblySymbol assemblySymbol)
+        private static bool ContainsAccessibleTypeMember(INamespaceOrTypeSymbol namespaceOrTypeSymbol, IAssemblySymbol assemblySymbol)
         {
             foreach (var typeMember in namespaceOrTypeSymbol.GetTypeMembers())
             {
@@ -568,7 +568,7 @@ private ImmutableArray<INamedTypeSymbol> GetAccessibleTypeMembers(INamespaceOrTy
             return builder.ToImmutable();
         }
 
-        private bool IncludeTypeMember(INamedTypeSymbol typeMember, IAssemblySymbol assemblySymbol)
+        private static bool IncludeTypeMember(INamedTypeSymbol typeMember, IAssemblySymbol assemblySymbol)
         {
             if (!IncludeSymbol(typeMember))
             {

diff --git a/...udio/Core/Def/Implementation/Library/ObjectBrowser/AbstractObjectBrowserLibraryManager.cs b/...udio/Core/Def/Implementation/Library/ObjectBrowser/AbstractObjectBrowserLibraryManager.cs
@@ -505,7 +505,7 @@ protected override bool TryExec(Guid commandGroup, uint commandId)
             return false;
         }
 
-        private async Task FindReferencesAsync(
+        private static async Task FindReferencesAsync(
             IStreamingFindUsagesPresenter presenter, SymbolListItem symbolListItem, Project project)
         {
             try

diff --git a/src/VisualStudio/Core/Def/Implementation/Log/VisualStudioErrorLogger.cs b/src/VisualStudio/Core/Def/Implementation/Log/VisualStudioErrorLogger.cs
@@ -34,7 +34,7 @@ public void LogException(object source, Exception exception)
             }
         }
 
-        private bool ShouldReportCrashDumps(object source) => HasRoslynPublicKey(source);
+        private static bool ShouldReportCrashDumps(object source) => HasRoslynPublicKey(source);
 
         private static string ToLogFormat(Exception exception)
             => exception.Message + Environment.NewLine + exception.StackTrace;

diff --git a/...VisualStudio/Core/Def/Implementation/Options/RoamingVisualStudioProfileOptionPersister.cs b/...VisualStudio/Core/Def/Implementation/Options/RoamingVisualStudioProfileOptionPersister.cs
@@ -215,7 +215,7 @@ public bool TryFetch(OptionKey optionKey, out object value)
             return true;
         }
 
-        private bool DeserializeCodeStyleOption(ref object value, Type type)
+        private static bool DeserializeCodeStyleOption(ref object value, Type type)
         {
             if (value is string serializedValue)
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/Preview/FileChange.cs b/src/VisualStudio/Core/Def/Implementation/Preview/FileChange.cs
@@ -128,7 +128,7 @@ private ChangeList GetEntireDocumentAsSpanChange(TextDocument document)
             return new ChangeList(new[] { entireSpanChild });
         }
 
-        private string GetDisplayText(string excerpt)
+        private static string GetDisplayText(string excerpt)
         {
             if (excerpt.Contains("\r\n"))
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/Preview/TopLevelChange.cs b/src/VisualStudio/Core/Def/Implementation/Preview/TopLevelChange.cs
@@ -68,7 +68,7 @@ public Solution GetUpdatedSolution(bool applyingChanges)
             return solution;
         }
 
-        private Solution ApplyFileChanges(Solution solution, IEnumerable<FileChange> fileChanges, bool applyingChanges)
+        private static Solution ApplyFileChanges(Solution solution, IEnumerable<FileChange> fileChanges, bool applyingChanges)
         {
             foreach (var fileChange in fileChanges)
             {
@@ -179,7 +179,7 @@ void ApplyFileChangesCore(
             }
         }
 
-        private Solution ApplyReferenceChanges(Solution solution, IEnumerable<ReferenceChange> referenceChanges)
+        private static Solution ApplyReferenceChanges(Solution solution, IEnumerable<ReferenceChange> referenceChanges)
         {
             foreach (var referenceChange in referenceChanges)
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/PreviewPane/PreviewPaneService.cs b/src/VisualStudio/Core/Def/Implementation/PreviewPane/PreviewPaneService.cs
@@ -113,7 +113,7 @@ object IPreviewPaneService.GetPreviewPane(DiagnosticData data, IReadOnlyList<obj
                 optionPageGuid: optionPageGuid);
         }
 
-        private Guid GetOptionPageGuidForOptionName(string optionName, string optionLanguage)
+        private static Guid GetOptionPageGuidForOptionName(string optionName, string optionLanguage)
         {
             if (optionName == nameof(NamingStyleOptions.NamingPreferences))
             {

diff --git a/src/VisualStudio/Core/Def/Implementation/Progression/GraphFormattedLabelExtension.cs b/src/VisualStudio/Core/Def/Implementation/Progression/GraphFormattedLabelExtension.cs
@@ -35,7 +35,7 @@ public string Label(GraphObject graphObject, string graphCommandDefinitionIdenti
                 RoslynGraphProperties.FormattedLabelWithContainingSymbol);
         }
 
-        private string GetStringPropertyForGraphObject(GraphObject graphObject, string graphCommandDefinitionIdentifier, GraphProperty propertyWithoutContainingSymbol, GraphProperty propertyWithContainingSymbol)
+        private static string GetStringPropertyForGraphObject(GraphObject graphObject, string graphCommandDefinitionIdentifier, GraphProperty propertyWithoutContainingSymbol, GraphProperty propertyWithContainingSymbol)
         {
 
             if (graphObject is GraphNode graphNode)

diff --git a/src/VisualStudio/Core/Def/Implementation/Progression/GraphProvider.cs b/src/VisualStudio/Core/Def/Implementation/Progression/GraphProvider.cs
@@ -307,25 +307,25 @@ private bool IsOverridable(GraphNode node)
                 !modifiers.IsSealed;
         }
 
-        private DeclarationModifiers GetModifiers(GraphNode node)
+        private static DeclarationModifiers GetModifiers(GraphNode node)
             => (DeclarationModifiers)node[RoslynGraphProperties.SymbolModifiers];
 
-        private bool CheckAccessibility(GraphNode node, Accessibility accessibility)
+        private static bool CheckAccessibility(GraphNode node, Accessibility accessibility)
             => node[RoslynGraphProperties.DeclaredAccessibility].Equals(accessibility);
 
-        private bool HasExplicitInterfaces(GraphNode node)
+        private static bool HasExplicitInterfaces(GraphNode node)
             => ((IList<SymbolKey>)node[RoslynGraphProperties.ExplicitInterfaceImplementations]).Count > 0;
 
-        private bool IsRoslynNode(GraphNode node)
+        private static bool IsRoslynNode(GraphNode node)
         {
             return node[RoslynGraphProperties.SymbolKind] != null
                 && node[RoslynGraphProperties.TypeKind] != null;
         }
 
-        private bool IsAnySymbolKind(GraphNode node, params SymbolKind[] symbolKinds)
+        private static bool IsAnySymbolKind(GraphNode node, params SymbolKind[] symbolKinds)
             => symbolKinds.Any(k => k.Equals(node[RoslynGraphProperties.SymbolKind]));
 
-        private bool IsAnyTypeKind(GraphNode node, params TypeKind[] typeKinds)
+        private static bool IsAnyTypeKind(GraphNode node, params TypeKind[] typeKinds)
             => typeKinds.Any(k => node[RoslynGraphProperties.TypeKind].Equals(k));
 
         private static readonly GraphCommandDefinition s_overridesCommandDefinition =

diff --git a/src/VisualStudio/Core/Def/Implementation/ProjectSystem/FileChangeWatcher.cs b/src/VisualStudio/Core/Def/Implementation/ProjectSystem/FileChangeWatcher.cs
@@ -281,7 +281,7 @@ public void StopWatchingFile(IFileWatchingToken token)
                 _fileChangeWatcher.EnqueueWork(service => UnsubscribeFileChangeEventsAsync(service, typedToken));
             }
 
-            private Task UnsubscribeFileChangeEventsAsync(IVsAsyncFileChangeEx service, FileWatchingToken typedToken)
+            private static Task UnsubscribeFileChangeEventsAsync(IVsAsyncFileChangeEx service, FileWatchingToken typedToken)
                 => service.UnadviseFileChangeAsync(typedToken.Cookie!.Value);
 
             public event EventHandler<string>? FileChanged;

diff --git a/...tion/ProjectSystem/MetadataReferences/VisualStudioFrameworkAssemblyPathResolverFactory.cs b/...tion/ProjectSystem/MetadataReferences/VisualStudioFrameworkAssemblyPathResolverFactory.cs
@@ -60,7 +60,7 @@ public Service(IThreadingContext threadingContext, VisualStudioWorkspace? worksp
                     // full URI format (i.e. file://c:/...). This will allow us to get the 
                     // actual local in the normal path format.
                     if (Uri.TryCreate(assembly.CodeBase, UriKind.RelativeOrAbsolute, out var uri) &&
-                        this.CanResolveType(assembly, fullyQualifiedTypeName))
+                        CanResolveType(assembly, fullyQualifiedTypeName))
                     {
                         return uri.LocalPath;
                     }
@@ -69,7 +69,7 @@ public Service(IThreadingContext threadingContext, VisualStudioWorkspace? worksp
                 return null;
             }
 
-            private bool CanResolveType(Assembly assembly, string? fullyQualifiedTypeName)
+            private static bool CanResolveType(Assembly assembly, string? fullyQualifiedTypeName)
             {
                 if (fullyQualifiedTypeName == null)
                 {

diff --git a/...f/Implementation/ProjectSystem/MetadataReferences/VisualStudioMetadataReferenceManager.cs b/...f/Implementation/ProjectSystem/MetadataReferences/VisualStudioMetadataReferenceManager.cs
@@ -215,7 +215,7 @@ private void GetStorageInfoFromTemporaryStorage(FileKey moduleFileKey, out ITemp
             pImage = directAccess.GetPointer();
         }
 
-        private void StreamCopy(Stream source, Stream destination, int start, int length)
+        private static void StreamCopy(Stream source, Stream destination, int start, int length)
         {
             source.Position = start;
 
@@ -311,7 +311,7 @@ private bool TryGetFileMappingFromMetadataImporter(FileKey fileKey, [NotNullWhen
 
         /// <exception cref="IOException"/>
         /// <exception cref="BadImageFormatException" />
-        private AssemblyMetadata CreateAssemblyMetadata(
+        private static AssemblyMetadata CreateAssemblyMetadata(
             FileKey fileKey, ModuleMetadata manifestModule, List<ITemporaryStreamStorage>? storages,
             Func<FileKey, List<ITemporaryStreamStorage>?, ModuleMetadata> moduleMetadataFactory)
         {