dotnet/roslyn

Only Left and returnValue gets checked for dynamic in signature

Open

#69.215 aberto em 25 de jul. de 2023

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C# (4.257 forks)batch import
Area-CompilersBughelp wanted

Métricas do repositório

Stars
 (20.414 stars)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (256 fundiu PRs em 30d)

Description

[jcouv:] the linked PR makes it clearer what the problem is. It just needs a test to verify the impact and fix.

Version Used: Main branch

Steps to Reproduce:

  1. I ran a roslyn analyser to check for identical subexpressions on this repo and encountered a diagnostic on this line.
  2. I checked to code if this was a correct diagnostic and to me it seemed like a correct diagnostic

Diagnostic Id: IdenticalSubexpression Expected Behavior: No diagnostic from an identical subexpression subexpression Actual Behavior:

I get a diagnostic from an identical subexpression subexpression on this line

Guia do colaborador

Direção de pesquisa
Investigue o analisador IdenticalSubexpression no código fonte do Roslyn, particularmente a lógica que verifica expressões dinâmicas em assinaturas. Crie um caso de teste de reprodução mínimo que acione o falso positivo e, em seguida, rastreie o caminho do código para entender por que o diagnóstico é emitido incorretamente.
Pilha de tecnologia
csharp
Domain
compilers
Tipo Issue
Bug
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.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Precisa de investigação
Prerequisites
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40