dotnet/roslyn

Refactor `CheckFeatureAvailability` to not accept a `Location` parameter

Aberta

#68.935 aberto em 8 de jul. de 2023

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)C# (4.257 forks)batch import
Area-CompilersArea-Performancehelp wanted

Métricas do repositório

Stars
 (20.414 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Currently, CheckFeatureAvailability overloads take a Location parameter. Causing caller to call syntaxNode.Location which allocates a SourceLocation. At the end, if the feature is available, the allocation is thrown away.

Example:

https://github.com/dotnet/roslyn/blob/b203101cbc6672a9761a7ce4917d26be593aed26/src/Compilers/CSharp/Portable/Binder/Binder_Patterns.cs#L574

Not sure how much this pattern is hit, but it's good to optimize this anyway.

Guia do colaborador

Direção de pesquisa
Encontre todas as sobrecargas de CheckFeatureAvailability que aceitam um parâmetro Location, analise seus chamadores para entender como remover o parâmetro e atualize os chamadores adequadamente.
Pilha de tecnologia
csharp
Domain
compilersbackend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70