dotnet/roslyn

Add predicate version of IOperation.Descendants

Aperta

#53.621 aperta il 22 mag 2021

 (1 commento) (0 reazioni) (1 assegnatario)C# (4257 fork)batch import
Area-CompilersConcept-APIFeature - IOperationFeature Requestapi-approvedhelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

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.

Guida contributor