dotnet/roslyn

Add predicate version of IOperation.Descendants

Ouverte

#53 621 ouverte le 22 mai 2021

 (1 commentaire) (0 réaction) (1 personne assignée)C# (4 257 forks)batch import
Area-CompilersConcept-APIFeature - IOperationFeature Requestapi-approvedhelp wanted

Métriques du dépôt

Stars
 (20 414 étoiles)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

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.

Guide contributeur