dotnet/roslyn

Add predicate version of IOperation.Descendants

Offen

#53.621 geöffnet am 22.05.2021

 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)C# (4.257 Forks)batch import
Area-CompilersConcept-APIFeature - IOperationFeature Requestapi-approvedhelp wanted

Repository-Metriken

Stars
 (20.414 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (256 gemergte PRs in 30 T)

Beschreibung

Background and Motivation

Today, we have a SyntaxNode helper DescendantsNodes that takes a predicate descendIntoChildren, which allows users to cut off enumeration if needed. We don't have a similar method for IOperation: Descendants does not take a predicate that can control this. We should add a companion method for IOperation here.

Proposed API

namespace Microsoft.CodeAnalysis.Operations
{
    public static class OperationExtensions
    {
+       public static IEnumerable<IOperation> Descendants(this IOperation? operation, Func<IOperation, bool> descendIntoChildren);
+       public static IEnumerable<IOperation> DescendantsAndSelf(this IOperation? operation, Func<IOperation, bool> descendIntoChildren);
    }

Usage examples

Where I need this currently is to express something like operation.Descendants(descendIntoChildren: static node => node is not (IAnonymousFunctionOperation or ILocalFunctionOperation)), where I need to cut off enumeration into lambdas or local functions.

Alternatives

N/A

Risks

Ideally, we'd name it DescendantNodes for consistency with syntax, but that ship sailed long ago.

Contributor Guide