Skip to content

Instantly share code, notes, and snippets.

@jaredpar
Created April 3, 2026 23:29
Show Gist options
  • Select an option

  • Save jaredpar/9325dd8f3088f9873ecaebf311a3c718 to your computer and use it in GitHub Desktop.

Select an option

Save jaredpar/9325dd8f3088f9873ecaebf311a3c718 to your computer and use it in GitHub Desktop.
Intentional no-action comments in Roslyn codebase

Intentional No-Action Comments in Roslyn

Comments across C# and VB files where developers explicitly indicated no action is taken.


C# Files

Compilers

File Line Comment
Compilers\Shared\ConsoleUtil.cs 36 // Nothing to do if we can't reset the console.
Compilers\Core\Portable\Text\SourceText.cs 1298 // do nothing
Compilers\Core\Portable\Text\SourceText.cs 1303 // do nothing
Compilers\Core\Portable\PEWriter\FullMetadataWriter.cs 321 // noop
Compilers\CSharp\Portable\CodeGen\EmitAddress.cs 40 // do nothing receiver ref must be already pushed
Compilers\CSharp\Portable\CodeGen\EmitAddress.cs 276 // do nothing, ref should be on the stack
Compilers\CSharp\Portable\CodeGen\EmitExpression.cs 1418 // do nothing. Unused local load has no side-effects.
Compilers\CSharp\Portable\Binder\SwitchBinder.cs 215 // Nothing to do at this point. The label will be bound later.
Compilers\CSharp\Portable\Binder\DecisionDagBuilder.cs 958 // nothing to do, as there are no tests for the zero elements of this tuple
Compilers\CSharp\Portable\Lowering\StateMachineRewriter\MethodToStateMachineRewriter.cs 804 // do nothing

Workspaces

File Line Comment
Workspaces\Core\Portable\Storage\SQLite\v2\SQLitePersistentStorage_SolutionSerialization.cs 43 // nothing to do. A solution row just needs the id of the data-name (which the caller handles).
Workspaces\Core\Portable\Workspace\Solution\SolutionCompilationState.cs 1528 // Nothing to do if already at this version.
Workspaces\Core\Portable\FindSymbols\FindReferences\FindReferencesSearchEngine.NonCascadingSymbolSet.cs 26 // Nothing to do here.
Workspaces\SharedUtilitiesAndExtensions\Compiler\Core\Log\EmptyLogBlock.cs 10 // no op log block
Workspaces\SharedUtilitiesAndExtensions\Compiler\Core\Utilities\SerializableBytes.cs 119 // nothing to do, this is a read-only stream

Features

File Line Comment
Features\Core\Portable\Wrapping\SeparatedSyntaxList\AbstractSeparatedSyntaxListWrapper.cs 72 // nothing to do with 0-1 items. Simple enough for users to just edit
Features\Core\Portable\SymbolSearch\Windows\SymbolSearchUpdateEngine.cs 72 // Nothing to do for the core symbol search engine.
Features\Core\Portable\SymbolSearch\SymbolSearchUpdateNoOpEngine.cs 18 // Nothing to do for the no-op version.
Features\CSharp\Portable\SolutionExplorer\CSharpSolutionExplorerSymbolTreeItemProvider.cs 498 // nothing to do here.
Features\CSharp\Portable\EditAndContinue\SyntaxComparer.cs 1347 // Do nothing for node that cannot have variable declarations inside.
Features\Core\Portable\Common\StartInlineRenameSessionOperation.cs 27 // Intentionally empty. Handling of this operation is special cased in CodeActionEditHandlerService.cs
Features\Core\Portable\Common\DocumentNavigationOperation.cs 27 // Intentionally empty. Handling of this operation is special cased in CodeActionEditHandlerService.cs
Features\Core\Portable\Completion\CompletionService_GetCompletions.cs 99 // Nothing to do if we didn't even get any regular items back
Features\Core\Portable\Contracts\EditAndContinue\IManagedHotReloadService.cs 47 // No-op if no debugger is present.
Features\Core\Portable\CodeRefactorings\SyncNamespace\AbstractSyncNamespaceCodeRefactoringProvider.State.cs 130 // No action required if namespace already matches folders.
Features\Core\Portable\CodeRefactorings\SyncNamespace\AbstractChangeNamespaceService.cs 207 // No action required if declared namespace already matches target.
Features\Core\Portable\ExtractMethod\MethodExtractor.VariableSymbol.cs 132 // do nothing for parameter
Features\Core\Portable\FindUsages\AbstractFindUsagesService.ProgressAdapter.cs 76 // Do nothing functions.
Features\Core\Portable\ExternalAccess\UnitTesting\SolutionCrawler\UnitTestingWorkCoordinator.UnitTestingAsyncWorkItemQueue.cs 165 // nothing to do
Features\Core\Portable\EmbeddedLanguages\Json\LanguageServices\JsonClassifier.cs 124 // nothing to do.
Features\Core\Portable\UnusedReferences\UpdateAction.cs 10 // No action needs to be performed.

EditorFeatures

File Line Comment
EditorFeatures\Core\Classification\TotalClassificationTaggerProvider.cs 219 // nothing to do if we didn't run into any string literals.
EditorFeatures\Core\BackgroundWorkIndicator\BackgroundWorkIndicatorScope.cs 73 // Nothing to do if the actual value didn't change.
EditorFeatures\Core\BackgroundWorkIndicator\BackgroundWorkIndicatorScope.cs 89 // Nothing to do if the actual value didn't change.
EditorFeatures\Core\LanguageServer\AlwaysActiveLanguageClientEventListener.cs 66 // Nothing to do here. There's no concept of unloading an ILanguageClient.
EditorFeatures\Core\QuickInfo\LazyToolTip.cs 70 // Do nothing, since this is a WPF event handler and propagating the exception would cause a crash
EditorFeatures\Core\QuickInfo\LazyToolTip.cs 90 // Do nothing, since this is a WPF event handler and propagating the exception would cause a crash
EditorFeatures\Core\Interactive\InteractiveGlobalUndoServiceFactory.cs 78 // Nothing to do.
EditorFeatures\Core\Suggestions\SuggestedActions\EditorSuggestedAction.cs 235 // do nothing

VisualStudio

File Line Comment
VisualStudio\VisualStudioDiagnosticsToolWindow\PerfMargin\PerfEventActivityLogger.cs 30 // do nothing
VisualStudio\Core\Def\Diagnostics\VisualStudioDiagnosticAnalyzerProvider.WorkspaceEventListener.cs 49 // Nothing to do here. We already kicked off the work to initialize the workspace.
VisualStudio\Core\Def\CallHierarchy\CallHierarchyItem.cs 155 // Do nothing.
VisualStudio\CSharp\Impl\ProjectSystemShim\CSharpProjectShim.ICSInputSet.cs 21 // Nothing to do here. We watch addition/removal of source files via the ICSharpProjectSite methods.
VisualStudio\CSharp\Impl\ProjectSystemShim\CSharpProjectShim.ICSInputSet.cs 26 // Nothing to do here. We watch addition/removal of source files via the ICSharpProjectSite methods.
VisualStudio\Core\Def\FindReferences\Contexts\WithoutReferencesFindUsagesContext.cs 48 // Nothing to do on completion.
VisualStudio\Core\Def\Workspace\VisualStudioTextUndoHistoryWorkspaceServiceFactory.cs 68 // Nothing to do in this case: textBuffer is correct!
VisualStudio\Core\Def\Workspace\GlobalUndoServiceFactory.NoOpUndoPrimitive.cs 14 // no op undo primitive
VisualStudio\Core\Def\Packaging\PackageInstallerServiceFactory.cs 433 // Nothing to do for any other events.
VisualStudio\DevKit\Impl\SourceLink\VSCodePdbSourceDocumentLogger.cs 22 // Do nothing, we just leave all the logs up.

LanguageServer

File Line Comment
LanguageServer\Protocol\Handler\Symbols\WorkspaceSymbolsHandler.cs 109 // do nothing, we already await the SearchAsync method which calls this in a finally right before returning.
LanguageServer\Protocol\Handler\Symbols\WorkspaceSymbolsHandler.cs 115 // do nothing, LSP doesn't support reporting progress towards completion.

Dependencies

File Line Comment
Dependencies\Collections\Segmented\ImmutableSegmentedHashSet\1.cs` 161 // Intentionally empty
Dependencies\Collections\Segmented\ImmutableSegmentedHashSet\1+ValueBuilder.cs` 136 // No action required
Dependencies\Collections\Internal\ArraySortHelper.cs 213 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 218 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 497 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 502 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 509 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 514 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 804 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 809 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 1044 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 1049 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 1056 // Intentionally empty
Dependencies\Collections\Internal\ArraySortHelper.cs 1061 // Intentionally empty

Other C# Files

File Line Comment
Scripting\Core\Hosting\ObjectFormatter\ObjectFormatterHelpers.cs 126 // no-op, ignore proxy if it is implemented incorrectly or can't be loaded
RoslynAnalyzers\Utilities\FlowAnalysis\...\PropertySetDataFlowOperationVisitor.cs 267 // Do nothing.
RoslynAnalyzers\Utilities\FlowAnalysis\...\PropertySetDataFlowOperationVisitor.cs 436 // Do nothing.
Tools\BuildValidator\Program.cs 136 // no-op
Analyzers\CSharp\Tests\UseSimpleUsingStatement\UseSimpleUsingStatementTests.cs 1529 // intentionally empty
Analyzers\CSharp\Tests\UseSimpleUsingStatement\UseSimpleUsingStatementTests.cs 1541 // intentionally empty
Analyzers\CSharp\Tests\UpdateProjectToAllowUnsafe\UpdateProjectToAllowUnsafeTests.cs 42 // no action offered if unsafe was already enabled

VB Files

VB Compiler

File Line Comment
Compilers\VisualBasic\Portable\CodeGen\EmitAddress.vb 58 ' do nothing, it should be on the stack
Compilers\VisualBasic\Portable\CodeGen\EmitAddress.vb 72 ' do nothing receiver ref must be already pushed
Compilers\VisualBasic\Portable\CodeGen\EmitExpression.vb 460 ' do nothing. Unused local load has no side-effects.
Compilers\VisualBasic\Portable\CodeGen\EmitConversion.vb 329 ' Do nothing, it is the same as identity.
Compilers\VisualBasic\Portable\CodeGen\Optimizer\StackScheduler.Analyzer.vb 504 ' do nothing
Compilers\VisualBasic\Portable\Scanner\Scanner.vb 2042 ' nothing to do
Compilers\VisualBasic\Portable\Declarations\MergedNamespaceDeclaration.vb 119 ' Do nothing
Compilers\VisualBasic\Portable\Semantics\OverloadResolution.vb 2099 ' Nothing to do.
Compilers\VisualBasic\Portable\Semantics\Operators.vb 736 ' Nothing to do
Compilers\VisualBasic\Portable\Semantics\Operators.vb 764 ' Nothing to do
Compilers\VisualBasic\Portable\Semantics\Operators.vb 789 ' Nothing to do
Compilers\VisualBasic\Portable\Binding\Binder_Invocation.vb 1014 ' Nothing to do
Compilers\VisualBasic\Portable\Binding\Binder_Expressions.vb 1392 ' Nothing to do.
Compilers\VisualBasic\Portable\Binding\Binder_Expressions.vb 1411 ' Nothing to do.
Compilers\VisualBasic\Portable\Binding\BackstopBinder.vb 204 ' No action needed.
Compilers\VisualBasic\Portable\Binding\BackstopBinder.vb 263 ' do nothing
Compilers\VisualBasic\Portable\Binding\BackstopBinder.vb 267 ' do nothing
Compilers\VisualBasic\Portable\Analysis\ForLoopVerification.vb 97 ' do nothing if the array is nothing
Compilers\VisualBasic\Portable\Analysis\FlowAnalysis\EntryPointsWalker.vb 54 ' Do nothing
Compilers\VisualBasic\Portable\Compilation\DocumentationComments\UnprocessedDocumentationCommentFinder.vb 161 ' Do nothing, the comment is properly located
Compilers\VisualBasic\Portable\CommandLine\VisualBasicCommandLineParser.vb 638 ' Do nothing as we no longer have any use for implementing this switch
Compilers\VisualBasic\Portable\CommandLine\VisualBasicCommandLineParser.vb 1087 ' Do nothing as we no longer have any use for implementing this switch
Compilers\VisualBasic\Portable\Symbols\EventSymbol.vb 226 ' Nothing to do, simply use the same error info.
Compilers\VisualBasic\Portable\Symbols\Symbol.vb 1001 ' Nothing to do, simply use the same error info.

VB Workspaces & Tools

File Line Comment
Workspaces\VisualBasic\Portable\FindSymbols\VisualBasicDeclaredSymbolInfoFactoryService.vb 136 ' Nothing to do in VB.
Workspaces\SharedUtilitiesAndExtensions\Workspace\VisualBasic\LanguageServices\InitializeParameter\VisualBasicInitializeParameterService.vb 54 ' Nothing to do in VB.
Tools\Source\CompilerGeneratorTools\Source\VisualBasicSyntaxGenerator\RedNodes\RedNodeWriter.vb 955 ' do nothing for abstract nodes
Tools\Source\CompilerGeneratorTools\Source\VisualBasicSyntaxGenerator\GreenNodes\GreenNodeWriter.vb 657 ' do nothing for abstract nodes

VB Tests

File Line Comment
VisualStudio\TestUtilities2\CodeModel\Mocks\MockVisualStudioWorkspace.vb 85 ' Nothing to do here
VisualStudio\Core\Test\Diagnostics\ExternalDiagnosticUpdateSourceTests.vb 256 ' do nothing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment